You are viewing a plain text version of this content. The canonical link for it is here.
Posted to common-commits@hadoop.apache.org by st...@apache.org on 2021/04/24 16:24:34 UTC
[hadoop] branch branch-3.3 updated: HADOOP-17535. ABFS:
ITestAzureBlobFileSystemCheckAccess test failure if no oauth key. (#2920)
This is an automated email from the ASF dual-hosted git repository.
stevel pushed a commit to branch branch-3.3
in repository https://gitbox.apache.org/repos/asf/hadoop.git
The following commit(s) were added to refs/heads/branch-3.3 by this push:
new 77fddcf HADOOP-17535. ABFS: ITestAzureBlobFileSystemCheckAccess test failure if no oauth key. (#2920)
77fddcf is described below
commit 77fddcfcb1cae69d67a3693a25b4023f530f5d78
Author: Steve Loughran <st...@cloudera.com>
AuthorDate: Wed Apr 21 16:06:06 2021 +0100
HADOOP-17535. ABFS: ITestAzureBlobFileSystemCheckAccess test failure if no oauth key. (#2920)
Contributed by Steve Loughran.
Change-Id: I165f5ed3a8486404403827b5c0338cf7f80c2bb1
---
.../hadoop/fs/azurebfs/ITestAzureBlobFileSystemCheckAccess.java | 5 ++---
1 file changed, 2 insertions(+), 3 deletions(-)
diff --git a/hadoop-tools/hadoop-azure/src/test/java/org/apache/hadoop/fs/azurebfs/ITestAzureBlobFileSystemCheckAccess.java b/hadoop-tools/hadoop-azure/src/test/java/org/apache/hadoop/fs/azurebfs/ITestAzureBlobFileSystemCheckAccess.java
index 2c7b776..e52071d 100644
--- a/hadoop-tools/hadoop-azure/src/test/java/org/apache/hadoop/fs/azurebfs/ITestAzureBlobFileSystemCheckAccess.java
+++ b/hadoop-tools/hadoop-azure/src/test/java/org/apache/hadoop/fs/azurebfs/ITestAzureBlobFileSystemCheckAccess.java
@@ -22,7 +22,6 @@ import java.io.IOException;
import java.lang.reflect.Field;
import java.util.List;
-import org.apache.hadoop.thirdparty.com.google.common.base.Preconditions;
import org.apache.hadoop.thirdparty.com.google.common.collect.Lists;
import org.junit.Assume;
import org.junit.Test;
@@ -316,8 +315,8 @@ public class ITestAzureBlobFileSystemCheckAccess
private void checkIfConfigIsSet(String configKey){
AbfsConfiguration conf = getConfiguration();
String value = conf.get(configKey);
- Preconditions.checkArgument((value != null && value.trim().length() > 1),
- configKey + " config is mandatory for the test to run");
+ Assume.assumeTrue(configKey + " config is mandatory for the test to run",
+ value != null && value.trim().length() > 1);
}
private void assertAccessible(Path testFilePath, FsAction fsAction)
---------------------------------------------------------------------
To unsubscribe, e-mail: common-commits-unsubscribe@hadoop.apache.org
For additional commands, e-mail: common-commits-help@hadoop.apache.org