address gemini comments

Author: Neenu1995

java-bigquery/google-cloud-bigquery-jdbc/src/main/java/com/google/cloud/bigquery/jdbc/PerConnectionFileHandler.java

@@ -37,7 +37,7 @@ class PerConnectionFileHandler extends Handler {
   private FileHandler defaultHandler;
 
   PerConnectionFileHandler(String baseLogPath, Level level) {
-    this.baseLogPath = baseLogPath;
+    this.baseLogPath = baseLogPath != null ? baseLogPath : "";
     this.level = level;
 
     try {
@@ -53,7 +53,8 @@ class PerConnectionFileHandler extends Handler {
       this.defaultHandler.setLevel(level);
       this.defaultHandler.setFormatter(BigQueryJdbcRootLogger.getFormatter());
     } catch (IOException e) {
-      System.err.println("Failed to initialize default log file: " + e.getMessage());
+      reportError(
+          "Failed to initialize default log file", e, java.util.logging.ErrorManager.OPEN_FAILURE);
     }
   }
 
@@ -86,8 +87,10 @@ public void publish(LogRecord record) {
                   fh.setFormatter(BigQueryJdbcRootLogger.getFormatter());
                   return fh;
                 } catch (IOException e) {
-                  System.err.println(
-                      "Failed to create log file for connection " + id + ": " + e.getMessage());
+                  reportError(
+                      "Failed to create log file for connection " + id,
+                      e,
+                      java.util.logging.ErrorManager.OPEN_FAILURE);
                   return defaultHandler;
                 }
               });
@@ -108,15 +111,19 @@ public void flush() {
     }
   }
 
-  @Override
   @Override
   public void close() throws SecurityException {
     for (FileHandler h : handlers.values()) {
-      try { h.close(); } catch (Exception e) {}
+      try {
+        h.close();
+      } catch (Exception e) {
+      }
     }
     try {
       if (defaultHandler != null) defaultHandler.close();
-    } finally { handlers.clear(); }
+    } finally {
+      handlers.clear();
+    }
   }
 
   public void closeHandler(String connectionId) {

java-bigquery/google-cloud-bigquery-jdbc/src/test/java/com/google/cloud/bigquery/jdbc/PerConnectionFileHandlerTest.java

@@ -31,8 +31,7 @@
 
 public class PerConnectionFileHandlerTest {
 
-  @TempDir
-  Path tempDir;
+  @TempDir Path tempDir;
 
   private PerConnectionFileHandler handler;
   private BigQueryConnection mockConnection;
@@ -72,7 +71,7 @@ public void testPublishDefault() throws IOException {
   @Test
   public void testPublishConnectionSpecific() throws IOException {
     BigQueryJdbcMdc.registerInstance(mockConnection, "123");
-    
+
     LogRecord record = new LogRecord(Level.INFO, "Test message connection 123");
     handler.publish(record);
     handler.flush();
@@ -86,7 +85,7 @@ public void testPublishConnectionSpecific() throws IOException {
   @Test
   public void testCloseHandler() {
     BigQueryJdbcMdc.registerInstance(mockConnection, "456");
-    
+
     LogRecord record = new LogRecord(Level.INFO, "Test message connection 456");
     handler.publish(record);
     handler.flush();