Apply suggestions from code review

Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>

Author: wu-sheng

CHANGES.md

@@ -24,7 +24,7 @@ Release Notes.
 * Add Elasticsearch Java client (co.elastic.clients:elasticsearch-java) plugin for 7.16.x-9.x.
 * Only publish `apm-application-toolkit` modules to Maven Central. Agent and plugins are distributed via download package and Docker images.
 * Add unified release script (`tools/releasing/release.sh`) with two-step flow: `prepare-vote` and `vote-passed`.
-* Fix `JDBCPluginConfig.Plugin.JDBC.SQL_BODY_MAX_LENGTH` was not honored by clickhouse-0.3.1 and clickhouse-0.3.2.x plugins.
+* Fix an issue where `JDBCPluginConfig.Plugin.JDBC.SQL_BODY_MAX_LENGTH` was not honored by clickhouse-0.3.1 and clickhouse-0.3.2.x plugins.
 
 All issues and pull requests are [here](https://github.com/apache/skywalking/milestone/249?closed=1)
 

apm-sniffer/apm-sdk-plugin/clickhouse-0.3.1-plugin/src/test/java/org/apache/skywalking/apm/plugin/jdbc/clickhouse/ClickHouseStatementTracingWrapperTest.java

@@ -82,7 +82,7 @@ public void shortSqlIsRecordedAsIs() throws Exception {
     @Test
     public void longSqlIsTruncatedToConfiguredLimit() throws Exception {
         JDBCPluginConfig.Plugin.JDBC.SQL_BODY_MAX_LENGTH = 16;
-        // 32 chars, twice the limit, so the helper must truncate to first 16 chars and append "..."
+        // Longer than the limit, so the helper must truncate to first 16 chars and append "..."
         String sql = "SELECT * FROM table_with_many_cols";
 
         ClickHouseStatementTracingWrapper.of(connectionInfo, "execute", sql, () -> Boolean.TRUE);

apm-sniffer/apm-sdk-plugin/clickhouse-0.3.2.x-plugin/src/test/java/org/apache/skywalking/apm/plugin/jdbc/clickhouse/ClickHousePrepareStatementTracingTest.java

@@ -83,7 +83,7 @@ public void shortSqlIsRecordedAsIs() throws Exception {
     @Test
     public void longSqlIsTruncatedToConfiguredLimit() throws Exception {
         JDBCPluginConfig.Plugin.JDBC.SQL_BODY_MAX_LENGTH = 16;
-        // 32 chars, twice the limit, so the helper must truncate to first 16 chars and append "..."
+        // Longer than the configured limit, so the helper must truncate to the first 16 chars and append "..."
         String sql = "SELECT * FROM table_with_many_cols";
 
         ClickHousePrepareStatementTracing.of(connectionInfo, "execute", sql, () -> Boolean.TRUE);