Skip to content

[FLINK-39293][table] MATCH_RECOGNIZE fails with SqlParserException in views#28015

Open
snuyanzin wants to merge 1 commit intoapache:masterfrom
snuyanzin:flink39293
Open

[FLINK-39293][table] MATCH_RECOGNIZE fails with SqlParserException in views#28015
snuyanzin wants to merge 1 commit intoapache:masterfrom
snuyanzin:flink39293

Conversation

@snuyanzin
Copy link
Copy Markdown
Contributor

@snuyanzin snuyanzin commented Apr 23, 2026

What is the purpose of the change

In FLINK-38493 there was change towards unparse functionality using for expanded query

It ports Calcite fixes
https://issues.apache.org/jira/browse/CALCITE-7465
https://issues.apache.org/jira/browse/CALCITE-7466
https://issues.apache.org/jira/browse/CALCITE-7467
https://issues.apache.org/jira/browse/CALCITE-7470
https://issues.apache.org/jira/browse/CALCITE-7471
https://issues.apache.org/jira/browse/CALCITE-7480
https://issues.apache.org/jira/browse/CALCITE-7486

Verifying this change

MatchRecognizeITCase.java

Does this pull request potentially affect one of the following parts:

  • Dependencies (does it add or upgrade a dependency): ( no)
  • The public API, i.e., is any changed class annotated with @Public(Evolving): ( no)
  • The serializers: (no)
  • The runtime per-record code paths (performance sensitive): (no)
  • Anything that affects deployment or recovery: JobManager (and its components), Checkpointing, Kubernetes/Yarn, ZooKeeper: (no )
  • The S3 file system connector: (no )

Documentation

  • Does this pull request introduce a new feature? (no)
  • If yes, how is the feature documented? (not applicable )

Was generative AI tooling used to co-author this PR?
  • Yes (please specify the tool below)

@flinkbot
Copy link
Copy Markdown
Collaborator

flinkbot commented Apr 23, 2026

CI report:

Bot commands The @flinkbot bot supports the following commands:
  • @flinkbot run azure re-run the last Azure build

@snuyanzin
Copy link
Copy Markdown
Contributor Author

@flinkbot run azure

Copy link
Copy Markdown
Contributor

@raminqaf raminqaf left a comment

Choose a reason for hiding this comment

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

Thanks for the back porting @snuyanzin! LGTM!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

community-reviewed PR has been reviewed by the community.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants