Skip to content
New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

修复clickhouse上线审核版本兼容性问题 #1428

Merged
merged 1 commit into from
Mar 29, 2022

Conversation

nick2wang
Copy link
Collaborator

fix #1426
clickhouse上线前语法检查使用explain ast,从v21.1.2.15版本起才支持检查非select语句,将explain_check的版本门槛提高到21.1.2

@codecov
Copy link

codecov bot commented Mar 28, 2022

Codecov Report

Merging #1428 (2f8377d) into master (8763fe2) will not change coverage.
The diff coverage is 100.00%.

@@           Coverage Diff           @@
##           master    #1428   +/-   ##
=======================================
  Coverage   74.86%   74.86%           
=======================================
  Files          82       82           
  Lines       13219    13219           
=======================================
  Hits         9897     9897           
  Misses       3322     3322           
Impacted Files Coverage Δ
sql/engines/clickhouse.py 68.67% <100.00%> (ø)

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 8763fe2...2f8377d. Read the comment docs.

Copy link
Collaborator

@LeoQuote LeoQuote left a comment

Choose a reason for hiding this comment

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

LGTM

@LeoQuote LeoQuote merged commit a8967be into hhyo:master Mar 29, 2022
@nick2wang nick2wang deleted the fix-ck-compatibility-bug branch March 29, 2022 09:02
# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

[ bug ]请简要描述Clickhouse集群DDL语句 命令报错
2 participants