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

[GLUTEN-2777][CH]Bug fix get_json_object parse json error while path is number #2780

Merged

Conversation

KevinyhZou
Copy link
Contributor

@KevinyhZou KevinyhZou commented Aug 16, 2023

What changes were proposed in this pull request?

(Please fill in changes proposed in this fix)

(Fixes: #2777)

based on pr: ClickHouse/ClickHouse#53470 ClickHouse/ClickHouse#57096

How was this patch tested?

UT

(If this patch involves UI changes, please attach a screenshot; otherwise, remove this)

@github-actions
Copy link

#2777

@github-actions
Copy link

Run Gluten Clickhouse CI

1 similar comment
@github-actions
Copy link

Run Gluten Clickhouse CI

@KevinyhZou KevinyhZou force-pushed the bug_fix_get_json_object_number_path branch from 8a3e45f to 54e25e0 Compare September 28, 2023 02:55
@github-actions
Copy link

Run Gluten Clickhouse CI

@github-actions
Copy link

Run Gluten Clickhouse CI

@github-actions
Copy link

github-actions bot commented Oct 7, 2023

Run Gluten Clickhouse CI

Copy link

This PR is stale because it has been open 45 days with no activity. Remove stale label or comment or this will be closed in 10 days.

@github-actions github-actions bot added the stale stale label Nov 22, 2023
Copy link

Run Gluten Clickhouse CI

Copy link

Run Gluten Clickhouse CI

Copy link

Run Gluten Clickhouse CI

@github-actions github-actions bot removed the stale stale label Nov 23, 2023
Copy link

github-actions bot commented Dec 6, 2023

Run Gluten Clickhouse CI

1 similar comment
Copy link

github-actions bot commented Dec 6, 2023

Run Gluten Clickhouse CI

Copy link
Contributor

@liuneng1994 liuneng1994 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

@liuneng1994 liuneng1994 merged commit 3274a91 into apache:main Dec 12, 2023
7 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

[CH] get_json_object path parse error while path is '$.123'
2 participants