This is an automated email from the ASF dual-hosted git repository.
w41ter pushed a commit to branch branch-3.0
in repository https://gitbox.apache.org/repos/asf/doris.git
The following commit(s) were added to refs/heads/branch-3.0 by this push:
new c72501254f3 branch-3.0: [fix](sql) Fix error for show create table
column comment #44958 (#45171)
c72501254f3 is described below
commit c72501254f34e81d9b1fc71edf0b213c3c360a57
Author: github-actions[bot]
<41898282+github-actions[bot]@users.noreply.github.com>
AuthorDate: Tue Dec 10 11:00:29 2024 +0800
branch-3.0: [fix](sql) Fix error for show create table column comment
#44958 (#45171)
Cherry-picked from #44958
Co-authored-by: Uniqueyou <134280716+wyxxx...@users.noreply.github.com>
---
.../src/main/java/org/apache/doris/catalog/Column.java | 2 +-
.../org/apache/doris/analysis/ShowCreateTableStmtTest.java | 14 ++++++++++++--
2 files changed, 13 insertions(+), 3 deletions(-)
diff --git a/fe/fe-core/src/main/java/org/apache/doris/catalog/Column.java
b/fe/fe-core/src/main/java/org/apache/doris/catalog/Column.java
index 52d0d33f1e9..6063ff89979 100644
--- a/fe/fe-core/src/main/java/org/apache/doris/catalog/Column.java
+++ b/fe/fe-core/src/main/java/org/apache/doris/catalog/Column.java
@@ -1008,7 +1008,7 @@ public class Column implements GsonPostProcessable {
sb.append(" ON UPDATE ").append(defaultValue).append("");
}
if (StringUtils.isNotBlank(comment)) {
- sb.append(" COMMENT '").append(getComment(true)).append("'");
+ sb.append(" COMMENT \"").append(getComment(true)).append("\"");
}
return sb.toString();
}
diff --git
a/fe/fe-core/src/test/java/org/apache/doris/analysis/ShowCreateTableStmtTest.java
b/fe/fe-core/src/test/java/org/apache/doris/analysis/ShowCreateTableStmtTest.java
index 353fbad9fab..0faf4a8f34e 100644
---
a/fe/fe-core/src/test/java/org/apache/doris/analysis/ShowCreateTableStmtTest.java
+++
b/fe/fe-core/src/test/java/org/apache/doris/analysis/ShowCreateTableStmtTest.java
@@ -32,7 +32,7 @@ public class ShowCreateTableStmtTest extends
TestWithFeService {
createDatabase("test");
useDatabase("test");
createTable("create table table1\n"
- + "(k1 int comment 'test column k1', k2 int comment 'test
column k2') comment 'test table1' "
+ + "(k1 int comment 'test column k1', k2 int comment 'test
column k2', `timestamp` DATE NOT NULL COMMENT '[''0000-01-01'',
''9999-12-31'']') comment 'test table1' "
+ "PARTITION BY RANGE(`k1`)\n"
+ "(\n"
+ " PARTITION `p01` VALUES LESS THAN (\"10\"),\n"
@@ -48,10 +48,20 @@ public class ShowCreateTableStmtTest extends
TestWithFeService {
String sql = "show create table table1";
ShowResultSet showResultSet = showCreateTable(sql);
String showSql = showResultSet.getResultRows().get(0).get(1);
- Assertions.assertTrue(showSql.contains("`k1` int NULL COMMENT 'test
column k1'"));
+ Assertions.assertTrue(showSql.contains("`k1` int NULL COMMENT \"test
column k1\""));
Assertions.assertTrue(showSql.contains("COMMENT 'test table1'"));
}
+ @Test
+ public void testColumnComment() throws Exception {
+ String sql = "show create table table1";
+ ShowResultSet showResultSet = showCreateTable(sql);
+ String showSql = showResultSet.getResultRows().get(0).get(1);
+ Assertions.assertTrue(showSql.contains("`k1` int NULL COMMENT \"test
column k1\""));
+ Assertions.assertTrue(showSql.contains("`k2` int NULL COMMENT \"test
column k2\""));
+ Assertions.assertTrue(showSql.contains("`timestamp` date NOT NULL
COMMENT \"['0000-01-01', '9999-12-31']\""));
+ }
+
@Test
public void testBrief() throws Exception {
String sql = "show brief create table table1";
---------------------------------------------------------------------
To unsubscribe, e-mail: commits-unsubscr...@doris.apache.org
For additional commands, e-mail: commits-h...@doris.apache.org
