Kuhu Shukla updated HDFS-5040:
    Attachment: HDFS-5040.005.patch

bq. Add fail assertion for exception cases. you can use ContractTestUtils.fail
If I understood that correctly, I have added try-catch for allowed=true cases 
to fail if any exception is encountered. I have used the Assert.fail() call as 
the previous patch.
Fixed the test failures in TestAuditLogger, TestDFSAdminWithHA and 
Other failures are unrelated and flaky.
Will wait on precommit before asking for further review. Thanks [~brahmareddy] 
for the reviews so far.

> Audit log for admin commands/ logging output of all DFS admin commands
> ----------------------------------------------------------------------
>                 Key: HDFS-5040
>                 URL: https://issues.apache.org/jira/browse/HDFS-5040
>             Project: Hadoop HDFS
>          Issue Type: New Feature
>          Components: namenode
>    Affects Versions: 3.0.0-alpha1
>            Reporter: Raghu C Doppalapudi
>            Assignee: Kuhu Shukla
>              Labels: BB2015-05-TBR
>         Attachments: HDFS-5040.001.patch, HDFS-5040.004.patch, 
> HDFS-5040.005.patch, HDFS-5040.patch, HDFS-5040.patch, HDFS-5040.patch
> enable audit log for all the admin commands/also provide ability to log all 
> the admin commands in separate log file, at this point all the logging is 
> displayed on the console.

This message was sent by Atlassian JIRA

To unsubscribe, e-mail: hdfs-issues-unsubscr...@hadoop.apache.org
For additional commands, e-mail: hdfs-issues-h...@hadoop.apache.org

Reply via email to