Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

[FLINK-37618][table-planner] Fix PTFs INTERVAL argument #26410

Open
wants to merge 4 commits into
base: master
Choose a base branch
from
Open

[FLINK-37618][table-planner] Fix PTFs INTERVAL argument #26410

wants to merge 4 commits into from
+134 −0

Conversation

juntaozhang
Copy link
Contributor

@juntaozhang juntaozhang commented Apr 6, 2025 •
edited
Loading

What is the purpose of the change

Support INTERVAL argument in PTFs
see details in https://issues.apache.org/jira/browse/FLINK-37618

Brief change log

add visit parameter of DayTimeIntervalType in CastAvoidanceChecker

Verifying this change

  • Added PROCESS_INTERVAL_ARGS in ProcessTableFunctionSemanticTests
  • Added testIntervalArgs in ProcessTableFunctionTest

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)

@flinkbot
Copy link
Collaborator

flinkbot commented Apr 6, 2025 •
edited
Loading

CI report:

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

@juntaozhang juntaozhang force-pushed the fix-PTF-interval-arg branch from ab4b7f2 to 5bc8d20 Compare April 12, 2025 06:44
@juntaozhang juntaozhang force-pushed the fix-PTF-interval-arg branch from 5bc8d20 to 9df64f3 Compare April 12, 2025 06:48
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@davidradl davidradl davidradl left review comments

Assignees
No one assigned
Labels
component=TableSQL/Planner
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
3 participants
@juntaozhang @flinkbot @davidradl