[PECO-1890] Add builder to ArrowResultChunk and other clean-up (#414)

These changes are part of team's OSS effort
- Add builder to improve readability and flexibility
- Simplify the usage of arrow classes to read arrow data
- Catch specific exceptions
- Cosmetic changes

Author: jayantsing-db

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

@@ -12,6 +12,7 @@
 import com.databricks.jdbc.common.StatementType;
 import com.databricks.jdbc.common.util.LoggingUtil;
 import com.databricks.jdbc.common.util.WarningUtil;
+import com.databricks.jdbc.exception.DatabricksParsingException;
 import com.databricks.jdbc.exception.DatabricksSQLException;
 import com.databricks.jdbc.exception.DatabricksSQLFeatureNotSupportedException;
 import com.databricks.jdbc.model.client.thrift.generated.TGetResultSetMetadataResp;
@@ -56,7 +57,8 @@ public DatabricksResultSet(
       ResultManifest resultManifest,
       StatementType statementType,
       IDatabricksSession session,
-      IDatabricksStatement parentStatement) {
+      IDatabricksStatement parentStatement)
+      throws DatabricksParsingException {
     this.statementStatus = statementStatus;
     this.statementId = statementId;
     this.executionResult =

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

@@ -9,6 +9,7 @@
 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;
 import com.databricks.jdbc.exception.DatabricksSQLException;
 import com.databricks.jdbc.exception.DatabricksSQLFeatureNotImplementedException;
 import com.databricks.jdbc.exception.DatabricksSQLFeatureNotSupportedException;
@@ -25,7 +26,8 @@ static IExecutionResult getResultSet(
       String statementId,
       IDatabricksSession session,
       IDatabricksStatement statement,
-      IDatabricksResultSet resultSet) {
+      IDatabricksResultSet resultSet)
+      throws DatabricksParsingException {
     IExecutionResult resultHandler = getResultHandler(data, manifest, statementId, session);
     if (manifest.getIsVolumeOperation() != null && manifest.getIsVolumeOperation()) {
       return new VolumeOperationResult(
@@ -42,7 +44,8 @@ static IExecutionResult getResultSet(
   }
 
   private static IExecutionResult getResultHandler(
-      ResultData data, ResultManifest manifest, String statementId, IDatabricksSession session) {
+      ResultData data, ResultManifest manifest, String statementId, IDatabricksSession session)
+      throws DatabricksParsingException {
     if (manifest.getFormat() == null) {
       throw new IllegalStateException("Empty response format");
     }