Skip to content

[FLINK-29096][table] Add test for json_value, which json path has bla…#20675

Closed
liuyongvs wants to merge 1 commit intoapache:masterfrom
liuyongvs:add_test_for_json_value
Closed

[FLINK-29096][table] Add test for json_value, which json path has bla…#20675
liuyongvs wants to merge 1 commit intoapache:masterfrom
liuyongvs:add_test_for_json_value

Conversation

@liuyongvs
Copy link
Copy Markdown
Contributor

What is the purpose of the change

Add test for json_value, which json path has blank characters

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: (yes / no / don't know)
  • 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)

@flinkbot
Copy link
Copy Markdown
Collaborator

flinkbot commented Aug 24, 2022

CI report:

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

@liuyongvs liuyongvs force-pushed the add_test_for_json_value branch 2 times, most recently from 2883800 to 1964550 Compare August 25, 2022 02:46
@liuyongvs liuyongvs force-pushed the add_test_for_json_value branch from 1964550 to 351cffe Compare August 25, 2022 11:51
@wuchong wuchong closed this in 60e5944 Aug 26, 2022
huangxiaofeng10047 pushed a commit to huangxiaofeng10047/flink that referenced this pull request Nov 3, 2022
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.

2 participants