iwasakims commented on a change in pull request #1950:
URL: https://github.com/apache/hadoop/pull/1950#discussion_r416354605



##########
File path: 
hadoop-tools/hadoop-aws/src/test/java/org/apache/hadoop/fs/s3a/s3guard/ITestS3GuardToolLocal.java
##########
@@ -165,18 +165,19 @@ public void testImportNoFilesystem() throws Throwable {
 
   @Test
   public void testInfoBucketAndRegionNoFS() throws Throwable {
-    intercept(FileNotFoundException.class,
+    intercept(UnknownStoreException.class,

Review comment:
       v2 prove was used as a default value when I ran the test.
   ```
     public static final int S3A_BUCKET_PROBE_DEFAULT = 2;
   ```
   
   The reaseon you did not see this may be that the ITestS3GuardToolLocal was 
just skipped. You are using DynamoDBMetadataStore rather than 
LocalMetadataStore?
   




----------------------------------------------------------------
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.

For queries about this service, please contact Infrastructure at:
[email protected]



---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]

Reply via email to