Aggarwal-Raghav commented on code in PR #5885:
URL: https://github.com/apache/hive/pull/5885#discussion_r2160237963


##########
itests/hive-jmh/src/main/resources/log4j2.properties:
##########
@@ -14,7 +14,6 @@
 # See the License for the specific language governing permissions and
 # limitations under the License.
 
-status = FATAL

Review Comment:
   Thanks for the review @okumin. Very valid question, when I was working on 
the PR, i also though about it.
   Based on the comments 
https://github.com/apache/logging-log4j2/issues/2794#issuecomment-2267564572 
   > Status Logger is intended for troubleshooting Log4j itself and its 
components. I strongly advise you to avoid configuring it unless you are 
troubleshooting a Log4j issue.
   
   Because of that I went ahead and removed. I don't have any strong opinion on 
this, If you think its better to keep, then I can update the PR.



-- 
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: gitbox-unsubscr...@hive.apache.org

For queries about this service, please contact Infrastructure at:
us...@infra.apache.org


---------------------------------------------------------------------
To unsubscribe, e-mail: gitbox-unsubscr...@hive.apache.org
For additional commands, e-mail: gitbox-h...@hive.apache.org

Reply via email to