kangpinghuang commented on a change in pull request #2941: drop index when 
index column is dropped
URL: https://github.com/apache/incubator-doris/pull/2941#discussion_r381058492
 
 

 ##########
 File path: fe/src/main/java/org/apache/doris/alter/SchemaChangeHandler.java
 ##########
 @@ -224,7 +225,17 @@ private void processDropColumn(DropColumnClause 
alterClause, OlapTable olapTable
                 }
             }
         }
-        
+
+        Iterator<Index> it = indexes.iterator();
+        while(it.hasNext()){
+            Index index = it.next();
+            if(index.getIndexType() == IndexDef.IndexType.BITMAP
 
 Review comment:
   If the column is dropped, all type of index should be dropped, so here 
should not add this if 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:
us...@infra.apache.org


With regards,
Apache Git Services

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

Reply via email to