You are viewing a plain text version of this content. The canonical link for it is here.

Posted to notifications@skywalking.apache.org by GitBox <gi...@apache.org> on 2020/10/03 14:18:40 UTC

[GitHub] [skywalking] ascrutae commented on a change in pull request #5626: Limit Sql body size

ascrutae commented on a change in pull request #5626:
URL: https://github.com/apache/skywalking/pull/5626#discussion_r499151687



##########
File path: apm-sniffer/apm-sdk-plugin/jdbc-commons/src/main/java/org/apache/skywalking/apm/plugin/jdbc/JDBCPluginConfig.java
##########
@@ -53,6 +60,13 @@
              * Set a negative number to save the complete parameter string to the tag.
              */
             public static int SQL_PARAMETERS_MAX_LENGTH = 512;
+            /**
+             * For the sake of performance, SkyWalking won't save the entire sql body into the tag, but only the first
+             * {@code SQL_BODY_MAX_LENGTH} characters.
+             * <p>
+             * Set a negative number to save the complete sql body to the tag.
+             */
+            public static int SQL_BODY_MAX_LENGTH = 2048;
         }
 
         public static class MARIADB {

Review comment:
       Me too, +1




----------------------------------------------------------------
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:
users@infra.apache.org