diff --git a/hudi-client/hudi-client-common/src/main/java/org/apache/hudi/client/transaction/lock/InProcessLockProvider.java b/hudi-client/hudi-client-common/src/main/java/org/apache/hudi/client/transaction/lock/InProcessLockProvider.java index 426d1cfaf..9e5a2379c 100644 --- a/hudi-client/hudi-client-common/src/main/java/org/apache/hudi/client/transaction/lock/InProcessLockProvider.java +++ b/hudi-client/hudi-client-common/src/main/java/org/apache/hudi/client/transaction/lock/InProcessLockProvider.java @@ -19,13 +19,14 @@ package org.apache.hudi.client.transaction.lock; -import org.apache.hadoop.conf.Configuration; import org.apache.hudi.common.config.LockConfiguration; import org.apache.hudi.common.config.TypedProperties; import org.apache.hudi.common.lock.LockProvider; import org.apache.hudi.common.lock.LockState; import org.apache.hudi.common.util.StringUtils; import org.apache.hudi.exception.HoodieLockException; + +import org.apache.hadoop.conf.Configuration; import org.apache.log4j.LogManager; import org.apache.log4j.Logger; import org.jetbrains.annotations.NotNull; @@ -92,7 +93,11 @@ public class InProcessLockProvider implements LockProvider { inProcessLockProvider.unlock(); }); - assertThrows(HoodieLockException.class, () -> { + assertDoesNotThrow(() -> { inProcessLockProvider.unlock(); }); } @@ -246,7 +246,7 @@ public class TestInProcessLockProvider { @Test public void testUnlockWithoutLock() { InProcessLockProvider inProcessLockProvider = new InProcessLockProvider(lockConfiguration, hadoopConfiguration); - assertThrows(HoodieLockException.class, () -> { + assertDoesNotThrow(() -> { inProcessLockProvider.unlock(); }); } diff --git a/hudi-client/hudi-client-common/src/test/java/org/apache/hudi/client/transaction/TestTransactionManager.java b/hudi-client/hudi-client-common/src/test/java/org/apache/hudi/client/transaction/TestTransactionManager.java index a1a7f6a31..22f801784 100644 --- a/hudi-client/hudi-client-common/src/test/java/org/apache/hudi/client/transaction/TestTransactionManager.java +++ b/hudi-client/hudi-client-common/src/test/java/org/apache/hudi/client/transaction/TestTransactionManager.java @@ -81,7 +81,7 @@ public class TestTransactionManager extends HoodieCommonTestHarness { }); transactionManager.endTransaction(); - assertThrows(HoodieLockException.class, () -> { + assertDoesNotThrow(() -> { transactionManager.endTransaction(); }); }