-
Notifications
You must be signed in to change notification settings - Fork 3.4k
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
[Enhancement] (nereids)implement showCharsetCommand in nereids #43155
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: 41112 ms
|
9f8e5f5
to
414a7d7
Compare
TPC-DS: Total hot run time: 196241 ms
|
ClickBench: Total hot run time: 31.93 s
|
run buildall |
TPC-H: Total hot run time: 41506 ms
|
TPC-DS: Total hot run time: 197332 ms
|
ClickBench: Total hot run time: 33.2 s
|
414a7d7
to
8425276
Compare
/** | ||
* Represents the command for SHOW CHARSETG | ||
*/ | ||
public class ShowCharsetCommand extends Command implements NoForward { |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
please use ShowCommand
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
done
8425276
to
77d56d2
Compare
run buildall |
TPC-H: Total hot run time: 40281 ms
|
TPC-DS: Total hot run time: 196854 ms
|
ClickBench: Total hot run time: 32.44 s
|
run cloud_p0 |
run buildall |
TPC-H: Total hot run time: 39678 ms
|
TPC-DS: Total hot run time: 196919 ms
|
ClickBench: Total hot run time: 32.68 s
|
run buildall |
TPC-H: Total hot run time: 39721 ms
|
TPC-DS: Total hot run time: 190582 ms
|
ClickBench: Total hot run time: 32.05 s
|
@@ -228,6 +228,7 @@ supportedShowStatement | |||
| SHOW DYNAMIC PARTITION TABLES ((FROM | IN) database=multipartIdentifier)? #showDynamicPartition | |||
| SHOW EVENTS ((FROM | IN) database=multipartIdentifier)? wildWhere? #showEvents | |||
| SHOW LAST INSERT #showLastInsert | |||
| SHOW ((CHAR SET) | CHARSET) wildWhere? #showCharset |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
| SHOW ((CHAR SET) | CHARSET) wildWhere? #showCharset | |
| SHOW ((CHAR SET) | CHARSET) #showCharset |
Since wildWhere is useless, we can remove it for simplicity
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
done - pls review
1b5ed0d
to
360fabd
Compare
run buildall |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM
PR approved by at least one committer and no changes requested. |
PR approved by anyone and no changes requested. |
TPC-H: Total hot run time: 40715 ms
|
TPC-DS: Total hot run time: 195857 ms
|
ClickBench: Total hot run time: 32.63 s
|
Issue Number: close #42749
implement showCharsetCommand in nereids
mysql> show charset;
+---------+---------------+-------------------+--------+
| Charset | Description | Default collation | Maxlen |
+---------+---------------+-------------------+--------+
| utf8mb4 | UTF-8 Unicode | utf8mb4_0900_bin | 4 |
+---------+---------------+-------------------+--------+
1 row in set (0.00 sec)