yangzhg commented on a change in pull request #4582:
URL: https://github.com/apache/incubator-doris/pull/4582#discussion_r486238872
##########
File path:
fe/fe-core/src/main/java/org/apache/doris/analysis/AlterTableStmt.java
##########
@@ -79,11 +82,28 @@ public void analyze(Analyzer analyzer) throws UserException
{
public void rewriteAlterClause(OlapTable table) throws UserException {
List<AlterClause> clauses = new ArrayList<>();
for (AlterClause alterClause : ops) {
- if (alterClause instanceof EnableFeatureClause
- && ((EnableFeatureClause) alterClause).getFeature() ==
EnableFeatureClause.Features.BATCH_DELETE) {
+ if (alterClause instanceof EnableFeatureClause) {
+ EnableFeatureClause.Features alterFeature =
((EnableFeatureClause) alterClause).getFeature();
+ if (alterFeature == null || alterFeature ==
EnableFeatureClause.Features.UNKNOWN) {
+ throw new AnalysisException("unknown feature for alter
clause");
+ }
if (table.getKeysType() != KeysType.UNIQUE_KEYS) {
- throw new AnalysisException("Batch delete only supported
in unique tables.");
+ throw new AnalysisException("enable feature only supported
in unique tables now.");
Review comment:
batch delete only support UNIQUE_KEYS but other may support all tables
----------------------------------------------------------------
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]
