Apache
/
hadoop
1
0
Fork 0
Code Issues Pull Requests Packages Projects Releases Wiki Activity

HADOOP-17535. ABFS: ITestAzureBlobFileSystemCheckAccess test failure if no oauth key. (#2920) 

Contributed by Steve Loughran.

Change-Id: I165f5ed3a8486404403827b5c0338cf7f80c2bb1
This commit is contained in:
Steve Loughran 2021-04-21 16:06:06 +01:00
parent 003deeeecf
commit 77fddcfcb1
No known key found for this signature in database
GPG Key ID: D22CF846DBB162A0
1 changed files with 2 additions and 3 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

5
hadoop-tools/hadoop-azure/src/test/java/org/apache/hadoop/fs/azurebfs/ITestAzureBlobFileSystemCheckAccess.java
View File

@ -22,7 +22,6 @@ import java.io.IOException;
import java.lang.reflect.Field; import java.lang.reflect.Field;
import java.util.List; 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.apache.hadoop.thirdparty.com.google.common.collect.Lists;
import org.junit.Assume; import org.junit.Assume;
import org.junit.Test; import org.junit.Test;
@ -316,8 +315,8 @@ public class ITestAzureBlobFileSystemCheckAccess
private void checkIfConfigIsSet(String configKey){ private void checkIfConfigIsSet(String configKey){
AbfsConfiguration conf = getConfiguration(); AbfsConfiguration conf = getConfiguration();
String value = conf.get(configKey); String value = conf.get(configKey);
Preconditions.checkArgument((value != null && value.trim().length() > 1), Assume.assumeTrue(configKey + " config is mandatory for the test to run",
configKey + " config is mandatory for the test to run"); value != null && value.trim().length() > 1);
} }
private void assertAccessible(Path testFilePath, FsAction fsAction) private void assertAccessible(Path testFilePath, FsAction fsAction)