EmmyMiao87 commented on a change in pull request #5770:
URL: https://github.com/apache/incubator-doris/pull/5770#discussion_r633468950
##########
File path: docs/zh-CN/sql-reference/sql-statements/Data Manipulation/SHOW
DATA.md
##########
@@ -48,6 +48,8 @@ SHOW DATA [FROM db_name[.table_name]];
5. 如果想查看各个 Partition 的大小，请参阅 `help show partitions`。
+6. 可以使用 ORDER BY 对任意列组合进行排序。
+
## example
1. 展示默认 db 的各个 table 的数据量，副本数量，汇总数据量和汇总副本数量。
Review comment:
可以增加一个使用order by 的实际例子。
##########
File path: fe/fe-core/src/main/java/org/apache/doris/analysis/ShowDataStmt.java
##########
@@ -17,7 +17,12 @@
package org.apache.doris.analysis;
+import com.google.common.base.Strings;
Review comment:
pay attention to the import order
##########
File path: fe/fe-core/src/main/java/org/apache/doris/analysis/ShowDataStmt.java
##########
@@ -270,6 +371,15 @@ public String toSql() {
return builder.toString();
Review comment:
Maybe it should not be returned here.
The reason is that even if there is no tableName, there may be an order by
clause.
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
For queries about this service, please contact Infrastructure at:
[email protected]
---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]
