chore: use synchronised() isntead of lock

Author: keshavdandeva

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

@@ -40,7 +40,6 @@
 import java.util.Objects;
 import java.util.concurrent.Callable;
 import java.util.concurrent.ConcurrentHashMap;
-import java.util.concurrent.locks.ReentrantLock;
 import java.util.logging.Handler;
 import java.util.logging.Logger;
 
@@ -120,7 +119,6 @@ static class TelemetryConfig {
 
   private static final ConcurrentHashMap<String, TelemetryConfig> connectionConfigs =
       new ConcurrentHashMap<>();
-  private static final ReentrantLock lock = new ReentrantLock();
 
   private BigQueryJdbcOpenTelemetry() {}
 
@@ -142,22 +140,17 @@ private BigQueryJdbcOpenTelemetry() {}
                 }));
   }
 
-  public static void ensureGlobalHandlerAttached() {
-    lock.lock();
-    try {
-      Logger logger = Logger.getLogger(BIGQUERY_NAMESPACE);
-      boolean present = false;
-      for (Handler h : logger.getHandlers()) {
-        if (h instanceof OpenTelemetryJulHandler) {
-          present = true;
-          break;
-        }
-      }
-      if (!present) {
-        logger.addHandler(new OpenTelemetryJulHandler());
+  public static synchronized void ensureGlobalHandlerAttached() {
+    Logger logger = Logger.getLogger(BIGQUERY_NAMESPACE);
+    boolean present = false;
+    for (Handler h : logger.getHandlers()) {
+      if (h instanceof OpenTelemetryJulHandler) {
+        present = true;
+        break;
       }
-    } finally {
-      lock.unlock();
+    }
+    if (!present) {
+      logger.addHandler(new OpenTelemetryJulHandler());
     }
   }