Use parameterized log messages instead of string concatenation

[yashmayya]

• Non-constant string concatenations are evaluated at runtime even when the actual log message isn't emitted and this has a non-zero performance impact.
• Using parameterized log messages helps avoid this issue since the string evaluation will only occur if the logger is enabled.
• While this change is particularly impactful for debug and trace log messages since the default log level is info, we should ideally use parameterized log messages instead of string concatenation for all log levels since log levels for loggers can be adjusted, and parameterized log messages are considered best practice.
• This patch doesn't modify log messages that are inside a log level check if block (here, for instance).

[codecov-commenter]

Codecov Report

Attention: Patch coverage is 3.65854% with 79 lines in your changes are missing coverage. Please review.

Project coverage is 35.16%. Comparing base (59551e4) to head (a6face7).
Report is 439 commits behind head on master.

Files	Patch %	Lines
...r/helix/SegmentOnlineOfflineStateModelFactory.java	0.00%	10 Missing ⚠️
.../impl/stats/SegmentPreIndexStatsCollectorImpl.java	0.00%	8 Missing ⚠️
...cordtransformer/SchemaConformingTransformerV2.java	0.00%	6 Missing ⚠️
...t/segment/local/utils/ConsistentDataPushUtils.java	0.00%	5 Missing ⚠️
...a/manager/realtime/RealtimeSegmentDataManager.java	0.00%	4 Missing ⚠️
...in/stream/kafka20/KafkaStreamMetadataProvider.java	0.00%	4 Missing ⚠️
...eam/kinesis/server/KinesisDataServerStartable.java	0.00%	4 Missing ⚠️
...in/stream/pulsar/PulsarStreamMetadataProvider.java	0.00%	4 Missing ⚠️
...pinot/query/runtime/plan/MultiStageQueryStats.java	0.00%	3 Missing ⚠️
...on/src/main/java/org/apache/pinot/serde/SerDe.java	0.00%	2 Missing ⚠️
... and 25 more

Additional details and impacted files

@@              Coverage Diff              @@
##             master   #13145       +/-   ##
=============================================
- Coverage     61.75%   35.16%   -26.59%     
+ Complexity      207        6      -201     
=============================================
  Files          2436     2440        +4     
  Lines        133233   134218      +985     
  Branches      20636    20785      +149     
=============================================
- Hits          82274    47203    -35071     
- Misses        44911    83534    +38623     
+ Partials       6048     3481     -2567     

Flag	Coverage Δ
custom-integration1	?
integration	<0.01% <0.00%> (-0.01%)	⬇️
integration1	<0.01% <0.00%> (-0.01%)	⬇️
integration2	0.00% <0.00%> (ø)
java-11	0.00% <0.00%> (-61.71%)	⬇️
java-21	35.16% <3.65%> (-26.46%)	⬇️
skip-bytebuffers-false	35.16% <3.65%> (-26.58%)	⬇️
skip-bytebuffers-true	<0.01% <0.00%> (-27.73%)	⬇️
temurin	35.16% <3.65%> (-26.59%)	⬇️
unittests	46.55% <6.97%> (-15.19%)	⬇️
unittests1	46.55% <6.97%> (-0.34%)	⬇️
unittests2	?

Flags with carried forward coverage won't be shown. Click here to find out more.

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

[Jackie-Jiang]

Thank you for cleaning this up!