Skip to content

[jdbc-v2] Fix parser grammar - #2433

Merged
chernser merged 2 commits into
mainfrom
jdbc_fix_2431
Jun 6, 2025
Merged

[jdbc-v2] Fix parser grammar#2433
chernser merged 2 commits into
mainfrom
jdbc_fix_2431

Conversation

@chernser

@chernser chernser commented Jun 5, 2025

Copy link
Copy Markdown
Contributor

Summary

Fixes parser grammar for:

  • SELECT INTERVAL '1 day' - when interval is define like literal
  • ( WITH .... ) - wrapped CTEs

Closes #2431
Closes #2429

Checklist

Delete items not relevant to your PR:

  • Closes #
  • Unit and integration tests covering the common scenarios were added
  • A human-readable description of the changes was provided to include in CHANGELOG
  • For significant changes, documentation in https://github.com/ClickHouse/clickhouse-docs was updated with further explanations or tutorials

@chernser
chernser requested a review from mzitnik June 5, 2025 20:43

@windsurf-bot windsurf-bot Bot left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

💡 To request another review, post a new comment with "/windsurf-review".

@sonarqubecloud

sonarqubecloud Bot commented Jun 5, 2025

Copy link
Copy Markdown

@chernser
chernser merged commit 052c3ab into main Jun 6, 2025
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.

[jdbc-v2] Antlr4 parser fails to parse mixed CTEs [jdbc-v2] Antlr4 parser throws org.antlr.v4.runtime.NoViableAltException for string intervals

2 participants