Respond to comments from @wcekan and @aklish

elide-core/src/main/java/com/yahoo/elide/Elide.java

@@ -51,7 +51,6 @@
  */
 @Slf4j
 public class Elide {
-
     @Getter private final ElideSettings elideSettings;
     @Getter private final AuditLogger auditLogger;
     @Getter private final DataStore dataStore;

elide-core/src/main/java/com/yahoo/elide/ElideResponse.java

@@ -7,23 +7,37 @@
 
 import lombok.Getter;
 
+import java.util.Optional;
+
 /**
  * Elide response object.
  */
 public class ElideResponse {
     @Getter private final int responseCode;
     @Getter private final String body;
-    @Getter private final Throwable failureReason;
+    @Getter private final Optional<Throwable> failureReason;
+
+    /**
+     * Constructor.
+     *
+     * @param responseCode HTTP response code
+     * @param body returned body string
+     */
+    @Deprecated
+    public ElideResponse(int responseCode, String body) {
+        this(responseCode, body, null);
+    }
 
     /**
      * Constructor.
      *
      * @param responseCode HTTP response code
      * @param body returned body string
+     * @param failureReason the reason the request failed
      */
     public ElideResponse(int responseCode, String body, Throwable failureReason) {
         this.responseCode = responseCode;
         this.body = body;
-        this.failureReason = failureReason;
+        this.failureReason = Optional.ofNullable(failureReason);
     }
 }

elide-graphql/src/main/java/com/yahoo/elide/graphql/GraphQLEndpoint.java

@@ -10,6 +10,7 @@
 import com.fasterxml.jackson.databind.ObjectMapper;
 import com.fasterxml.jackson.databind.node.ArrayNode;
 import com.fasterxml.jackson.databind.node.JsonNodeFactory;
+import com.google.common.collect.ImmutableMap;
 import com.yahoo.elide.Elide;
 import com.yahoo.elide.ElideSettings;
 import com.yahoo.elide.core.DataStoreTransaction;
@@ -70,7 +71,6 @@ public class GraphQLEndpoint {
     public GraphQLEndpoint(
             @Named("elide") Elide elide,
             @Named("elideUserExtractionFunction") DefaultOpaqueUserFunction getUser) {
-        log.error("Started ~~");
         this.elide = elide;
         this.elideSettings = elide.getElideSettings();
         this.getUser = getUser;
@@ -179,13 +179,12 @@ private Response executeGraphQLRequest(
             requestScope.getPermissionExecutor().executeCommitChecks();
             if (query.trim().startsWith(MUTATION)) {
                 if (!result.getErrors().isEmpty()) {
-                    HashMap<String, Object> abortedResponseObject = new HashMap<String, Object>() {
-                        {
-                            put("errors", result.getErrors());
-                            put("data", null);
-                        }
-                    };
+                    Map<String, Object> abortedResponseObject = ImmutableMap.of(
+                            "errors", result.getErrors(),
+                            "data", null
+                    );
                     // Do not commit. Throw OK response to process tx.close correctly.
+                    // (default transaction implementations throw an IOException if you leave a dangling SQL transaction)
                     throw new WebApplicationException(
                             Response.ok(mapper.writeValueAsString(abortedResponseObject)).build());
                 }