Skip to content

Conversation

@vladimirg-db
Copy link
Contributor

@vladimirg-db vladimirg-db commented Oct 26, 2025

What changes were proposed in this pull request?

Add new golden file tests for analysis edge-cases discovered during Analyzer support and development.

Why are the changes needed?

Harden Spark testing coverage.

Does this PR introduce any user-facing change?

Test-only change.

How was this patch tested?

Adding new golden files.

Was this patch authored or co-authored using generative AI tooling?

No.

@github-actions github-actions bot added the SQL label Oct 26, 2025
@vladimirg-db vladimirg-db force-pushed the vladimir-golubev_data/new-golden-files-for-analyzer-edge-cases-2 branch from fe79bb1 to 0b32837 Compare October 26, 2025 16:47
Copy link
Contributor

@mihailotim-db mihailotim-db left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Would be great if we could somehow format all queries in a uniform way (LLM?), if it's not too tedious

@vladimirg-db vladimirg-db force-pushed the vladimir-golubev_data/new-golden-files-for-analyzer-edge-cases-2 branch 2 times, most recently from f9810da to ffdd613 Compare October 27, 2025 11:53
@vladimirg-db vladimirg-db force-pushed the vladimir-golubev_data/new-golden-files-for-analyzer-edge-cases-2 branch from ffdd613 to 7ff2649 Compare October 27, 2025 15:52
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants