Hi Sunil G,
Thanks for updating,
In yarn_service_resourcemanager_service_protos, ReplaceLabelsOnNodeRequestProto still used NodeIdToLabelsProto.
So I feel we can keep the same.
I think you can move NodeIdToLabelsProto to yarn_server_resourcemanager_service_protos, yarn_protos supposes to include only client-side-visible objects.
I feel existing test case testGetNodeToLabelsResponsePBImpl which is present in TestPBImplRecords internally covers the verification for NodeIdToLabelsInfoProto. Pls suggest if otherwise.
I guess so, but I think it's better to add it separately to make sure it will be tested.
Took a quick look at TestClientRMService, have you checked NodeLabel.isExclusive after received from RM? I didn't find it..