GutoVeronezi commented on PR #6973: URL: https://github.com/apache/cloudstack/pull/6973#issuecomment-1347285676
@DaanHoogland I understand your point in avoiding building a String for low-level logging and not using it; however, I think this proposal is not the best option for handling the situation. As you pointed out in the comment https://github.com/apache/cloudstack/pull/6925#issuecomment-1346446696 and from conversations we had along the time, Log4j2 would solve our issues of string evaluation when it is not necessary. Unfortunately, the community could not proceed with the upgrade from Log4j to Log4j2, as it would demand a huge work effort, and also due to the lack of time and having other priorities; however, now we managed to put efforts into this huge change. We separated the whole project into two phases: 1. Upgrade from Log4j to Log4j2; 2. Normalize the use of the Log4j2 syntax (as Log4j2 has back compatibility with Log4j, we could separate the scopes); We are already working on the first phase; however, as it is a huge change, it is going to take some time until releasing the first draft. As soon as we get the first draft done, we are going to open a well-detailed PR explaining the changes and the effects they will have on ACS. -- 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]
