surahman commented on issue #3762:
URL:
https://github.com/apache/incubator-heron/issues/3762#issuecomment-1018887773
I believe it would be a fairly quick fix to employ the bridge adapter in
**_Option 1_**.
I had a look at **_Option 2_** in the list and I could not find references
to most of the mentioned functions. There are similarly named logging functions
for Python but the logging backend for them is different. The two areas in
**_Option 2_** that will raise issues:
* `String concatenation like logger.info("hi " + userName)`
* `Calls to org.apache.log4j.Logger.setLevel() or similar methods`
Because of the line length cap at 80 (why? this max line length is a legacy
from the punched card era), we have to split line lengths. I mostly used format
strings because they are more efficient but there were places in the code where
there was the use of concatenation.
We do set levels for logging, but this is mostly through a parameter to the
log message function. I could not find a reference to the use of `setLevel` in
the Java codebase.
--
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]
