Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Add logging module, deprecate sql-show #23856

Open
Pace2Car opened this issue Jan 31, 2023 · 3 comments
Open

Add logging module, deprecate sql-show #23856

Pace2Car opened this issue Jan 31, 2023 · 3 comments

Comments

@Pace2Car
Copy link
Contributor

Pace2Car commented Jan 31, 2023

Feature Request

For English only, other languages will not accept.

Please pay attention on issues you submitted, because we maybe need more details.
If no response anymore and we cannot make decision by current information, we will close it.

Please answer these questions before submitting your issue. Thanks!

Describe the feature you would like.

In order to unify log configuration management and provide more log configurations in the future.

Add logging module under the kernel, add logging global rule, persist the log configuration to the governance center, move the porps sql-show to logger ShardingSphere-SQL's props enable

Keep using props sql-show, and mark it as deprecated

@RaigorJiang
Copy link
Contributor

Hi @Pace2Car
I can guess what you are going to do from the description, but it doesn't feel detailed enough.
Hope you can provide more information.

@Pace2Car
Copy link
Contributor Author

Hi @Pace2Car I can guess what you are going to do from the description, but it doesn't feel detailed enough. Hope you can provide more information.

The current ShardingSphere log implementation uses logback, and the log configuration only provides a few parameters to configure.In cluster mode, Proxy uses logback.xml for custom log configuration, which only takes effect on the current node and cannot be modified globally.

In this feature, I want to add the log configuration to the global rule so that the log configuration can be unified in cluster mode. Then, the log configuration can be managed by DistSQL.

In this case, the configuration of topic ShardingSphere-SQLcorresponding to sql-show will also be persisted to the governance center, so sql-show should be moved to this logger configuration.

@linghengqian
Copy link
Member

I only noticed this issue after I opened #32389. In fact, there are too many adapters for Slf4j API, at least there are third-party adapters for Log4j1, Log4j2, Logback, Reload4j, Jakarta Commons Logging and Apache Commons Logging.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

4 participants