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

[409] Improve compatibility of get_json_object #420

Open
wants to merge 761 commits into
base: clickhouse_backend
Choose a base branch
from

Conversation

lgbo-ustc
Copy link

Changelog category (leave one):

  • Improvement

Changelog entry (a user-readable short description of the changes that goes to CHANGELOG.md):

...

Documentation entry for user-facing changes

  • Documentation is written (mandatory for new features)

Information about CI checks: https://clickhouse.com/docs/en/development/continuous-integration/

closed #409

kitaisreal and others added 30 commits April 4, 2022 13:41
Backport ClickHouse#35770 to 22.3: Fix enable LLVM for JIT compilation in CMake
…ash_v2

One more try to resurrect build hash
Backport ClickHouse#35733 to 22.3: Added settings for insert of invalid IPv6, IPv4 values
Backport ClickHouse#35820 to 22.3: Avoid processing per-column TTL multiple times
- Allow define version as file
- Add inline cache
- Fix auto_release_type function
@kyligence-git
Copy link
Collaborator

Can one of the admins verify this patch?

@lgbo-ustc
Copy link
Author

test this please with 1333

@lgbo-ustc lgbo-ustc marked this pull request as ready for review April 11, 2023 12:21
@liuneng1994
Copy link
Collaborator

liuneng1994 commented Apr 13, 2023

对CH的修改提交给社区吗

@lgbo-ustc
Copy link
Author

对CH的修改提交给社区吗

晚点我再提个pr,估计有点难合并进去。标准sql的相关函数对jsonpath的支持也是很简单的

@liuneng1994
Copy link
Collaborator

对CH的修改提交给社区吗

晚点我再提个pr,估计有点难合并进去。标准sql的相关函数对jsonpath的支持也是很简单的

如果社区不接受的话,把这个函数实现挪出来合适吗,尽量避免大段CH源码的改动

@lgbo-ustc
Copy link
Author

对CH的修改提交给社区吗

晚点我再提个pr,估计有点难合并进去。标准sql的相关函数对jsonpath的支持也是很简单的

如果社区不接受的话,把这个函数实现挪出来合适吗,尽量避免大段CH源码的改动

主要是ck社区对这个没有需求。这些path格式在我们线上大量使用到了

@lwz9103 lwz9103 force-pushed the clickhouse_backend branch 2 times, most recently from dc60d55 to 8066113 Compare May 26, 2023 03:43
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.

get_json_object doesn't support field name which with number begin