Skip to content

Add more test cases for json_query function #25650

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

Closed
wants to merge 1 commit into from

Conversation

XavieLee
Copy link
Member

Description

Add more test cases for json_query function to help developer understand the usage of this function.

Additional context and related issues

#25545

Release notes

(x) This is not user-visible or is docs only, and no release notes are required.
( ) Release notes are required. Please propose a release note for me.
( ) Release notes are required, with the following suggested text:

@cla-bot cla-bot bot added the cla-signed label Apr 23, 2025
@XavieLee XavieLee requested a review from kasiafi April 23, 2025 03:56
@XavieLee XavieLee self-assigned this Apr 23, 2025
@XavieLee XavieLee force-pushed the update/testcase/json_query branch from 7274be5 to 3e38d6a Compare April 23, 2025 04:43
@XavieLee XavieLee force-pushed the update/testcase/json_query branch from 3e38d6a to 121b953 Compare April 24, 2025 08:04
@XavieLee
Copy link
Member Author

@chenjian2664 updated.

@XavieLee XavieLee force-pushed the update/testcase/json_query branch 2 times, most recently from b6a2c8f to d6cde4b Compare April 24, 2025 10:52
@XavieLee XavieLee force-pushed the update/testcase/json_query branch from d6cde4b to dd49ec8 Compare May 5, 2025 06:34
@XavieLee XavieLee closed this by deleting the head repository May 13, 2025
@XavieLee
Copy link
Member Author

Hi @kasiafi , all fixed, can u help review it again in #25783 .

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Development

Successfully merging this pull request may close these issues.

3 participants