Riza Suminto has posted comments on this change. ( 
http://gerrit.cloudera.org:8080/22214 )

Change subject: IMPALA-11979: Add 'executor_id' to customize consistent 
scheduling behavior
......................................................................


Patch Set 3:

(1 comment)

http://gerrit.cloudera.org:8080/#/c/22214/3/common/protobuf/statestore_service.proto
File common/protobuf/statestore_service.proto:

http://gerrit.cloudera.org:8080/#/c/22214/3/common/protobuf/statestore_service.proto@88
PS3, Line 88:   // Unique identifier of an executor within a single executor 
group (e.g. 3rd out of 8
            :   // executors). This is used for consistently scheduling scan 
ranges. This is unique
            :   // within an executor group, but executors in different 
executor groups can use the
            :   // same identifier.
            :   optional string executor_id = 15;
executor_id is potentially confused with backend_id.

Perhaps scheduling_seed is less confusing. We can also say that scheduling_seed 
must be unique between backends, but can stay the same between restart, unlike 
backend_id or ip_address.



--
To view, visit http://gerrit.cloudera.org:8080/22214
To unsubscribe, visit http://gerrit.cloudera.org:8080/settings

Gerrit-Project: Impala-ASF
Gerrit-Branch: master
Gerrit-MessageType: comment
Gerrit-Change-Id: Ie01c7d119cc88766082dbfca3ff685354d01f71f
Gerrit-Change-Number: 22214
Gerrit-PatchSet: 3
Gerrit-Owner: Joe McDonnell <[email protected]>
Gerrit-Reviewer: Impala Public Jenkins <[email protected]>
Gerrit-Reviewer: Riza Suminto <[email protected]>
Gerrit-Comment-Date: Thu, 07 Aug 2025 18:18:32 +0000
Gerrit-HasComments: Yes

Reply via email to