Skip to content

fix(table): query table data with no column comments#1488

Merged
yhilmare merged 1 commit intodev/4.2.3from
wenmu_423_fix_query_table_no_comments
Jan 31, 2024
Merged

fix(table): query table data with no column comments#1488
yhilmare merged 1 commit intodev/4.2.3from
wenmu_423_fix_query_table_no_comments

Conversation

@LuckyPickleZZ
Copy link
Copy Markdown
Member

What type of PR is this?

type-bug

What this PR does / why we need it:

When querying table data, there is no column comment info. But it works correctly in sql console.
This is because the query api didn't set SqlAsyncExecuteReq#showTableColumnInfo to true when calling execute method.

Which issue(s) this PR fixes:

Fixes #1482

@LuckyPickleZZ LuckyPickleZZ added the type-bug Something isn't working label Jan 31, 2024
@LuckyPickleZZ LuckyPickleZZ added this to the ODC 4.2.3-bp milestone Jan 31, 2024
@LuckyPickleZZ LuckyPickleZZ self-assigned this Jan 31, 2024
Copy link
Copy Markdown
Contributor

@yhilmare yhilmare left a comment

Choose a reason for hiding this comment

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

lgtm

@yhilmare yhilmare merged commit 5bfe989 into dev/4.2.3 Jan 31, 2024
@yhilmare yhilmare deleted the wenmu_423_fix_query_table_no_comments branch January 31, 2024 05:39
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

type-bug Something isn't working

Projects

Archived in project

Development

Successfully merging this pull request may close these issues.

2 participants