Remove logging

java/flight/flight-jdbc-driver/src/main/java/org/apache/arrow/driver/jdbc/ArrowFlightConnection.java

@@ -62,7 +62,6 @@ private ArrowFlightConnection(final ArrowFlightJdbcDriver driver, final AvaticaF
                                 final BufferAllocator allocator,
                                 final ArrowFlightSqlClientHandler clientHandler) {
     super(driver, factory, url, properties);
-    System.out.println("Connected to " + url + "\t" + properties);
     this.config = Preconditions.checkNotNull(config, "Config cannot be null.");
     this.allocator = Preconditions.checkNotNull(allocator, "Allocator cannot be null.");
     this.clientHandler = Preconditions.checkNotNull(clientHandler, "Handler cannot be null.");
@@ -84,10 +83,8 @@ static ArrowFlightConnection createNewConnection(final ArrowFlightJdbcDriver dri
                                                    String url, final Properties properties,
                                                    final BufferAllocator allocator)
       throws SQLException {
-    System.out.println("Clearing url: " + url + "\t" + properties);
     url = replaceSemiColons(url);
     final ArrowFlightConnectionConfigImpl config = new ArrowFlightConnectionConfigImpl(properties);
-    System.out.println("Connecting to " + url + "\t" + properties);
     final ArrowFlightSqlClientHandler clientHandler = createNewClientHandler(config, allocator);
     return new ArrowFlightConnection(driver, factory, url, properties, config, allocator, clientHandler);
   }

java/flight/flight-jdbc-driver/src/main/java/org/apache/arrow/driver/jdbc/ArrowFlightJdbcFlightStreamResultSet.java

@@ -120,7 +120,6 @@ private void loadNewFlightStream() throws SQLException {
 
   @Override
   protected AvaticaResultSet execute() throws SQLException {
-    System.out.println("Executing using connection properties: " + connection.properties);
     final FlightInfo flightInfo = ((ArrowFlightInfoStatement) statement).executeFlightInfoQuery();
 
     if (flightInfo != null) {

java/flight/flight-jdbc-driver/src/test/java/org/apache/arrow/driver/jdbc/ArrowFlightJdbcDriverTest.java

@@ -214,7 +214,7 @@ public void testDriverUrlParsingMechanismShouldReturnTheDesiredArgsFromUrl() thr
 
   @SuppressWarnings("unchecked")
   @Test
-  public void testDriverUrlParsingMechanismShouldReturnTheDesiredArgsFromUrlWithSemicolon() throws Exception {
+  public void testDriverUrlParsingMechanismShouldReturnTheDesiredArgsFromUrlWithMultipleSemicolons() throws Exception {
     final Driver driver = new ArrowFlightJdbcDriver();
 
     final Method parseUrl = driver.getClass().getDeclaredMethod("getUrlsArgs", String.class);
@@ -239,6 +239,31 @@ public void testDriverUrlParsingMechanismShouldReturnTheDesiredArgsFromUrlWithSe
     assertEquals(parsedArgs.get("a"), "b");
   }
 
+  @SuppressWarnings("unchecked")
+  @Test
+  public void testDriverUrlParsingMechanismShouldReturnTheDesiredArgsFromUrlWithOneSemicolon() throws Exception {
+    final Driver driver = new ArrowFlightJdbcDriver();
+
+    final Method parseUrl = driver.getClass().getDeclaredMethod("getUrlsArgs", String.class);
+
+    parseUrl.setAccessible(true);
+
+    final Map<Object, Object> parsedArgs = (Map<Object, Object>) parseUrl.invoke(driver,
+        "jdbc:arrow-flight://localhost:2222/;key1=value1");
+
+    // Check size == the amount of args provided (prefix not included!)
+    assertEquals(5, parsedArgs.size());
+
+    // Check host == the provided host
+    assertEquals(parsedArgs.get(ArrowFlightConnectionProperty.HOST.camelName()), "localhost");
+
+    // Check port == the provided port
+    assertEquals(parsedArgs.get(ArrowFlightConnectionProperty.PORT.camelName()), 2222);
+
+    // Check all other non-default arguments
+    assertEquals(parsedArgs.get("key1"), "value1");
+  }
+
   /**
    * Tests whether an exception is thrown upon attempting to connect to a
    * malformed URI.

java/flight/flight-jdbc-driver/src/test/java/org/apache/arrow/driver/jdbc/utils/AccessorTestUtils.java

@@ -107,8 +107,7 @@ public List<Object> toList(ValueVector vector) throws Exception {
     }
 
     public <R> void assertAccessorGetter(ValueVector vector, CheckedFunction<T, R> getter,
-                                         MatcherGetter<T, R> matcherGetter)
-        throws Exception {
+                                         MatcherGetter<T, R> matcherGetter) throws Exception {
       iterate(vector, (accessor, currentRow) -> {
         R object = getter.apply(accessor);
         boolean wasNull = accessor.wasNull();