The GitHub Actions job "Binary Compatibility" on pekko.git/feature/artery-frequency-sketch has failed. Run started by GitHub user He-Pin (triggered by He-Pin).
Head commit for run: 378d21d6722188a718b39f2211c592c72f2bdb2c / He-Pin <[email protected]> feat: replace CountMinSketch with FastFrequencySketch in Artery compression Motivation: The Artery compression system uses a CountMinSketch (128KB per connection) for heavy hitter detection. Pekko already has FastFrequencySketch in pekko-actor (used for cluster sharding passivation) which is ~32x more memory efficient (~4KB) and includes TinyLFU reset for natural aging of frequency data, enabling the heavy hitters table to adapt to changing traffic patterns. Modification: - Replace CountMinSketch with FastFrequencySketch in InboundCompression - Modify TopHeavyHitters to support non-monotonic weights (needed because FrequencySketch's periodic reset halves all counters) - Add fixHeapUp method for bidirectional heap property restoration - Remove CountMinSketch.java (INTERNAL API, no external users) - Update benchmark to use FastFrequencySketch - Update legal files to remove CountMinSketch attribution - Add tests for weight decrease and heap upward restoration Result: Artery compression uses ~4KB instead of ~128KB per inbound connection. Heavy hitters now adapt to changing traffic patterns via TinyLFU aging. Tests: - remote/testOnly ...HeavyHittersSpec: 13/13 passed - remote/testOnly ...CompressionTableSpec: 3/3 passed - remote/testOnly ...OutboundCompressionSpec: 2/2 passed Refs: https://github.com/akka/akka-core/issues/31093 Report URL: https://github.com/apache/pekko/actions/runs/26720355454 With regards, GitHub Actions via GitBox --------------------------------------------------------------------- To unsubscribe, e-mail: [email protected] For additional commands, e-mail: [email protected]
