Hybrid Results are enabled by default in SEA Mode (#771)

* Made Hybrid results default

* Fixed unit test

* Added Fake service stubs

* Added stubs for Default hybrid mode

* Reverted to old stubs

* Reverted stubs

* Fix

Author: jprakash-db

src/main/java/com/databricks/jdbc/common/DatabricksJdbcUrlParams.java

@@ -91,7 +91,7 @@ public enum DatabricksJdbcUrlParams {
   ENABLE_REQUEST_TRACING("EnableRequestTracing", "flag to enable request tracing", "0"),
   HTTP_CONNECTION_POOL_SIZE("HttpConnectionPoolSize", "Maximum HTTP connection pool size", "100"),
   ENABLE_SQL_EXEC_HYBRID_RESULTS(
-      "EnableSQLExecHybridResults", "flag to enable hybrid results", "0"),
+      "EnableSQLExecHybridResults", "flag to enable hybrid results", "1"),
   ENABLE_COMPLEX_DATATYPE_SUPPORT(
       "EnableComplexDatatypeSupport",
       "flag to enable native support of complex data types as java objects",

src/test/java/com/databricks/jdbc/dbclient/impl/sqlexec/DatabricksSdkClientTest.java

@@ -86,7 +86,7 @@ private void setupClientMocks(boolean includeResults, boolean async) {
             .setSessionId(SESSION_ID)
             .setWarehouseId(WAREHOUSE_ID)
             .setStatement(STATEMENT)
-            .setDisposition(Disposition.EXTERNAL_LINKS)
+            .setDisposition(Disposition.INLINE_OR_EXTERNAL_LINKS)
             .setFormat(Format.ARROW_STREAM)
             .setRowLimit(100L)
             .setParameters(params);

src/test/java/com/databricks/jdbc/integration/IntegrationTestUtil.java

@@ -171,6 +171,8 @@ public static Connection getValidJDBCConnection() throws SQLException {
     Properties connectionProperties = new Properties();
     connectionProperties.put(DatabricksJdbcUrlParams.USER.getParamName(), getDatabricksUser());
     connectionProperties.put(DatabricksJdbcUrlParams.PASSWORD.getParamName(), getDatabricksToken());
+    connectionProperties.put(
+        DatabricksJdbcUrlParams.ENABLE_SQL_EXEC_HYBRID_RESULTS.getParamName(), '0');
 
     if (DriverUtil.isRunningAgainstFake()) {
       connectionProperties.put(
@@ -202,6 +204,8 @@ public static Connection getValidJDBCConnection(Properties connectionProperties)
       throws SQLException {
     connectionProperties.put(DatabricksJdbcUrlParams.USER.getParamName(), getDatabricksUser());
     connectionProperties.put(DatabricksJdbcUrlParams.PASSWORD.getParamName(), getDatabricksToken());
+    connectionProperties.put(
+        DatabricksJdbcUrlParams.ENABLE_SQL_EXEC_HYBRID_RESULTS.getParamName(), '0');
 
     if (DriverUtil.isRunningAgainstFake()) {
       connectionProperties.put(

src/test/java/com/databricks/jdbc/integration/fakeservice/tests/UCVolumeIntegrationTests.java

@@ -355,6 +355,7 @@ private Connection getConnection() throws SQLException {
     Properties connProps = new Properties();
     connProps.put(DatabricksJdbcUrlParams.USER.getParamName(), getDatabricksUser());
     connProps.put(DatabricksJdbcUrlParams.PASSWORD.getParamName(), getDatabricksToken());
+    connProps.put(DatabricksJdbcUrlParams.ENABLE_SQL_EXEC_HYBRID_RESULTS.getParamName(), '0');
     connProps.put(
         DatabricksJdbcUrlParams.CONN_CATALOG.getParamName(),
         FakeServiceConfigLoader.getProperty(DatabricksJdbcUrlParams.CONN_CATALOG.getParamName()));