Re: [PR] Add support for `PRINT` statement for SQL Server [datafusion-sqlparser-rs]

2025-04-18 Thread via GitHub
iffyio merged PR #1811: URL: https://github.com/apache/datafusion-sqlparser-rs/pull/1811 -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comment. To unsubscribe, e-mail: github-unsubscr

Re: [PR] Add support for `PRINT` statement for SQL Server [datafusion-sqlparser-rs]

2025-04-16 Thread via GitHub
aharpervc commented on code in PR #1811: URL: https://github.com/apache/datafusion-sqlparser-rs/pull/1811#discussion_r2047549413 ## tests/sqlparser_mssql.rs: ## @@ -2053,3 +2053,37 @@ fn parse_drop_trigger() { } ); } + +#[test] +fn parse_print() { +let print_

Re: [PR] Add support for `PRINT` statement for SQL Server [datafusion-sqlparser-rs]

2025-04-16 Thread via GitHub
aharpervc commented on code in PR #1811: URL: https://github.com/apache/datafusion-sqlparser-rs/pull/1811#discussion_r2047240782 ## tests/sqlparser_mssql.rs: ## @@ -2053,3 +2053,37 @@ fn parse_drop_trigger() { } ); } + +#[test] +fn parse_print() { +let print_

Re: [PR] Add support for `PRINT` statement for SQL Server [datafusion-sqlparser-rs]

2025-04-15 Thread via GitHub
iffyio commented on code in PR #1811: URL: https://github.com/apache/datafusion-sqlparser-rs/pull/1811#discussion_r2046111411 ## tests/sqlparser_mssql.rs: ## @@ -2053,3 +2053,37 @@ fn parse_drop_trigger() { } ); } + +#[test] +fn parse_print() { +let print_str

Re: [PR] Add support for `PRINT` statement for SQL Server [datafusion-sqlparser-rs]

2025-04-15 Thread via GitHub
aharpervc commented on code in PR #1811: URL: https://github.com/apache/datafusion-sqlparser-rs/pull/1811#discussion_r2044897576 ## src/parser/mod.rs: ## @@ -617,6 +617,9 @@ impl<'a> Parser<'a> { } // `COMMENT` is snowflake specific https://do

Re: [PR] Add support for `PRINT` statement for SQL Server [datafusion-sqlparser-rs]

2025-04-15 Thread via GitHub
aharpervc commented on code in PR #1811: URL: https://github.com/apache/datafusion-sqlparser-rs/pull/1811#discussion_r2044896976 ## src/ast/mod.rs: ## @@ -4050,6 +4050,14 @@ pub enum Statement { arguments: Vec, options: Vec, }, +/// ```sql +/// PR

Re: [PR] Add support for `PRINT` statement for SQL Server [datafusion-sqlparser-rs]

2025-04-15 Thread via GitHub
aharpervc commented on PR #1811: URL: https://github.com/apache/datafusion-sqlparser-rs/pull/1811#issuecomment-2806527507 Rebased & updated -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the sp

Re: [PR] Add support for `PRINT` statement for SQL Server [datafusion-sqlparser-rs]

2025-04-15 Thread via GitHub
aharpervc commented on code in PR #1811: URL: https://github.com/apache/datafusion-sqlparser-rs/pull/1811#discussion_r2044896976 ## src/ast/mod.rs: ## @@ -4050,6 +4050,14 @@ pub enum Statement { arguments: Vec, options: Vec, }, +/// ```sql +/// PR

Re: [PR] Add support for `PRINT` statement for SQL Server [datafusion-sqlparser-rs]

2025-04-14 Thread via GitHub
iffyio commented on code in PR #1811: URL: https://github.com/apache/datafusion-sqlparser-rs/pull/1811#discussion_r2043654919 ## src/parser/mod.rs: ## @@ -617,6 +617,9 @@ impl<'a> Parser<'a> { } // `COMMENT` is snowflake specific https://docs.

Re: [PR] Add support for `PRINT` statement for SQL Server [datafusion-sqlparser-rs]

2025-04-14 Thread via GitHub
aharpervc commented on code in PR #1811: URL: https://github.com/apache/datafusion-sqlparser-rs/pull/1811#discussion_r2042340044 ## src/ast/mod.rs: ## @@ -4050,6 +4050,14 @@ pub enum Statement { arguments: Vec, options: Vec, }, +/// ```sql +/// PR

Re: [PR] Add support for `PRINT` statement for SQL Server [datafusion-sqlparser-rs]

2025-04-14 Thread via GitHub
aharpervc commented on code in PR #1811: URL: https://github.com/apache/datafusion-sqlparser-rs/pull/1811#discussion_r2042341589 ## tests/sqlparser_mssql.rs: ## @@ -2036,3 +2036,37 @@ fn parse_mssql_merge_with_output() { OUTPUT $action, deleted.ProductID INTO dsi.temp_

[PR] Add support for `PRINT` statement for SQL Server [datafusion-sqlparser-rs]

2025-04-11 Thread via GitHub
aharpervc opened a new pull request, #1811: URL: https://github.com/apache/datafusion-sqlparser-rs/pull/1811 Reference: https://learn.microsoft.com/en-us/sql/t-sql/language-elements/print-transact-sql?view=sql-server-ver16 Making `message` a `Box` instead of an enum of (national) stri