yiguolei commented on a change in pull request #8259:
URL: https://github.com/apache/incubator-doris/pull/8259#discussion_r818235814



##########
File path: 
fe/fe-core/src/main/java/org/apache/doris/analysis/ModifyTablePropertiesClause.java
##########
@@ -55,7 +55,10 @@ public void analyze(Analyzer analyzer) throws 
AnalysisException {
                 throw new AnalysisException("Can only change storage type to 
COLUMN");
             }
         } else if 
(properties.containsKey(PropertyAnalyzer.PROPERTIES_DISTRIBUTION_TYPE)) {
-            throw new AnalysisException("Cannot change distribution type for 
olap table now");
+            if 
(!properties.get(PropertyAnalyzer.PROPERTIES_DISTRIBUTION_TYPE).equalsIgnoreCase("random"))
 {
+                throw new AnalysisException("Can only change distribution type 
from RANDOM to HASH");

Review comment:
       From random to hash?  Or from hash to random?




-- 
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: commits-unsubscr...@doris.apache.org

For queries about this service, please contact Infrastructure at:
us...@infra.apache.org



---------------------------------------------------------------------
To unsubscribe, e-mail: commits-unsubscr...@doris.apache.org
For additional commands, e-mail: commits-h...@doris.apache.org

Reply via email to