snvijaya commented on issue #1842: HADOOP-16730 : ABFS: Add Authorizer Interface
URL: https://github.com/apache/hadoop/pull/1842#issuecomment-591386525
@ThomasMarquardt - I have 2 open conversations for which I have added my
queries. Kindly request you to provide your comments.
snvijaya commented on issue #1842: HADOOP-16730 : ABFS: Add Authorizer Interface
URL: https://github.com/apache/hadoop/pull/1842#issuecomment-590257367
@steveloughran - Have updated the PR with fixes for comments added. Kindly
request you to take a look.
snvijaya commented on issue #1842: HADOOP-16730 : ABFS: Add Authorizer Interface
URL: https://github.com/apache/hadoop/pull/1842#issuecomment-589607680
Have fixed the open comments. @steveloughran - Can you please check. Thanks.
snvijaya commented on issue #1842: HADOOP-16730 : ABFS: Add Authorizer Interface
URL: https://github.com/apache/hadoop/pull/1842#issuecomment-588568589
@steveloughran @ThomasMarquardt - Have updated the review based on the
recent suggestions form Thomas. Test results are as pasted above.
snvijaya commented on issue #1842: HADOOP-16730 : ABFS: Add Authorizer Interface
URL: https://github.com/apache/hadoop/pull/1842#issuecomment-588568283
With namespace enabled account (East US 2)
`[INFO] Tests run: 52, Failures: 0, Errors: 0, Skipped: 0`
`[WARNING] Tests run: 407,
snvijaya commented on issue #1842: HADOOP-16730 : ABFS: Add Authorizer Interface
URL: https://github.com/apache/hadoop/pull/1842#issuecomment-586471534
@steveloughran - Thanks for taking time to review this change. I have
addressed almost all the comments and the ones I havent response is
snvijaya commented on issue #1842: HADOOP-16730 : ABFS: Add Authorizer Interface
URL: https://github.com/apache/hadoop/pull/1842#issuecomment-585734846
Reran the tests after the last fix for findbugs :
Test results for account without namespace enabled (East US 2):
`[INFO] Tests
snvijaya commented on issue #1842: HADOOP-16730 : ABFS: Add Authorizer Interface
URL: https://github.com/apache/hadoop/pull/1842#issuecomment-585350897
Test results for account without namespace enabled (East US 2):
[INFO] Tests run: 52, Failures: 0, Errors: 0, Skipped: 0
[ERROR]