[GH] (pekko/feature/artery-frequency-sketch): Workflow run "Scalafmt" failed!

[GitBox <gi...@apache.org>]

The GitHub Actions job "Scalafmt" on pekko.git/feature/artery-frequency-sketch has failed.
Run started by GitHub user He-Pin (triggered by He-Pin).

Head commit for run:
83cd9ae3a6523e9abdafcbd0de3915bd99e7abae / He-Pin <he...@gmail.com>
fix: address review feedback - keep CountMinSketch and make configurable

Motivation:
Reviewer (pjfanning) requested keeping the original CountMinSketch code
and making the compression scheme configurable so users can choose.

Modification:
- Restored CountMinSketch.java (was deleted in previous commit)
- Restored legal files for CountMinSketch attribution
- Added configuration option  to choose
  between 'count-min-sketch' and 'fast-frequency-sketch' (default)
- Created FrequencySketch abstraction trait with wrappers for both
  CountMinSketch and FastFrequencySketch
- Reverted CountMinSketchBenchmark to use CountMinSketch directly
  (FastFrequencySketch is private[pekko] and not accessible from bench-jmh)
- Removed MiMa filter since CountMinSketch is no longer removed

Result:
Users can now choose between CountMinSketch (legacy, ~128KB per connection)
and FastFrequencySketch (default, ~4KB per connection with TinyLFU aging).

Tests:
- remote/testOnly ...HeavyHittersSpec: 13/13 passed
- remote/testOnly ...CompressionTableSpec: 3/3 passed
- remote/compile: passed
- bench-jmh/compile: passed
- remote/mimaReportBinaryIssues: passed

Refs: #3023

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

With regards,
GitHub Actions via GitBox


---------------------------------------------------------------------
To unsubscribe, e-mail: notifications-unsubscribe@pekko.apache.org
For additional commands, e-mail: notifications-help@pekko.apache.org