wuchong commented on code in PR #2577:
URL: https://github.com/apache/fluss/pull/2577#discussion_r2778442443
##########
fluss-dist/src/main/resources/conf/log4j-console.properties:
##########
@@ -57,3 +57,9 @@ appender.rolling.strategy.max=${env:MAX_LOG_FILE_NUMBER:-10}
# Suppress the irrelevant (wrong) warnings from the Netty channel handler
logger.netty.name=org.jboss.netty.channel.DefaultChannelPipeline
logger.netty.level=OFF
+# don't print jindo log
+logger.jindo_fsStats.name=com.aliyun.jindodata.common.FsStats
+logger.jindo_fsStats.level=OFF
+logger.jindo_jnative.name=com.aliyun.jindodata.jnative.NativeLogger
+logger.jindo_jnative.level=OFF
Review Comment:
@swuferhong @luoyuxia This looks odd because the open-source Fluss
distribution doesn't include any Jindo dependencies, yet the log4j
configuration is disabling Jindo's logging. Such changes should be kept
internal—for example, in `fluss-ali/fluss-ali-dist`. Moreover, we may need to
add logging configurations for other internal dependencies in the future as
well.
--
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]
