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.

Sign up for GitHub

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

Jump to bottom

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

Closed
windyHe2020 opened this issue Mar 28, 2022 · 7 comments · Fixed by #1428
Closed

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

windyHe2020 opened this issue Mar 28, 2022 · 7 comments · Fixed by #1428

Comments

@windyHe2020
Copy link

windyHe2020 commented Mar 28, 2022 •
edited
Loading

在提交 issue 前，请查阅以下资源，请先进行搜索来保证没有类似的 issue。
文档 | FAQ

重现步骤

1、Clickhouse集群SQL上线中：
执行语句：ALTER TABLE t1 on cluster bigdata ADD COLUMN create_time DateTime64(3) DEFAULT now64(3) COMMENT '创建时间';/(ALTER TABLE t1 ADD COLUMN create_time DateTime64(3) DEFAULT now64(3) COMMENT '创建时间';)
2、报错：
审核信息列：explain语法检查错误：Code: 62.
DB::Exception: Syntax error: failed at position 19 ('TABLE'): TABLE t1 on cluster bigdata ADD COLUMN create_time DateTime64(3) DEFAULT now64(3) COMMENT '创建时间'. Expected one of: Equals, Dot, token.
3、手工去数据库中执行命令，命令成功。

期待结果和实际结果

期待结果：语句执行成功。

实际结果:SQL检查不通过，系统自动驳回。

截图

image

错误日志

[2022-03-28 15:49:43,051][MainThread:139859518359360][task_id:default][clickhouse.py:123][WARNING]- ClickHouse语句执行报错，语句：explain ast ALTER TABLE t1 on cluster bigdata ADD COLUMN create_time DateTime64(3) DEFAULT now64(3) COMMENT '创建时间'，错误信息Code: 62.
DB::Exception: Syntax error: failed at position 19 ('TABLE'): TABLE t1 on cluster bigdata ADD COLUMN create_time DateTime64(3) DEFAULT now64(3) COMMENT '创建时间'. Expected one of: Equals, Dot, token. Stack trace:

  1. /build/obj-x86_64-linux-gnu/../contrib/poco/Foundation/src/Exception.cpp:27: Poco::Exception::Exception(std::__1::basic_string<char, std::__1::char_traits, std::__1::allocator > const&, int) @ 0x18b66d50 in /usr/lib/debug/usr/bin/clickhouse
  2. /build/obj-x86_64-linux-gnu/../src/Common/Exception.cpp:37: DB::Exception::Exception(std::__1::basic_string<char, std::__1::char_traits, std::__1::allocator > const&, int) @ 0xe5bc32d in /usr/lib/debug/usr/bin/clickhouse
  3. /build/obj-x86_64-linux-gnu/../src/Parsers/parseQuery.cpp:325: DB::parseQuery(DB::IParser&, char const*, char const*, std::__1::basic_string<char, std::__1::char_traits, std::__1::allocator > const&, unsigned long, unsigned long) (.cold) @ 0x1678fb1b in /usr/lib/debug/usr/bin/clickhouse
  4. /build/obj-x86_64-linux-gnu/../contrib/libcxx/include/type_traits:3696: DB::executeQueryImpl(char const*, char const*, DB::Context&, bool, DB::QueryProcessingStage::Enum, bool, DB::ReadBuffer*) @ 0x15b9852e in /usr/lib/debug/usr/bin/clickhouse
  5. /build/obj-x86_64-linux-gnu/../src/Interpreters/executeQuery.cpp:675: DB::executeQuery(std::__1::basic_string<char, std::__1::char_traits, std::__1::allocator > const&, DB::Context&, bool, DB::QueryProcessingStage::Enum, bool) @ 0x15b9a672 in /usr/lib/debug/usr/bin/clickhouse
  6. /build/obj-x86_64-linux-gnu/../src/Server/TCPHandler.cpp:253: DB::TCPHandler::runImpl() @ 0x16235ed5 in /usr/lib/debug/usr/bin/clickhouse
  7. /build/obj-x86_64-linux-gnu/../src/Server/TCPHandler.cpp:1219: DB::TCPHandler::run() @ 0x16236c40 in /usr/lib/debug/usr/bin/clickhouse
  8. /build/obj-x86_64-linux-gnu/../contrib/poco/Net/src/TCPServerConnection.cpp:57: Poco::Net::TCPServerConnection::start() @ 0x18a84d0b in /usr/lib/debug/usr/bin/clickhouse
  9. /build/obj-x86_64-linux-gnu/../contrib/libcxx/include/atomic:856: Poco::Net::TCPServerDispatcher::run() @ 0x18a85428 in /usr/lib/debug/usr/bin/clickhouse
  10. /build/obj-x86_64-linux-gnu/../contrib/poco/Foundation/include/Poco/Mutex_POSIX.h:59: Poco::PooledThread::run() @ 0x18c03b26 in /usr/lib/debug/usr/bin/clickhouse
  11. /build/obj-x86_64-linux-gnu/../contrib/poco/Foundation/include/Poco/AutoPtr.h:223: Poco::ThreadImpl::runnableEntry(void*) @ 0x18bfef20 in /usr/lib/debug/usr/bin/clickhouse
  12. start_thread @ 0x740b in /usr/lib64/libpthread-2.26.so
  13. __clone @ 0xed09f in /usr/lib64/libc-2.26.so

