Address comments

Author: samikshya-db

databricks-sdk-java/src/main/java/com/databricks/sdk/core/DatabricksConfig.java

@@ -17,11 +17,8 @@
 import java.time.Duration;
 import java.util.*;
 import org.apache.http.HttpMessage;
-import org.slf4j.Logger;
-import org.slf4j.LoggerFactory;
 
 public class DatabricksConfig {
-  private static final Logger LOG = LoggerFactory.getLogger(DatabricksConfig.class);
   private CredentialsProvider credentialsProvider = new DefaultCredentialsProvider();
 
   @ConfigAttribute(env = "DATABRICKS_HOST")
@@ -654,19 +651,7 @@ public OpenIDConnectEndpoints getOidcEndpoints() throws IOException {
     if (discoveryUrl == null) {
       return fetchDefaultOidcEndpoints();
     }
-    try {
-      OpenIDConnectEndpoints oidcEndpoints = fetchOidcEndpointsFromDiscovery();
-      if (oidcEndpoints != null) {
-        return oidcEndpoints;
-      }
-    } catch (Exception e) {
-      LOG.warn(
-              "Failed to fetch OIDC Endpoints using discovery URL: {}. Error: {}. \nDefaulting to fetch OIDC using default endpoint.",
-              discoveryUrl,
-              e.getMessage(),
-              e);
-    }
-    return fetchDefaultOidcEndpoints();
+    return fetchOidcEndpointsFromDiscovery();
   }
 
   private OpenIDConnectEndpoints fetchOidcEndpointsFromDiscovery() {
@@ -756,7 +741,6 @@ public DatabricksEnvironment getDatabricksEnvironment() {
   }
 
   private DatabricksConfig clone(Set<String> fieldsToSkip) {
-    fieldsToSkip.add("LOG");
     DatabricksConfig newConfig = new DatabricksConfig();
     for (Field f : DatabricksConfig.class.getDeclaredFields()) {
       if (fieldsToSkip.contains(f.getName())) {

databricks-sdk-java/src/test/java/com/databricks/sdk/core/DatabricksConfigTest.java

@@ -166,35 +166,6 @@ public void testDiscoveryEndpoint() throws IOException {
     }
   }
 
-  @Test
-  public void testDiscoveryEndpointFetchFallback() throws IOException {
-    String discoveryUrlSuffix = "/test.discovery.url";
-    String OIDCResponse =
-        "{\n"
-            + "  \"authorization_endpoint\": \"https://test.auth.endpoint/oidc/v1/authorize\",\n"
-            + "  \"token_endpoint\": \"https://test.auth.endpoint/oidc/v1/token\"\n"
-            + "}";
-
-    try (FixtureServer server =
-        new FixtureServer()
-            .with("GET", discoveryUrlSuffix, "", 400)
-            .with("GET", "/oidc/.well-known/oauth-authorization-server", OIDCResponse, 200)) {
-
-      String discoveryUrl = server.getUrl() + discoveryUrlSuffix;
-
-      OpenIDConnectEndpoints oidcEndpoints =
-          new DatabricksConfig()
-              .setHost(server.getUrl())
-              .setDiscoveryUrl(discoveryUrl)
-              .setHttpClient(new CommonsHttpClient.Builder().withTimeoutSeconds(30).build())
-              .getOidcEndpoints();
-
-      assertEquals(
-          "https://test.auth.endpoint/oidc/v1/authorize", oidcEndpoints.getAuthorizationEndpoint());
-      assertEquals("https://test.auth.endpoint/oidc/v1/token", oidcEndpoints.getTokenEndpoint());
-    }
-  }
-
   @Test
   public void testNewWithWorkspaceHost() {
     DatabricksConfig config =