The GitHub Actions job "Binary Compatibility" on pekko.git/main has succeeded. Run started by GitHub user He-Pin (triggered by He-Pin).
Head commit for run: c0e9406bfdf9794b78eff5266f7b9434e519af04 / He-Pin(kerr) <[email protected]> fix: expand Array arguments in MarkerLoggingAdapter log templates (#3263) Motivation: MarkerLoggingAdapter.format1 passed an Array argument to the varargs format without spreading it, so the whole array was treated as a single template argument. A call like markerLog.info(marker, "{} {} {}", Array("a", "b", "c")) rendered as "ArraySeq(a, b, c) {} {}" instead of "a b c". The base LoggingAdapter expands arrays correctly; the marker copy did not. DiagnosticMarkerBusLoggingAdapter inherits the same method and was affected too. Modification: - Spread the array elements as varargs in both Array branches of MarkerLoggingAdapter.format1 so each element fills its own placeholder. Result: Array arguments to the single-argument marker log template methods are now expanded into separate placeholders, matching LoggingAdapter behavior. Tests: - actor-tests/testOnly org.apache.pekko.event.MarkerLoggingSpec - all passed. Added directional tests (non-primitive and primitive arrays) that fail before the fix (rendering ArraySeq(...)) and pass after. References: Fixes #3257 Report URL: https://github.com/apache/pekko/actions/runs/28333708320 With regards, GitHub Actions via GitBox --------------------------------------------------------------------- To unsubscribe, e-mail: [email protected] For additional commands, e-mail: [email protected]
