The GitHub Actions job "Oracle Integration Tests" on 
pekko-persistence-jdbc.git/slf4j-lazy-logging has succeeded.
Run started by GitHub user pjfanning (triggered by pjfanning).

Head commit for run:
1857c9eda018435b86e44446cfb0edb3a192fe26 / PJ Fanning 
<[email protected]>
Defer debug log message construction to the logging backend

Motivation:
Several debug calls built their message eagerly with string interpolation, so
the interpolation (and, in JournalMigrator, a `mkString` over the tag set for
every migrated event) ran even when debug logging was disabled.

Modification:
Replace interpolation with `{}` placeholders and pass the values as logging
arguments, and guard the JournalMigrator call — whose argument needs a
`mkString` — with `isDebugEnabled`.

Result:
No message or tag-set string is built unless the backend will emit the line.
Logging output is unchanged.

Tests:
- scalafmt on the three changed files/no reformatting needed
- No test added or run: the change only moves message formatting into the
  logging backend and emits identical output; compilation and the existing
  suites are covered by CI

References:
None - follow-up cleanup on the debug logging in these three files

Report URL: 
https://github.com/apache/pekko-persistence-jdbc/actions/runs/34543271459

With regards,
GitHub Actions via GitBox


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

Reply via email to