[GitHub] [iotdb] mychaow commented on a change in pull request #1854: [IOTDB-884] group createMultiTimeseriesPlan by partitionGroup

[GitBox <gi...@apache.org>]

mychaow commented on a change in pull request #1854:
URL: https://github.com/apache/iotdb/pull/1854#discussion_r511353495



##########
File path: cluster/src/main/java/org/apache/iotdb/cluster/server/member/MetaGroupMember.java
##########
@@ -1478,6 +1479,11 @@ private TSStatus forwardPlan(Map<PhysicalPlan, PartitionGroup> planGroupMap, Phy
         // belongs to NodeB, when NodeA returns a success while NodeB returns a failure, the
         // failure and success should be placed into proper positions in TSStatus.subStatus
         status = forwardInsertTabletPlan(planGroupMap, (InsertTabletPlan) plan);
+      } else if (plan instanceof CreateMultiTimeSeriesPlan) {
+        // CreateMultiTimeSeriesPlans contain many rows, each will correspond to a TSStatus as its
+        // execution result, as the plan is split and the sub-plans may have interleaving ranges,
+        // we must assure that each TSStatus is placed to the right position

Review comment:
       Thank you!




----------------------------------------------------------------
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:
users@infra.apache.org