steveloughran commented on a change in pull request #1208: HADOOP-16423. S3Guard fsck: Check metadata consistency between S3 and metadatastore (log) URL: https://github.com/apache/hadoop/pull/1208#discussion_r321810743
########## File path: hadoop-tools/hadoop-aws/src/test/java/org/apache/hadoop/fs/s3a/s3guard/ITestS3GuardToolDynamoDB.java ########## @@ -289,4 +291,30 @@ public void testDestroyUnknownTable() throws Throwable { "-meta", "dynamodb://" + getTestTableName(DYNAMODB_TABLE)); } + @Test + public void testCLIFsckWithoutParam() throws Exception { + intercept(ExitUtil.ExitException.class, () -> run(Fsck.NAME)); + } + + @Test + public void testCLIFsckWithParam() throws Exception { + final int result = run(S3GuardTool.Fsck.NAME, "-check", + "s3a://" + getFileSystem().getBucket()); Review comment: this test failed for me during a parallel run. This parallelizable test should have a path which we know is there but is private to this test; we can have another one which invokes on a missing path. The full root scan should be run in the ITestS3GuardDDBRootOperations test, before any cleanup ---------------------------------------------------------------- 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: us...@infra.apache.org With regards, Apache Git Services --------------------------------------------------------------------- To unsubscribe, e-mail: common-issues-unsubscr...@hadoop.apache.org For additional commands, e-mail: common-issues-h...@hadoop.apache.org