The GitHub Actions job "Nightly Builds (1.7)" on pekko.git/main has failed.
Run started by GitHub user pjfanning (triggered by pjfanning).

Head commit for run:
17fb00e51e847997a31cf6f130545ba3893285b8 / PJ Fanning 
<[email protected]>
consistent-hashing shard allocation strategy (#3270)

* Apply remaining changes

* manual changes

* more

* more

* more

* compile issues

* build issues

* unused imports

* docs: add Colocate Shards and EventsBySliceFirehoseQuery documentation (#57)

Motivation:
The consistent-hashing shard allocation and EventsBySliceFirehoseQuery code
were ported in this PR, but the corresponding user-facing documentation from
the upstream commits was not yet included.

Modification:
- Add the "Colocate Shards" section to typed/cluster-sharding.md (from akka
  522f566128), wired to the existing ConsistentHashingShardAllocationCompileOnly
  examples.
- Add the EventsBySliceFirehoseQuery scalability paragraph to
  persistence-query.md and a `See also [[EventsBySliceFirehoseQuery]]`
  cross-reference to both EventsBySliceQuery DSL traits (from akka 362ab9301e).

Result:
The Cluster Sharding and Persistence Query docs describe the new
consistent-hashing colocation and firehose query features.

Tests: Not run - docs only (sbt "docs/paradox" builds successfully)

References:
Ports docs from akka/akka-core@522f566128 (akka#31967) and
akka/akka-core@362ab9301e (akka#31957), which are now Apache licensed.
Completes documentation for apache/pekko#3270.

* 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).

* scalafmt

* Update consistent-hashing-allocation.excludes

* Update ConsistentHashingShardAllocationCompileOnlyTest.java

---------

Co-authored-by: copilot-swe-agent[bot] 
<[email protected]>
Co-authored-by: He-Pin(kerr) <[email protected]>

Report URL: https://github.com/apache/pekko/actions/runs/28833901428

With regards,
GitHub Actions via GitBox


---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]

Reply via email to