SNOW-1491028 : Snowflake JDBC Driver 4.2 compliant issue. (#1857)

src/main/java/net/snowflake/client/jdbc/SnowflakeConnectionV1.java

@@ -597,7 +597,16 @@ public int getHoldability() throws SQLException {
   @Override
   public void setHoldability(int holdability) throws SQLException {
 
-    throw new SnowflakeLoggedFeatureNotSupportedException(sfSession);
+    raiseSQLExceptionIfConnectionIsClosed();
+    if ((holdability != ResultSet.CLOSE_CURSORS_AT_COMMIT
+        && holdability != ResultSet.HOLD_CURSORS_OVER_COMMIT)) {
+      throw new SQLException("The given parameter is not a ResultSet holdability constant.");
+    }
+    // HOLD_CURSORS_OVER_COMMIT holdability is currently not supported.
+    // no-op if the holdability is CLOSE_CURSORS_AT_COMMIT
+    if (holdability == ResultSet.HOLD_CURSORS_OVER_COMMIT) {
+      throw new SnowflakeLoggedFeatureNotSupportedException(sfSession);
+    }
   }
 
   @Override

src/test/java/net/snowflake/client/jdbc/ConnectionFeatureNotSupportedIT.java

@@ -43,6 +43,8 @@ public void testFeatureNotSupportedException() throws Throwable {
       expectFeatureNotSupportedException(connection::createSQLXML);
       expectFeatureNotSupportedException(
           () -> connection.createStruct("fakeType", new Object[] {}));
+      expectFeatureNotSupportedException(
+          () -> connection.setHoldability(ResultSet.HOLD_CURSORS_OVER_COMMIT));
     }
   }
 

src/test/java/net/snowflake/client/jdbc/ConnectionIT.java

@@ -743,7 +743,7 @@ public void testGetTypeMap() throws Throwable {
   public void testHolderbility() throws Throwable {
     try (Connection connection = getConnection()) {
       try {
-        connection.setHoldability(0);
+        connection.setHoldability(ResultSet.HOLD_CURSORS_OVER_COMMIT);
       } catch (SQLFeatureNotSupportedException ex) {
         // nop
       }

src/test/java/net/snowflake/client/jdbc/ConnectionLatestIT.java

@@ -37,6 +37,7 @@
 import java.sql.PreparedStatement;
 import java.sql.ResultSet;
 import java.sql.SQLException;
+import java.sql.SQLFeatureNotSupportedException;
 import java.sql.Statement;
 import java.time.Duration;
 import java.util.ArrayList;
@@ -1589,6 +1590,22 @@ public void shouldGetDifferentTimestampLtzConsistentBetweenFormats() throws Exce
     }
   }
 
+  @Test
+  public void testSetHoldability() throws Throwable {
+    try (Connection connection = getConnection()) {
+      try {
+        connection.setHoldability(ResultSet.CLOSE_CURSORS_AT_COMMIT);
+      } catch (SQLFeatureNotSupportedException ex) {
+        fail("should not fail");
+      }
+      // return an empty type map. setTypeMap is not supported.
+      assertEquals(ResultSet.CLOSE_CURSORS_AT_COMMIT, connection.getHoldability());
+      connection.close();
+      expectConnectionAlreadyClosedException(
+          () -> connection.setHoldability(ResultSet.CLOSE_CURSORS_AT_COMMIT));
+    }
+  }
+
   /** Added in > 3.14.5 and modified in > 3.18.0 */
   @Test
   public void shouldGetOverridenConnectionAndSocketTimeouts() throws Exception {

src/test/java/net/snowflake/client/pooling/LogicalConnectionFeatureNotSupportedLatestIT.java

@@ -66,8 +66,6 @@ public void testLogicalConnectionFeatureNotSupported() throws SQLException {
     expectFeatureNotSupportedException(logicalConnection::createBlob);
     expectFeatureNotSupportedException(logicalConnection::createNClob);
     expectFeatureNotSupportedException(logicalConnection::createSQLXML);
-    expectFeatureNotSupportedException(
-        () -> logicalConnection.setHoldability(ResultSet.CLOSE_CURSORS_AT_COMMIT));
     expectFeatureNotSupportedException(
         () -> logicalConnection.setHoldability(ResultSet.HOLD_CURSORS_OVER_COMMIT));
     expectFeatureNotSupportedException(