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.

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

[Bug] [metadata] Failed to query clickhouse table description information #3494

Closed
3 tasks done
suxinshuo opened this issue May 15, 2024 · 3 comments · Fixed by #3504
Closed
3 tasks done

[Bug] [metadata] Failed to query clickhouse table description information #3494

suxinshuo opened this issue May 15, 2024 · 3 comments · Fixed by #3504
Assignees
Labels
Bug Something isn't working
Milestone

Comments

@suxinshuo
Copy link
Contributor

Search before asking

  • I had searched in the issues and found no similar issues.

What happened

问题1: 查询 ck 表描述信息报错

image

问题原因: 当查询 clickhouse 描述信息的时候,如果表字段有 Array(String) 类型,查询会报错,是因为下面这行代码,replace 以后变成空字符串,转数字报错。
image

问题2: ck 表描述信息中,字段类型展示有误

例如下面这个表
image
实际的字段类型
image

What you expected to happen

字段信息可以正常展示,如下图所示
image

How to reproduce

新建 clickhouse 表,包含 Array(String) 类型的字段,进入数据源查询页面查询该表的描述信息

Anything else

No response

Version

dev

Are you willing to submit PR?

  • Yes I am willing to submit a PR!

Code of Conduct

@suxinshuo suxinshuo added Bug Something isn't working Waiting for reply Waiting for reply labels May 15, 2024
@Zzm0809
Copy link
Contributor

Zzm0809 commented May 15, 2024

社区无此环境进行修复, 如你有意愿可进行修复

@suxinshuo
Copy link
Contributor Author

社区无此环境进行修复, 如你有意愿可进行修复

可以的

@Zzm0809 Zzm0809 removed the Waiting for reply Waiting for reply label May 16, 2024
@Zzm0809 Zzm0809 added this to the 1.1.0 milestone May 16, 2024
@Zzm0809
Copy link
Contributor

Zzm0809 commented May 16, 2024

社区无此环境进行修复, 如你有意愿可进行修复

可以的

已分配

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Bug Something isn't working
Projects
Status: ToDo
Development

Successfully merging a pull request may close this issue.

2 participants