deniskuzZ commented on code in PR #5661: URL: https://github.com/apache/hive/pull/5661#discussion_r1978314877
########## 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: oh, we removed the use of it in https://github.com/apache/hive/pull/5129/files#diff-394eb2401c3389f5de729f3f1c6a423aaf0041d89d406212b14abf90bc650358L189, so should be fine to drop from the `RawStore` interface -- 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