chunjun icon indicating copy to clipboard operation
chunjun copied to clipboard

upgrade-clickhouse-client

Open xbkaishui opened this issue 3 years ago • 2 comments

current clickhouse client is too old, new ck cluster newer than 20.x, it throws an exception

java.lang.RuntimeException: no column default_expresseion in columns list database table name type default_type default_expression
	at ru.yandex.clickhouse.response.ClickHouseResultSet.asColNum(ClickHouseResultSet.java:616)
	at ru.yandex.clickhouse.response.ClickHouseResultSet.getString(ClickHouseResultSet.java:251)
	at ru.yandex.clickhouse.ClickHouseDatabaseMetadata.getColumns(ClickHouseDatabaseMetadata.java:863)
	at com.dtstack.flinkx.connector.jdbc.util.JdbcUtil.getTableMetaData(JdbcUtil.java:143)
	at com.dtstack.flinkx.connector.jdbc.sink.JdbcOutputFormat.getTableMetaData(JdbcOutputFormat.java:135)
	at com.dtstack.flinkx.connector.jdbc.sink.JdbcOutputFormat.initColumnList(JdbcOutputFormat.java:121)

should update the client version.

xbkaishui avatar Jan 05 '22 09:01 xbkaishui

@deadwind4 please help to check. thanks

xbkaishui avatar Jan 05 '22 09:01 xbkaishui

感谢提交pr,能建立一下对应的issue,把修复的问题或者功能具体描述下,方便其他小伙伴进行review,另外,能合并下commit提交吗?目前这个存在多个重复commit。pr commit 模版[hotfix-#issueID][#fix-module] #fix-commit.

FlechazoW avatar Mar 03 '22 03:03 FlechazoW

Thank you for this pr, but it has been too long to merge this. You can offer this pr base on the branch of 'master'.

FlechazoW avatar Sep 28 '22 09:09 FlechazoW