the function version() has some problem

[WUWEI111222333]

Summary
the deploy mode is single

[wangjili8417]

I'd like to work on the issue.

[arunma]

To add some details to the issue, this does not work only for the MySQL command line client. When the command is issued from DBeaver, it seems to be alright.

[sundy-li]

@WUWEI111222333

I did not get it, why did the resulting change in the same query ?

[sundy-li]

I'd like to work on the issue.

It's already rewritten to literal in type checker

[b41sh]

@WUWEI111222333

I did not get it, why did the resulting change in the same query ?

as alias does not work, the output is always version(), not the user-specified alias title.

[WUWEI111222333]

the difference is the space ,i think...

[b41sh]

the difference is the space ,i think... !

Both MySQLFederated and ClickHouseFederated will return version value by regular matching select version(), which is different from the return value of the version function. This will result in different clients getting different version values.

• ClickHouse 8.12.14
• MySQL 8.0.26-v0.4.1-alpha-ee44cc6(rust-1.68.0-nightly-2023-03-04T09:33:30.594399Z)
• version function DatabendQuery v0.4.1-alpha-ee44cc6(rust-1.68.0-nightly-2023-03-04T09:33:30.594399Z)

Can we remove the regular matching from both MySQLFederated and ClickHouseFederated, use the version function for all clients? @sundy-li

@TCeason

[TCeason]

the difference is the space ,i think... !

Both MySQLFederated and ClickHouseFederated will return version value by regular matching select version(), which is different from the return value of the version function. This will result in different clients getting different version values.

• ClickHouse 8.12.14

• MySQL 8.0.26-v0.4.1-alpha-ee44cc6(rust-1.68.0-nightly-2023-03-04T09:33:30.594399Z)

• version function DatabendQuery v0.4.1-alpha-ee44cc6(rust-1.68.0-nightly-2023-03-04T09:33:30.594399Z)

Can we remove the regular matching from both MySQLFederated and ClickHouseFederated, use the version function for all clients? @sundy-li

@TCeason

MySQL Fed is hard code with 8.0.26 because some gui will judge MySQL version.

So the version() is a temp value to pass gui use MySQL driver connect to Databend query.

[b41sh]

the difference is the space ,i think... !

Both MySQLFederated and ClickHouseFederated will return version value by regular matching select version(), which is different from the return value of the version function. This will result in different clients getting different version values.

• ClickHouse 8.12.14
• MySQL 8.0.26-v0.4.1-alpha-ee44cc6(rust-1.68.0-nightly-2023-03-04T09:33:30.594399Z)
• version function DatabendQuery v0.4.1-alpha-ee44cc6(rust-1.68.0-nightly-2023-03-04T09:33:30.594399Z)

Can we remove the regular matching from both MySQLFederated and ClickHouseFederated, use the version function for all clients? @sundy-li
@TCeason

MySQL Fed is hard code with 8.0.26 because some gui will judge MySQL version.

So the version() is a temp value to pass gui use MySQL driver connect to Databend query.

@TCeason Can we remove the version regular match from MySQLFederated and ClickHouseFederated? Instead, return a different version value in the version function based on the type of SessionType. This would ensure that the same client calling version function in different ways would get the same return value.

[TCeason]

the difference is the space ,i think... !

Both MySQLFederated and ClickHouseFederated will return version value by regular matching select version(), which is different from the return value of the version function. This will result in different clients getting different version values.

• ClickHouse 8.12.14

• MySQL 8.0.26-v0.4.1-alpha-ee44cc6(rust-1.68.0-nightly-2023-03-04T09:33:30.594399Z)

• version function DatabendQuery v0.4.1-alpha-ee44cc6(rust-1.68.0-nightly-2023-03-04T09:33:30.594399Z)

Can we remove the regular matching from both MySQLFederated and ClickHouseFederated, use the version function for all clients? @sundy-li

@TCeason

MySQL Fed is hard code with 8.0.26 because some gui will judge MySQL version.

So the version() is a temp value to pass gui use MySQL driver connect to Databend query.

@TCeason Can we remove the version regular match from MySQLFederated and ClickHouseFederated? Instead, return a different version value in the version function based on the type of SessionType. This would ensure that the same client calling version function in different ways would get the same return value.

I think that's ok. cc@BohuTANG