Kyligence / ClickHouse

ClickHouse® is a free analytics DBMS for big data
https://clickhouse.com
Apache License 2.0
14 stars 17 forks source link

[409] Improve compatibility of `get_json_object` #420

Open lgbo-ustc opened 1 year ago

lgbo-ustc commented 1 year ago

Changelog category (leave one):

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

...

Documentation entry for user-facing changes

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

closed #409

kyligence-git commented 1 year ago

Can one of the admins verify this patch?

lgbo-ustc commented 1 year ago

test this please with 1333

liuneng1994 commented 1 year ago

对CH的修改提交给社区吗

lgbo-ustc commented 1 year ago

对CH的修改提交给社区吗

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

liuneng1994 commented 1 year ago

对CH的修改提交给社区吗

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

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

lgbo-ustc commented 1 year ago

对CH的修改提交给社区吗

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

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

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