superdiaodiao commented on code in PR #46713:
URL: https://github.com/apache/doris/pull/46713#discussion_r1909011755
##########
regression-test/suites/check_before_quit/check_before_quit.groovy:
##########
@@ -253,10 +253,21 @@ suite("check_before_quit", "nonConcurrent,p0") {
logger.info("show all tabkes: ${allTables}")
for (int j = 0;j < allTables.size();j ++) {
def tbl = allTables[j][0]
- def createTableSql = sql "show create table ${db}.${tbl}"
+ def createTableSql = ""
+ try {
+ createTableSql = sql "show create table ${db}.${tbl}"
+ logger.info("create table sql: ${createTableSql}")
+ } catch (Exception e) {
+ if e.getMessage().contains("not support async materialized
view") {
+ createTableSql = sql "show create materialized view ${tbl}"
+ logger.info("create materialized view sql:
${createTableSql}")
Review Comment:
Although this PR is changed for the regression test, would it better to make
this log level to **ERROR** or **WARNNING**? After all, this log is inside the
exception part.
--
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.
To unsubscribe, e-mail: [email protected]
For queries about this service, please contact Infrastructure at:
[email protected]
---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]
