elek opened a new pull request #1850: URL: https://github.com/apache/ozone/pull/1850
## What changes were proposed in this pull request? `ozone version` command prints out an annoying warning about missing log4j (first 3 lines): ``` SLF4J: Failed to load class "org.slf4j.impl.StaticLoggerBinder". SLF4J: Defaulting to no-operation (NOP) logger implementation SLF4J: See http://www.slf4j.org/codes.html#StaticLoggerBinder for further details. ////////////// //////////////////// //////// //////////////// ////// //////////////// ///// //////////////// / ///// //////// /// //// //////// ///// ///// //////////////// ///// //////////////// // //// /////////////// ///// ///// /////////////// //// ///// ////// ///// ////// ////// ///// /////////// //////// ////// //////////// /// ////////// / 1.1.0-SNAPSHOT(Denali) Source code repository [email protected]:apache/ozone.git -r aa2f05485fecbfbec01661e189f60dc590abfc4b Compiled by elek on 2021-01-27T12:58Z Compiled with protoc 2.5.0 From source with checksum c83ca7b0afebec969e72499bb78d45d With Apache Ratis: 1.1.0-eb66796d-SNAPSHOT from de7172d9c1140dba2dbd4516b11f6a4f720d2f12 Using HDDS 1.1.0-SNAPSHOT Source code repository [email protected]:apache/ozone.git -r aa2f05485fecbfbec01661e189f60dc590abfc4b Compiled by elek on 2021-01-27T12:57Z Compiled with protoc 2.5.0 ``` We should use a classpath definition which includes `slf4j-log4j` binding ## How was this patch tested? Executed `ozone version` with or without the patch. ---------------------------------------------------------------- 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. For queries about this service, please contact Infrastructure at: [email protected] --------------------------------------------------------------------- To unsubscribe, e-mail: [email protected] For additional commands, e-mail: [email protected]
