Remove trailing &

Author: girarda

airbyte-integrations/connectors/destination-mysql/src/main/java/io/airbyte/integrations/destination/mysql/MySQLDestination.java

@@ -109,7 +109,7 @@ public JsonNode toJdbcConfig(final JsonNode config) {
     // since zero dates are placeholders, we convert them to null by default
     if (!additionalParameters.isEmpty()) {
       jdbcUrl.append("?");
-      additionalParameters.forEach(x -> jdbcUrl.append(x).append("&"));
+      jdbcUrl.append(String.join("&", additionalParameters));
     }
 
     final ImmutableMap.Builder<Object, Object> configBuilder = ImmutableMap.builder()

airbyte-integrations/connectors/destination-mysql/src/test/java/io/airbyte/integrations/destination/mysql/MySQLDestinationTest.java

@@ -68,14 +68,14 @@ private JsonNode buildConfigNoExtraJdbcParametersWithoutSsl() {
   void testNoExtraParams() {
     final JsonNode jdbcConfig = getDestination().toJdbcConfig(buildConfigNoJdbcParameters());
     final String url = jdbcConfig.get("jdbc_url").asText();
-    assertEquals("jdbc:mysql://localhost:1337/db?verifyServerCertificate=false&zeroDateTimeBehavior=convertToNull&requireSSL=true&useSSL=true&", url);
+    assertEquals("jdbc:mysql://localhost:1337/db?verifyServerCertificate=false&zeroDateTimeBehavior=convertToNull&requireSSL=true&useSSL=true", url);
   }
 
   @Test
   void testEmptyExtraParams() {
     final JsonNode jdbcConfig = getDestination().toJdbcConfig(buildConfigWithExtraJdbcParameters(""));
     final String url = jdbcConfig.get("jdbc_url").asText();
-    assertEquals("jdbc:mysql://localhost:1337/db?verifyServerCertificate=false&zeroDateTimeBehavior=convertToNull&requireSSL=true&useSSL=true&", url);
+    assertEquals("jdbc:mysql://localhost:1337/db?verifyServerCertificate=false&zeroDateTimeBehavior=convertToNull&requireSSL=true&useSSL=true", url);
   }
 
   @Test
@@ -84,7 +84,7 @@ void testExtraParams() {
     final JsonNode jdbcConfig = getDestination().toJdbcConfig(buildConfigWithExtraJdbcParameters(extraParam));
     final String url = jdbcConfig.get("jdbc_url").asText();
     assertEquals(
-        "jdbc:mysql://localhost:1337/db?key1=value1&key2=value2&key3=value3&verifyServerCertificate=false&zeroDateTimeBehavior=convertToNull&requireSSL=true&useSSL=true&",
+        "jdbc:mysql://localhost:1337/db?key1=value1&key2=value2&key3=value3&verifyServerCertificate=false&zeroDateTimeBehavior=convertToNull&requireSSL=true&useSSL=true",
         url);
   }
 
@@ -113,7 +113,7 @@ void testExtraParameterNoSsl() {
     final JsonNode jdbcConfig = getDestination().toJdbcConfig(buildConfigWithExtraJdbcParametersWithNoSsl(extraParam));
     final String url = jdbcConfig.get("jdbc_url").asText();
     assertEquals(
-        "jdbc:mysql://localhost:1337/db?key1=value1&key2=value2&key3=value3&zeroDateTimeBehavior=convertToNull&",
+        "jdbc:mysql://localhost:1337/db?key1=value1&key2=value2&key3=value3&zeroDateTimeBehavior=convertToNull",
         url);
   }
 
@@ -122,7 +122,7 @@ void testNoExtraParameterNoSsl() {
     final JsonNode jdbcConfig = getDestination().toJdbcConfig(buildConfigNoExtraJdbcParametersWithoutSsl());
     final String url = jdbcConfig.get("jdbc_url").asText();
     assertEquals(
-        "jdbc:mysql://localhost:1337/db?zeroDateTimeBehavior=convertToNull&",
+        "jdbc:mysql://localhost:1337/db?zeroDateTimeBehavior=convertToNull",
         url);
   }