RaigorJiang opened a new issue, #35385:
URL: https://github.com/apache/shardingsphere/issues/35385

   Related to #24046, #32389
   
   The original intention of `shardingsphere-logging` is to allow users to 
dynamically modify the cluster's log configuration, such as change the log 
level or add new logger, through DistSQL, to avoid modifying log files, such as 
logback.xml, on each computing node.
   
   Currently, it is indeed possible to dynamically set the log level through 
DistSQL, for example.
   ```sql
   set dist variable system-log-level = 'DEBUG';
   ```
   
   However, if you want to use DistSQL to implement the full functionality of 
the log configuration file, its syntax is too complex and difficult to use.
   
   In addition, the most important thing is two limitations:
   1. The logging module needs to instantiate a specific log configuration 
object, which will bind it to the log implementation, resulting in a strong 
dependency;
   
   2. Log components such as logback have their own configuration priority 
strategy. When logback.xml exists, the configuration in the code is actually 
invalid.
   
   Therefore, the `shardingsphere-logging` module cannot meet the functional 
requirements. It is recommended to remove it and restore to the xml 
configuration method.
   
   If the user needs to dynamically update the configuration, use the method 
provided by the log component itself, such as file scan.
   


-- 
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: 
notifications-unsubscr...@shardingsphere.apache.org.apache.org

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

Reply via email to