You are viewing a plain text version of this content. The canonical link for it is here.
Posted to notifications@shardingsphere.apache.org by GitBox <gi...@apache.org> on 2022/10/26 06:41:28 UTC

[GitHub] [shardingsphere] smartbetter opened a new issue, #21764: [SQLLogger] SQLStatement log

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

   I don't need a SQLStatement, SQLStatement prints too much.
   
   Can you add a switch to the SQLStatement of SQLLogger to control whether to print? For example:
   
   `
   public final class SQLLogger {
   
       public static void logSQL(final QueryContext queryContext, final boolean showSQLStatement, final boolean showSimple, final ExecutionContext executionContext) {
           log("Logic SQL: {}", queryContext.getSql());
           if (showSQLStatement){
               log("SQLStatement: {}", queryContext.getSqlStatementContext().getSqlStatement());
           }
           if (showSimple) {
               logSimpleMode(executionContext.getExecutionUnits());
           } else {
               logNormalMode(executionContext.getExecutionUnits());
           }
       }
       ...
   }
   `


-- 
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-unsubscribe@shardingsphere.apache.org.apache.org

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


[GitHub] [shardingsphere] RaigorJiang commented on issue #21764: [SQLLogger] SQLStatement log

Posted by GitBox <gi...@apache.org>.
RaigorJiang commented on issue #21764:
URL: https://github.com/apache/shardingsphere/issues/21764#issuecomment-1305195383

   How about change the level of `SQLStatement` to debug?


-- 
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-unsubscribe@shardingsphere.apache.org

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


[GitHub] [shardingsphere] smartbetter commented on issue #21764: [SQLLogger] SQLStatement log

Posted by GitBox <gi...@apache.org>.
smartbetter commented on issue #21764:
URL: https://github.com/apache/shardingsphere/issues/21764#issuecomment-1294565524

   > You can use `sql-show: false` to close sql print.
   
   No, I need print Logic SQL and Actual SQL.


-- 
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-unsubscribe@shardingsphere.apache.org

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


[GitHub] [shardingsphere] RaigorJiang commented on issue #21764: [SQLLogger] SQLStatement log

Posted by GitBox <gi...@apache.org>.
RaigorJiang commented on issue #21764:
URL: https://github.com/apache/shardingsphere/issues/21764#issuecomment-1308224881

   @smartbetter Are you interested in submitting this change?


-- 
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-unsubscribe@shardingsphere.apache.org

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


[GitHub] [shardingsphere] smartbetter commented on issue #21764: [SQLLogger] SQLStatement log

Posted by GitBox <gi...@apache.org>.
smartbetter commented on issue #21764:
URL: https://github.com/apache/shardingsphere/issues/21764#issuecomment-1331582303

   To new issues: https://github.com/apache/shardingsphere/issues/22453


-- 
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-unsubscribe@shardingsphere.apache.org

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


[GitHub] [shardingsphere] strongduanmu commented on issue #21764: [SQLLogger] SQLStatement log

Posted by GitBox <gi...@apache.org>.
strongduanmu commented on issue #21764:
URL: https://github.com/apache/shardingsphere/issues/21764#issuecomment-1291764494

   You can use `sql-show: false` to close sql print.


-- 
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-unsubscribe@shardingsphere.apache.org

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


[GitHub] [shardingsphere] smartbetter commented on issue #21764: [SQLLogger] SQLStatement log

Posted by GitBox <gi...@apache.org>.
smartbetter commented on issue #21764:
URL: https://github.com/apache/shardingsphere/issues/21764#issuecomment-1311574811

   > @smartbetter Are you interested in submitting this change?
   
   submitting pull request?


-- 
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-unsubscribe@shardingsphere.apache.org

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


[GitHub] [shardingsphere] smartbetter commented on issue #21764: [SQLLogger] SQLStatement log

Posted by GitBox <gi...@apache.org>.
smartbetter commented on issue #21764:
URL: https://github.com/apache/shardingsphere/issues/21764#issuecomment-1307067259

   > How about change the level of `SQLStatement` to debug?
   
   I think so.


-- 
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-unsubscribe@shardingsphere.apache.org

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


[GitHub] [shardingsphere] RaigorJiang commented on issue #21764: [SQLLogger] SQLStatement log

Posted by GitBox <gi...@apache.org>.
RaigorJiang commented on issue #21764:
URL: https://github.com/apache/shardingsphere/issues/21764#issuecomment-1335995545

   Removed in #22453.


-- 
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-unsubscribe@shardingsphere.apache.org

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


[GitHub] [shardingsphere] RaigorJiang commented on issue #21764: [SQLLogger] SQLStatement log

Posted by GitBox <gi...@apache.org>.
RaigorJiang commented on issue #21764:
URL: https://github.com/apache/shardingsphere/issues/21764#issuecomment-1311768587

   > > @smartbetter Are you interested in submitting this change?
   > 
   > submitting pull request?
   
   Yes, Pull Request.


-- 
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-unsubscribe@shardingsphere.apache.org

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


[GitHub] [shardingsphere] RaigorJiang closed issue #21764: [SQLLogger] SQLStatement log

Posted by GitBox <gi...@apache.org>.
RaigorJiang closed issue #21764: [SQLLogger] SQLStatement log
URL: https://github.com/apache/shardingsphere/issues/21764


-- 
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-unsubscribe@shardingsphere.apache.org

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


[GitHub] [shardingsphere] smartbetter commented on issue #21764: [SQLLogger] SQLStatement log

Posted by GitBox <gi...@apache.org>.
smartbetter commented on issue #21764:
URL: https://github.com/apache/shardingsphere/issues/21764#issuecomment-1308231696

   > @smartbetter Are you interested in submitting this change?
   
   @RaigorJiang Yes, But I think add a switch to the SQLStatement of SQLLogger to control whether to print, That's better


-- 
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-unsubscribe@shardingsphere.apache.org

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