Closed aramissennyeydd closed 3 months ago
Followup to #4750, this was accidentally reverted in https://github.com/microsoft/rushstack/pull/4750/commits/86c54bb24ed557506093607c01e26eb6240155a0. We need to use operation name instead of phase name to cluster by since shards have the same phase but different operation names. I found this while trying to support clustering in our local instance.
Tested on the build-tests/rush-redis-cobuild-plugin-integration-test/sandbox/sharded-repo. You can see in the below screenshot that shards for package a and b are cobuilt across both machines.
build-tests/rush-redis-cobuild-plugin-integration-test/sandbox/sharded-repo
None.
Summary
Followup to #4750, this was accidentally reverted in https://github.com/microsoft/rushstack/pull/4750/commits/86c54bb24ed557506093607c01e26eb6240155a0. We need to use operation name instead of phase name to cluster by since shards have the same phase but different operation names. I found this while trying to support clustering in our local instance.
How it was tested
Tested on the
build-tests/rush-redis-cobuild-plugin-integration-test/sandbox/sharded-repo
. You can see in the below screenshot that shards for package a and b are cobuilt across both machines.Impacted documentation
None.