fix - Fallback to Buildship when failed to generate named pipe (#1576)

extension/jdtls.ext/com.microsoft.gradle.bs.importer/src/com/microsoft/gradle/bs/importer/GradleBuildServerProjectImporter.java

@@ -191,7 +191,6 @@ public class GradleBuildServerProjectImporter extends AbstractProjectImporter {
         IPath rootPath = ResourceUtils.filePathFromURI(rootFolder.toURI().toString());
         BuildServerConnection buildServer = ImporterPlugin.getBuildServerConnection(rootPath, true);
         if (buildServer == null) {
-            JavaLanguageServerPlugin.logError("Reach the maximum number of attempts to connect to the build server, use BuildShip instead");
             this.isResolved = false;
             return;
         }

extension/jdtls.ext/com.microsoft.gradle.bs.importer/src/com/microsoft/gradle/bs/importer/ImporterPlugin.java

@@ -11,6 +11,7 @@ import org.eclipse.core.runtime.IPath;
 import org.eclipse.core.runtime.IStatus;
 import org.eclipse.core.runtime.Plugin;
 import org.eclipse.core.runtime.Status;
+import org.eclipse.jdt.ls.core.internal.JavaLanguageServerPlugin;
 import org.eclipse.jdt.ls.core.internal.managers.DigestStore;
 import org.eclipse.lsp4j.jsonrpc.Launcher;
 import org.osgi.framework.BundleContext;
@@ -113,6 +114,7 @@ public class ImporterPlugin extends Plugin {
             instance.buildServers.put(rootPath, Pair.of(server, client));
             return server;
         } catch (NamedPipeConnectionException e) {
+            JavaLanguageServerPlugin.logException("Failed to connect to build server using named pipe.", e);
             return null;
         } catch (IOException e) {
             throw new CoreException(new Status(IStatus.ERROR, PLUGIN_ID,

extension/jdtls.ext/com.microsoft.gradle.bs.importer/src/com/microsoft/gradle/bs/importer/NamedPipeStream.java

@@ -92,7 +92,7 @@ public class NamedPipeStream {
             Utils.sendTelemetry(JavaLanguageServerPlugin.getProjectsManager().getConnection(),
                         telemetry);
             if (attempts == MAX_ATTEMPTS) {
-                throw new NamedPipeConnectionException("Failed to connect to extension", MAX_ATTEMPTS);
+                throw new NamedPipeConnectionException(String.format("Failed to connect to extension, Max attempts: %d", MAX_ATTEMPTS));
             }
         }
 
@@ -122,7 +122,7 @@ public class NamedPipeStream {
             int bytesLength = Math.min(availableLength / 2, randomLength);
 
             if (bytesLength < 16) {
-                throw new IllegalArgumentException("Unable to generate a random pipe name with character length less than 16");
+                throw new NamedPipeConnectionException("Unable to generate a random pipe name with character length less than 16");
             }
             return Paths.get(tmpDir, generateRandomHex(bytesLength) + ".sock").toString();
         }

extension/jdtls.ext/com.microsoft.gradle.bs.importer/src/com/microsoft/gradle/bs/importer/model/NamedPipeConnectionException.java

@@ -1,7 +1,7 @@
 package com.microsoft.gradle.bs.importer.model;
 
 public class NamedPipeConnectionException extends RuntimeException {
-  public NamedPipeConnectionException(String message, int maxAttempts) {
-    super(String.format("%s, Max attempts: %d", message, maxAttempts));
+  public NamedPipeConnectionException(String message) {
+    super(message);
   }
 }