Skip to content

Add parser features and enable 8 more tests#17

Merged
kyleconroy merged 1 commit intomainfrom
claude/fix-skipped-parser-tests-bZk0N
Dec 18, 2025
Merged

Add parser features and enable 8 more tests#17
kyleconroy merged 1 commit intomainfrom
claude/fix-skipped-parser-tests-bZk0N

Conversation

@kyleconroy
Copy link
Copy Markdown
Collaborator

Features added:

  • SAVE TRANSACTION legacy name format support (e.g., "5:a.b", "-100:[a].[b]")
  • DECLARE TABLE variable support with full table definition parsing
  • IncludeColumns marshaling for IndexDefinition
  • SetVariableStatement property/method syntax (dot and double-colon separators)
  • DROP EXTERNAL MODEL statement support
  • ALTER SERVER CONFIGURATION SET PROCESS AFFINITY support

Tests enabled:

  • SaveTransactionStatementTests
  • BaselinesCommon_SaveTransactionStatementTests
  • DeclareTableStatementTests
  • BaselinesCommon_DeclareTableStatementTests
  • Baselines150_DeclareTableVariableTests150
  • DeclareTableVariableTests150
  • AlterServerConfigurationStatementTests
  • Baselines100_AlterServerConfigurationStatementTests

Features added:
- SAVE TRANSACTION legacy name format support (e.g., "5:a.b", "-100:[a].[b]")
- DECLARE TABLE variable support with full table definition parsing
- IncludeColumns marshaling for IndexDefinition
- SetVariableStatement property/method syntax (dot and double-colon separators)
- DROP EXTERNAL MODEL statement support
- ALTER SERVER CONFIGURATION SET PROCESS AFFINITY support

Tests enabled:
- SaveTransactionStatementTests
- BaselinesCommon_SaveTransactionStatementTests
- DeclareTableStatementTests
- BaselinesCommon_DeclareTableStatementTests
- Baselines150_DeclareTableVariableTests150
- DeclareTableVariableTests150
- AlterServerConfigurationStatementTests
- Baselines100_AlterServerConfigurationStatementTests
@kyleconroy kyleconroy merged commit 5df3fae into main Dec 18, 2025
1 check passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants