-
Notifications
You must be signed in to change notification settings - Fork 3.7k
[fix](show) show create table show index comment error #36306
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
Conversation
|
Thank you for your contribution to Apache Doris. Since 2024-03-18, the Document has been moved to doris-website. |
|
run buildall |
TPC-H: Total hot run time: 39296 ms |
TPC-DS: Total hot run time: 173805 ms |
ClickBench: Total hot run time: 31.1 s |
|
run buildall |
TPC-H: Total hot run time: 40482 ms |
TPC-DS: Total hot run time: 170360 ms |
ClickBench: Total hot run time: 31.39 s |
|
run buildall |
|
run performance |
|
run p0 |
TPC-H: Total hot run time: 39501 ms |
TPC-DS: Total hot run time: 169707 ms |
ClickBench: Total hot run time: 30.64 s |
|
run p0 |
|
run buildall |
TPC-H: Total hot run time: 39899 ms |
TPC-DS: Total hot run time: 171740 ms |
ClickBench: Total hot run time: 30.44 s |
|
replace screenshot with text for git msg and search engine firendly |
|
@morrySnow ok, done |
still have screenshot in description |
|
@morrySnow removed. |
fe/fe-core/src/main/java/org/apache/doris/nereids/parser/LogicalPlanBuilder.java
Outdated
Show resolved
Hide resolved
|
run buildall |
TPC-H: Total hot run time: 39872 ms |
TPC-DS: Total hot run time: 174354 ms |
ClickBench: Total hot run time: 30.12 s |
|
PR approved by at least one committer and no changes requested. |
|
PR approved by anyone and no changes requested. |
…or (apache#36306) Fix two bugs that cause show create table index comment error 1. comment has extra single quotes. 2. quota does not escape in show create table result. ``` COMMENT ''test index" comment'' to COMMENT 'test index\" comment' ``` Refs PR: apache#6560 apache#28346
|
@morrySnow May need to remove dev/2.0 tags, doesn't seem to affect 2.0? |
Proposed changes
Fix two bugs that cause show create table index comment error
Refs PR:
#6560
#28346