pjfanning opened a new pull request, #616:
URL: https://github.com/apache/pekko-persistence-jdbc/pull/616
### Motivation
Several debug calls built their message eagerly with string interpolation,
so the interpolation ran even when debug logging was disabled. In
`JournalMigrator` that also meant a `mkString` over the tag set for every
migrated event.
### 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
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: [email protected]
For queries about this service, please contact Infrastructure at:
[email protected]
---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]
