commit 95110991e154f84c6b08a38d65a684d18bb6adcc
author Googler <noreply@google.com> Mon Jan 07 02:39:16 2019 -0800
committer Copybara-Service <copybara-piper@google.com> Mon Jan 07 02:41:17 2019 -0800
tree 820b546493f4218de97d7d103c9dd272091070f1
parent eb5598c8076aabf23cad7bcded6a19950876a0e6

Automatic code cleanup.

PiperOrigin-RevId: 228130835

src/main/java/com/google/devtools/build/lib/runtime/BlazeCommandDispatcher.java

diff --git a/src/main/java/com/google/devtools/build/lib/runtime/BlazeCommandDispatcher.java b/src/main/java/com/google/devtools/build/lib/runtime/BlazeCommandDispatcher.java
index d76738f..fb5ebf2 100644
--- a/src/main/java/com/google/devtools/build/lib/runtime/BlazeCommandDispatcher.java
+++ b/src/main/java/com/google/devtools/build/lib/runtime/BlazeCommandDispatcher.java
@@ -226,6 +226,23 @@
     }
   }
 
+  /**
+   * For testing ONLY. Same as {@link #exec(InvocationPolicy, List, OutErr, LockingMode, String,
+   * long, Optional<List<Pair<String, String>>>)}, but automatically uses the current time.
+   */
+  @VisibleForTesting
+  public BlazeCommandResult exec(List<String> args, String clientDescription, OutErr originalOutErr)
+      throws InterruptedException {
+    return exec(
+        InvocationPolicy.getDefaultInstance(),
+        args,
+        originalOutErr,
+        LockingMode.ERROR_OUT,
+        clientDescription,
+        runtime.getClock().currentTimeMillis(),
+        Optional.empty() /* startupOptionBundles */);
+  }
+
   private BlazeCommandResult execExclusively(
       OriginalUnstructuredCommandLineEvent unstructuredServerCommandLineEvent,
       InvocationPolicy invocationPolicy,
@@ -526,23 +543,6 @@
     return BlazeCommandResult.exitCode(earlyExitCode);
   }
 
-  /**
-   * For testing ONLY. Same as {@link #exec(InvocationPolicy, List, OutErr, LockingMode, String,
-   * long, Optional<List<Pair<String, String>>>)}, but automatically uses the current time.
-   */
-  @VisibleForTesting
-  public BlazeCommandResult exec(List<String> args, String clientDescription, OutErr originalOutErr)
-      throws InterruptedException {
-    return exec(
-        InvocationPolicy.getDefaultInstance(),
-        args,
-        originalOutErr,
-        LockingMode.ERROR_OUT,
-        clientDescription,
-        runtime.getClock().currentTimeMillis(),
-        Optional.empty() /* startupOptionBundles */);
-  }
-
   private OutErr bufferOut(OutErr outErr, boolean fully) {
     OutputStream wrappedOut;
     if (fully) {

src/main/java/com/google/devtools/build/lib/runtime/BlazeCommandResult.java

diff --git a/src/main/java/com/google/devtools/build/lib/runtime/BlazeCommandResult.java b/src/main/java/com/google/devtools/build/lib/runtime/BlazeCommandResult.java
index de861a5..e053b26 100644
--- a/src/main/java/com/google/devtools/build/lib/runtime/BlazeCommandResult.java
+++ b/src/main/java/com/google/devtools/build/lib/runtime/BlazeCommandResult.java
@@ -45,6 +45,10 @@
     return shutdown;
   }
 
+  public static BlazeCommandResult shutdown(ExitCode exitCode) {
+    return new BlazeCommandResult(exitCode, null, true);
+  }
+
   @Nullable public ExecRequest getExecRequest() {
     return execDescription;
   }
@@ -53,9 +57,6 @@
     return new BlazeCommandResult(exitCode, null, false);
   }
 
-  public static BlazeCommandResult shutdown(ExitCode exitCode) {
-    return new BlazeCommandResult(exitCode, null, true);
-  }
   public static BlazeCommandResult execute(ExecRequest execDescription) {
     return new BlazeCommandResult(
         ExitCode.SUCCESS, Preconditions.checkNotNull(execDescription), false);

src/main/java/com/google/devtools/build/lib/runtime/BlazeRuntime.java

diff --git a/src/main/java/com/google/devtools/build/lib/runtime/BlazeRuntime.java b/src/main/java/com/google/devtools/build/lib/runtime/BlazeRuntime.java
index 437a04ea..b1a34d6 100644
--- a/src/main/java/com/google/devtools/build/lib/runtime/BlazeRuntime.java
+++ b/src/main/java/com/google/devtools/build/lib/runtime/BlazeRuntime.java
@@ -990,7 +990,8 @@
                 startupOptions.shutdownOnLowSysMem,
                 startupOptions.idleServerTasks);
       } catch (ReflectiveOperationException | IllegalArgumentException e) {
-        throw new AbruptExitException("gRPC server not compiled in", ExitCode.BLAZE_INTERNAL_ERROR);
+        throw new AbruptExitException(
+            "gRPC server not compiled in", ExitCode.BLAZE_INTERNAL_ERROR, e);
       }
 
       // Register the signal handler.

src/main/java/com/google/devtools/build/lib/runtime/ClientOptions.java

diff --git a/src/main/java/com/google/devtools/build/lib/runtime/ClientOptions.java b/src/main/java/com/google/devtools/build/lib/runtime/ClientOptions.java
index db6f988..431c4fb 100644
--- a/src/main/java/com/google/devtools/build/lib/runtime/ClientOptions.java
+++ b/src/main/java/com/google/devtools/build/lib/runtime/ClientOptions.java
@@ -78,7 +78,7 @@
       try {
         blazeRc = Integer.valueOf(input.substring(0, colonPos));
       } catch (NumberFormatException e) {
-        throw new OptionsParsingException(ERROR_MESSAGE);
+        throw new OptionsParsingException(ERROR_MESSAGE, e);
       }
 
       if (blazeRc < 0) {