Minor suggestion on InlineJsonResult (#519)

Author: jackyhu-db

src/main/java/com/databricks/jdbc/api/impl/ExecutionResultFactory.java

@@ -6,7 +6,6 @@
 import com.databricks.jdbc.api.callback.IDatabricksResultSetHandle;
 import com.databricks.jdbc.api.callback.IDatabricksStatementHandle;
 import com.databricks.jdbc.api.impl.arrow.ArrowStreamResult;
-import com.databricks.jdbc.api.impl.inline.InlineJsonResult;
 import com.databricks.jdbc.api.impl.volume.VolumeOperationResult;
 import com.databricks.jdbc.common.util.DatabricksThriftUtil;
 import com.databricks.jdbc.exception.DatabricksParsingException;

…bc/api/impl/inline/InlineJsonResult.java …icks/jdbc/api/impl/InlineJsonResult.javasrc/main/java/com/databricks/jdbc/api/impl/inline/InlineJsonResult.java renamed to src/main/java/com/databricks/jdbc/api/impl/InlineJsonResult.java src/main/java/com/databricks/jdbc/api/impl/inline/InlineJsonResult.java renamed to src/main/java/com/databricks/jdbc/api/impl/InlineJsonResult.java

@@ -1,6 +1,5 @@
-package com.databricks.jdbc.api.impl.inline;
+package com.databricks.jdbc.api.impl;
 
-import com.databricks.jdbc.api.impl.IExecutionResult;
 import com.databricks.jdbc.exception.DatabricksSQLException;
 import com.databricks.jdbc.model.core.ResultData;
 import com.databricks.jdbc.model.core.ResultManifest;
@@ -15,18 +14,14 @@ public class InlineJsonResult implements IExecutionResult {
   private boolean isClosed;
 
   public InlineJsonResult(ResultManifest resultManifest, ResultData resultData) {
-    this.data = getDataList(resultData.getDataArray());
-    this.currentRow = -1;
-    this.isClosed = false;
+    this(getDataList(resultData.getDataArray()));
   }
 
   public InlineJsonResult(Object[][] rows) {
-    this.data =
+    this(
         Arrays.stream(rows)
             .map(row -> Arrays.stream(row).collect(Collectors.toList()))
-            .collect(Collectors.toList());
-    this.currentRow = -1;
-    this.isClosed = false;
+            .collect(Collectors.toList()));
   }
 
   public InlineJsonResult(List<List<Object>> rows) {
@@ -53,7 +48,7 @@ private static List<List<Object>> getDataList(Collection<Collection<String>> dat
   @Override
   public Object getObject(int columnIndex) throws DatabricksSQLException {
     if (isClosed()) {
-      throw new DatabricksSQLException("Method called on closed result");
+      throw new DatabricksSQLException("Result is already closed");
     }
     if (currentRow == -1) {
       throw new DatabricksSQLException("Cursor is before first row");

src/test/java/com/databricks/jdbc/api/impl/DatabricksResultSetTest.java

@@ -9,7 +9,6 @@
 import com.databricks.jdbc.api.IDatabricksSession;
 import com.databricks.jdbc.api.callback.IDatabricksResultSetHandle;
 import com.databricks.jdbc.api.callback.IDatabricksStatementHandle;
-import com.databricks.jdbc.api.impl.inline.InlineJsonResult;
 import com.databricks.jdbc.common.StatementType;
 import com.databricks.jdbc.exception.DatabricksSQLException;
 import com.databricks.jdbc.exception.DatabricksSQLFeatureNotSupportedException;

src/test/java/com/databricks/jdbc/api/impl/ExecutionResultFactoryTest.java

@@ -8,7 +8,6 @@
 import com.databricks.jdbc.api.callback.IDatabricksResultSetHandle;
 import com.databricks.jdbc.api.callback.IDatabricksStatementHandle;
 import com.databricks.jdbc.api.impl.arrow.ArrowStreamResult;
-import com.databricks.jdbc.api.impl.inline.InlineJsonResult;
 import com.databricks.jdbc.api.impl.volume.VolumeOperationResult;
 import com.databricks.jdbc.exception.DatabricksParsingException;
 import com.databricks.jdbc.exception.DatabricksSQLFeatureNotSupportedException;