版本信息

应用版本/分支：
Archery (v1.8.3)
部署方式：Docker、手工部署
手工部署
@windyHe2020 windyHe2020 changed the title [ bug ]请简要描述Clickhouse集群DDL语句增加 on cluster 命令报错 [ bug ]请简要描述Clickhouse集群DDL语句 命令报错 Mar 28, 2022
@nick2wang
Copy link
Collaborator

麻烦提供下clickhouse实例的版本号，我本地使用v22.1.3.7测试没有报语法错误，可能是explain ast的兼容性问题

@nick2wang
Copy link
Collaborator

然后改写下sql重新检测下看是否还报错：
ALTER TABLE 表t1所在的数据库名.t1 on cluster bigdata ADD COLUMN create_time DateTime64(3) DEFAULT now64(3) COMMENT '创建时间';

@windyHe2020
Copy link
Author

麻烦提供下clickhouse实例的版本号，我本地使用v22.1.3.7测试没有报语法错误，可能是explain ast的兼容性问题

是的，看报错是explain ast的兼容性问题，我命令窗执行直接explain ast ……也会报错,加库名也是相同的报错。我用的版本是clickhouse 20.8.9.6,clickhouse的explain 好像做的不是很完善，太新的版本bug又比较多……
image

@nick2wang
Copy link
Collaborator

nick2wang commented Mar 28, 2022 •
edited
Loading

explain ast从v21.1.2.15起才支持非select语句，后面我改下兼容问题，低版本先跳过检查

@windyHe2020
Copy link
Author

explain ast从v21.1.2.15起才支持非select语句，后面我改下兼容问题，低版本先跳过检查

好的，幸苦！如果跳过低版本跳过检查，语句执行报错时（包含语法错误），能不能考虑前台打印出来错误信息啊~

@nick2wang
Copy link
Collaborator

explain ast从v21.1.2.15起才支持非select语句，后面我改下兼容问题，低版本先跳过检查

好的，幸苦！如果跳过低版本跳过检查，语句执行报错时（包含语法错误），能不能考虑前台打印出来错误信息啊~

执行报错默认会展示到前台“审核/执行信息”字段

@windyHe2020
Copy link
Author

explain ast从v21.1.2.15起才支持非select语句，后面我改下兼容问题，低版本先跳过检查

好的，幸苦！如果跳过低版本跳过检查，语句执行报错时（包含语法错误），能不能考虑前台打印出来错误信息啊~

执行报错默认会展示到前台“审核/执行信息”字段

好的

@nick2wang nick2wang mentioned this issue Mar 28, 2022
Merged
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

修复clickhouse上线审核版本兼容性问题 nick2wang/Archery
2 participants
@nick2wang @windyHe2020