Skip to content

[FLINK-27241][table] Support DROP PARTITIONS statement for partitioned table#22720

Merged
luoyuxia merged 4 commits into
apache:masterfrom
luoyuxia:FLINK-27241-drop-partition
Jun 21, 2023
Merged

[FLINK-27241][table] Support DROP PARTITIONS statement for partitioned table#22720
luoyuxia merged 4 commits into
apache:masterfrom
luoyuxia:FLINK-27241-drop-partition

Conversation

@luoyuxia

@luoyuxia luoyuxia commented Jun 6, 2023

Copy link
Copy Markdown
Contributor

What is the purpose of the change

To make Flink support DROP partition statement for partitioned table.

Brief change log

  • support to parse alter table drop partition syntax which will convert the sql statement to a SqlAddPartitions
  • create a SqlAlterTableDropPartitionConverter to convert the SqlDropPartitions to DropPartitionsOperation.

Verifying this change

UT

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

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

Documentation

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

@flinkbot

flinkbot commented Jun 6, 2023

Copy link
Copy Markdown
Collaborator

CI report:

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

@ruanhang1993 ruanhang1993 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.

LGTM.

@luoyuxia luoyuxia force-pushed the FLINK-27241-drop-partition branch from c81aa66 to 8cd8762 Compare June 20, 2023 12:17
@luoyuxia luoyuxia changed the title [FLINK-27241][table] Support DROP PARTITION statement for partitioned table [FLINK-27241][table] Support DROP PARTITIONS statement for partitioned table Jun 21, 2023
@luoyuxia

Copy link
Copy Markdown
Contributor Author

@ruanhang1993 Thanks for reviewing. Merging...

@luoyuxia luoyuxia merged commit f1233a3 into apache:master Jun 21, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants