nija-at opened a new pull request, #47667:
URL: https://github.com/apache/spark/pull/47667

   ### What changes were proposed in this pull request?
   
   In a previous commit a713a66e, log level in the connect client
   was lowered from info to debug. As part of that, the explicit
   check for logging level was removed.
   
   Revert this change and put the logging level check back.
   
   ### Why are the changes needed?
   
   The explicit check was there so as to not convert the proto
   to a string, done during string interpolation, when not at the
   expected log level. Conversion from proto to string can be
   expensive for some proto messages.
   
   ### Does this PR introduce _any_ user-facing change?
   
   A previously reverted perfomance improvement is now
   put back in place.
   
   ### Was this patch authored or co-authored using generative AI tooling?
   
   No.
   


-- 
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]

Reply via email to