[GitHub] [doris] qzsee commented on a diff in pull request #21906: [Bug][Colocate] when adding a table to the colocate group, we should check that the number of buckets per partition is the same

[via GitHub]

qzsee commented on code in PR #21906:
URL: https://github.com/apache/doris/pull/21906#discussion_r1267449822


##########
fe/fe-core/src/main/java/org/apache/doris/catalog/Env.java:
##########
@@ -4042,7 +4042,7 @@ public void modifyTableColocate(Database db, OlapTable 
table, String assignedGro
                 }
             } else {
                 // set to an already exist colocate group, check if this table 
can be added to this group.
-                groupSchema.checkColocateSchema(table);

Review Comment:
   We add a table with many partitions to the colocate group, and we need to 
check whether all partitions comply with the colocate group specification



-- 
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