deniskuzZ commented on code in PR #5661:
URL: https://github.com/apache/hive/pull/5661#discussion_r1978303430


##########
standalone-metastore/metastore-server/src/main/java/org/apache/hadoop/hive/metastore/RawStore.java:
##########
@@ -140,17 +140,6 @@ public interface RawStore extends Configurable {
    */
   boolean openTransaction();
 
-  /**
-   * Opens a new one or the one already created. Every call of this function 
must
-   * have corresponding commit or rollback function call.
-   *
-   * @param isolationLevel The transaction isolation level. Only possible to 
set on the first call.
-   * @return an active transaction
-   */
-  default boolean openTransaction(String isolationLevel) {
-    throw new UnsupportedOperationException("Setting isolation level for this 
Store is not supported");
-  }
-

Review Comment:
   hi @pvary, do you recall why a new signature for `openTransaction` with 
`isolation` level param was introduced in 
[HIVE-26882](https://issues.apache.org/jira/browse/HIVE-26882)? 



-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: gitbox-unsubscr...@hive.apache.org

For queries about this service, please contact Infrastructure at:
us...@infra.apache.org


---------------------------------------------------------------------
To unsubscribe, e-mail: gitbox-unsubscr...@hive.apache.org
For additional commands, e-mail: gitbox-h...@hive.apache.org

Reply via email to