T-SQL Alter Table about tree-sitter-sql

[esetnik]

Ok in that case we should probably reopen this one as well since there appear to be two separate issues in the same statement. The issue I originally reported was fixed in taozhi8833998/node-sql-parser#1372. Thanks for your help! Happy to test out a potential solution.

[matthias-Q]

The SQL statement works fine for my:

Can you update the grammar on your machine?

[esetnik]

Hmm. I am using the latest parser from nvim-treesitter. I believe it is nvim-treesitter/nvim-treesitter@411e771. I'm not too familiar with how to update grammar directly. Can you clarify how I should do that?

[esetnik]

I apologize the error appears to be coming from https://github.com/fannheyward/coc-sql and not tree-sitter-sql

[matthias-Q]

OK, no problem. Upon further investigation, T-SQL seems to support to add multiple columns in one alter table statement (like in your screenshot).
This parse aims to be permissive and we mostly consider MySQL, MariaDB, PostgreSQL and a some Hive statements. These dialect do not support this style of altering tables.

If you want you can reopen it. I definately get an error for statements like:

alter table classname 
add
col1 varchar(255) not null default('empty'),
col2 varchar(255) not null default('empty2');

The SQL in nvim-treesitter gets updated when you update the plugin. :TSUpdate should also work.