Skip to content

Conversation

@kyleconroy
Copy link
Collaborator

  • Add Files and MirrorToClauses to BackupDatabaseStatement
  • Parse READ_WRITE_FILEGROUPS and MIRROR TO clauses in BACKUP DATABASE
  • Add STATS_STREAM option to UPDATE STATISTICS WITH clause
  • Support ADD/DROP PERSISTED in ALTER TABLE ALTER COLUMN
  • Fix TOP clause parsing to handle parenthesized query expressions with EXCEPT/UNION

claude added 8 commits January 2, 2026 07:34
- Add Files and MirrorToClauses to BackupDatabaseStatement
- Parse READ_WRITE_FILEGROUPS and MIRROR TO clauses in BACKUP DATABASE
- Add STATS_STREAM option to UPDATE STATISTICS WITH clause
- Support ADD/DROP PERSISTED in ALTER TABLE ALTER COLUMN
- Fix TOP clause parsing to handle parenthesized query expressions with EXCEPT/UNION
- Parse EXECUTE AS with string literals (e.g., EXECUTE AS 'dbo')
- Parse NOT FOR REPLICATION in triggers
- Parse EXTERNAL NAME for CLR triggers (MethodSpecifier)
- Convert DDL event types properly (DENY_DATABASE -> DenyDatabase)
- Fix ExecuteAsTriggerOption JSON marshaling to include Literal field
- Add DropClusteredConstraintWaitAtLowPriorityLockOption AST type
- Parse WAIT_AT_LOW_PRIORITY (MAX_DURATION, ABORT_AFTER_WAIT) options
- Add JSON marshaling for low priority lock wait options
- Enable AlterTableDropTableElementStatementTests140 tests
- Make MINUTES/SECONDS optional in parseWaitAtLowPriorityOption
- Only output Unit field in JSON when it's not empty
- Enable AlterTableSwitchStatementTests120 tests
- Add AlterDatabaseCollateStatement AST type and parsing
- Add FileDeclaration parsing to AlterDatabaseModifyFileStatement
- Update JSON marshaling to include UseCurrent field and proper field ordering
- Handle SCOPED as database name when not followed by CREDENTIAL/CONFIGURATION
- Mark PhaseOne tests with outdated JSON format as todo (need ScriptDom regeneration)

Enabled tests: AlterDatabaseStatementTests, BaselinesCommon_AlterDatabaseStatementTests
- Add DistinctPredicate AST type for comparing values with NULL handling
- Parse IS [NOT] DISTINCT FROM in boolean expressions
- Special case: IS [NOT] DISTINCT FROM NULL converts to IS [NOT] NULL
- Add JSON marshaling for DistinctPredicate

Note: MergeStatementTests160 remains todo due to pre-existing MERGE subquery issue
… marshalers

- Added NextValueForExpression AST type for NEXT VALUE FOR sequence syntax
- Added parseNextValueForExpression() with OVER clause support
- Added JSON marshaling for NextValueForExpression
- Fixed field ordering in ALTER DATABASE statement marshalers
- Added UseCurrent field to AlterDatabaseModifyFileStatement struct
- Fixed parseMergeSourceTableReference to handle derived table subqueries
- Added TableAlias field to MergeSpecification for target table alias
- Move target alias to MergeSpecification.TableAlias per ScriptDOM convention
- Enabled MergeStatementTests160 and Baselines160_MergeStatementTests160 tests
@kyleconroy kyleconroy merged commit e5e310d into main Jan 2, 2026
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.

3 participants