use list

Author: laurit

instrumentation/jdbc/library/src/main/java/io/opentelemetry/instrumentation/jdbc/internal/JdbcData.java

@@ -11,8 +11,9 @@
 import java.sql.Connection;
 import java.sql.PreparedStatement;
 import java.sql.Statement;
+import java.util.ArrayList;
 import java.util.Collection;
-import java.util.LinkedHashSet;
+import java.util.List;
 import java.util.Map;
 import java.util.WeakHashMap;
 
@@ -99,20 +100,18 @@ public static Long getPreparedStatementBatchSize(PreparedStatement statement) {
    * any time.
    */
   public static final class StatementBatchInfo {
-    private final Collection<String> statements = new LinkedHashSet<>();
-    private long batchSize;
+    private final List<String> statements = new ArrayList<>();
 
     void add(String sql) {
       statements.add(sql);
-      batchSize++;
     }
 
     public Collection<String> getStatements() {
       return statements;
     }
 
     public long getBatchSize() {
-      return batchSize;
+      return statements.size();
     }
   }
 

instrumentation/jdbc/library/src/main/java/io/opentelemetry/instrumentation/jdbc/internal/OpenTelemetryStatement.java

@@ -29,8 +29,8 @@
 import java.sql.SQLException;
 import java.sql.SQLWarning;
 import java.sql.Statement;
-import java.util.Collection;
-import java.util.LinkedHashSet;
+import java.util.ArrayList;
+import java.util.List;
 
 /**
  * This class is internal and is hence not for public use. Its APIs are unstable and can change at
@@ -44,7 +44,7 @@ public class OpenTelemetryStatement<S extends Statement> implements Statement {
   protected final String query;
   protected final Instrumenter<DbRequest, Void> instrumenter;
 
-  private final Collection<String> batchCommands = new LinkedHashSet<>();
+  private final List<String> batchCommands = new ArrayList<>();
   protected long batchSize;
 
   OpenTelemetryStatement(