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.

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

[FLINK-28298][table][python] Support left and right in Table API #20104

Merged
merged 1 commit into from
Jul 4, 2022

Conversation

a49a
Copy link
Contributor

@a49a a49a commented Jun 29, 2022

What is the purpose of the change

Support decode and encode built-in functions in Table API.

Brief change log

  • Support left and right built-in functions in Table API.
  • Support left and right built-in functions in Python Table API.

Verifying this change

This change added tests and can be verified as follows:

  • Added test that validates that left and right in Table API work
  • Added test that validates that left and right in Python Table API work

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): (yes)
  • 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? (yes)
  • If yes, how is the feature documented? (docs)

@a49a a49a marked this pull request as draft June 29, 2022 13:03
@flinkbot
Copy link
Collaborator

flinkbot commented Jun 29, 2022

CI report:

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

@a49a a49a force-pushed the bif-left-right branch 2 times, most recently from 559dfef to 045a28e Compare July 1, 2022 05:41
@a49a a49a marked this pull request as ready for review July 1, 2022 08:50
@dianfu dianfu merged commit da720c3 into apache:master Jul 4, 2022
liujiawinds pushed a commit to liujiawinds/flink that referenced this pull request Jul 22, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
3 participants