The GitHub Actions job "Scalafmt" on pekko.git/copilot/copy-akka-core-pulls has failed. Run started by GitHub user pjfanning (triggered by pjfanning).
Head commit for run: 6f7be4b437f2f0cda5072e37b9e6e42e8251fd5f / He-Pin(kerr) <[email protected]> fix: update ConsistentHashingShardAllocationStrategySpec for pekko protocol hash (#58) * fix: update ConsistentHashingShardAllocationStrategySpec for pekko protocol Motivation: ConsistentHashingShardAllocationStrategySpec was ported from Akka 2.8.3 with test expectations based on the akka:// protocol. Since Pekko uses pekko:// as the protocol name, Address.toString produces different strings, causing ConsistentHash to compute different hash values and map shards to different nodes. Modification: Updated all test expectations to match the consistent hash results computed with pekko:// addresses. Key changes: - 3-node: shard "2" maps to regionA (was regionB), shard "3" maps to regionA (was regionC) - 4-node: shard "0" maps to regionD (was regionC), shard "2" to regionA (was regionD) - 2-node: shard "2" maps to regionA (was regionB), shard "3" to regionA (was regionB) - Rebalance assertions updated to reflect the new 4-node shard distribution Result: All 10 ConsistentHashingShardAllocationStrategySpec tests pass. No changes to production code. Tests: sbt "cluster-sharding / Test / testOnly org.apache.pekko.cluster.sharding.ConsistentHashingShardAllocationStrategySpec" Tests: succeeded 10, failed 0 References: Fixes apache/pekko#3270 * fix: update stale comment in ConsistentHashingShardAllocationStrategySpec In Pekko's consistent hash, shards 2, 3, 10 and 14 are all already at the right place (not just 10 and 14 as in Akka). Report URL: https://github.com/apache/pekko/actions/runs/28807700117 With regards, GitHub Actions via GitBox --------------------------------------------------------------------- To unsubscribe, e-mail: [email protected] For additional commands, e-mail: [email protected]
