The GitHub Actions job "Unit Tests" on 
pekko-projection.git/refactor/replace-reflection-with-methodhandles has failed.
Run started by GitHub user He-Pin (triggered by He-Pin).

Head commit for run:
91a65a065f79e51910a0ba5f0e1f639577835803 / 虎鸣 <[email protected]>
refactor: cache protobuf parser method lookup

Motivation:
Address PR review feedback that repeatedly calling MethodHandles.publicLookup() 
has unnecessary overhead while keeping the MethodHandle-based parser resolution 
behavior.

Modification:
Store a shared public Lookup for Java protobuf parser discovery and cache 
parser MethodHandles with a ConcurrentHashMap putIfAbsent pattern to avoid 
non-atomic TrieMap getOrElseUpdate lookup races.

Result:
ProtoAnySerialization reuses the public lookup and aligns parser handle caching 
with the existing ConsumerFilterStore MethodHandle cache style.

Tests:
- scalafmt --mode diff-ref=origin/main
- scalafmt --list --mode diff-ref=origin/main
- sbt "grpc-test / Test / testOnly 
org.apache.pekko.projection.grpc.internal.ProtoAnySerializationSpec"
- sbt "jdbc / Test / testOnly 
org.apache.pekko.projection.jdbc.JdbcProjectionTest 
org.apache.pekko.projection.jdbc.JdbcOffsetStoreSpec"
- sbt "jdbc / Test / testOnly 
org.apache.pekko.projection.jdbc.H2JdbcOffsetStoreSpec"
- sbt "jdbc-int-test / Test / testOnly 
org.apache.pekko.projection.jdbc.JdbcProjectionSpec"
- sbt "examples / Test / compile"
- git diff --check
- qodercli stdout review: No must-fix findings (/tmp/project-qoder-review-2.log)

References:
Refs apache/pekko-projection#561

Report URL: https://github.com/apache/pekko-projection/actions/runs/28802114980

With regards,
GitHub Actions via GitBox


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

Reply via email to