Skip to content

[FLINK-36843][table-planner] Remove deprecated method in table-planner#27822

Open
balassai wants to merge 1 commit intoapache:masterfrom
balassai:FLINK-36843
Open

[FLINK-36843][table-planner] Remove deprecated method in table-planner#27822
balassai wants to merge 1 commit intoapache:masterfrom
balassai:FLINK-36843

Conversation

@balassai
Copy link
Contributor

@balassai balassai commented Mar 25, 2026

What is the purpose of the change

Remove deprecated methods that annotated with Deprecated and commented with to be removed before 2.0 in flink-table-planner.

Brief change log

  • Removed methods marked as deprecated.

Verifying this change

This change removes methods. No new tests have been added, existing tests are run.

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)

@flinkbot
Copy link
Collaborator

flinkbot commented Mar 25, 2026

CI report:

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

Copy link
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 contribution. The CI is failing, we have to make sure that the removal of these methods don't affect any of the tests

@github-actions github-actions bot added the community-reviewed PR has been reviewed by the community. label Mar 25, 2026
@davidradl
Copy link
Contributor

@balassai Just to confirm, master will be used for the next version so we can safely remove deprecated methodscn you confirm that this will not be backported.

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.

4 participants