walterddr commented on code in PR #10673:
URL: https://github.com/apache/pinot/pull/10673#discussion_r1175372855
##########
pinot-common/src/main/proto/worker.proto:
##########
@@ -72,21 +72,17 @@ message StagePlan {
int32 stageId = 1;
string instanceId = 2;
StageNode stageRoot = 3;
- map<int32, StageMetadata> stageMetadata = 4;
+ map<int32, WorkerMetadata> workerMetadata = 4;
}
-message StageMetadata {
- repeated string instances = 1;
- repeated string dataSources = 2;
- map<string, SegmentMap> instanceToSegmentMap = 3;
+message WorkerMetadata {
+ string instances = 1;
+ string dataSources = 2;
+ map<string, SegmentList> instanceToSegment = 3;
Review Comment:
this is no longer needed to point from instance to table type to segments,
b/c worker metadata contextually only work on 1 worker
--
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...@pinot.apache.org
For queries about this service, please contact Infrastructure at:
us...@infra.apache.org
---------------------------------------------------------------------
To unsubscribe, e-mail: commits-unsubscr...@pinot.apache.org
For additional commands, e-mail: commits-h...@pinot.apache.org
