Anuj Modi

Results 18 comments of Anuj Modi

------------------------ :::: AGGREGATED TEST RESULT :::: HNS-OAuth ======================== [INFO] Results: [INFO] [WARNING] Tests run: 137, Failures: 0, Errors: 0, Skipped: 3 [INFO] Results: [INFO] [WARNING] Tests run: 340, Failures: 0,...

> I think it will depend on the fact first commit works as it its and is not dependent on second one. We need to merge these separately such that...

------------------------ :::: AGGREGATED TEST RESULT :::: HNS-OAuth ======================== [INFO] Results: [INFO] [WARNING] Tests run: 137, Failures: 0, Errors: 0, Skipped: 2 [INFO] Results: [INFO] [ERROR] Errors: [ERROR] ITestAbfsTerasort.test_120_terasort:262->executeStage:206 » IO...

------------------------------ :::: AGGREGATED TEST RESULT :::: ============================================================ HNS-OAuth ============================================================ [WARNING] Tests run: 137, Failures: 0, Errors: 0, Skipped: 2 [WARNING] Tests run: 620, Failures: 0, Errors: 0, Skipped: 76 [WARNING]...

Thanks @steveloughran for the review. This is good to be merged now.

> +1 pending that change to abfs.md that @anmolanmol1234 spotted Which change are you referring to?? I can see a comment regarding the typo which anmol spotted and already fixed...

> I'm afraid you are very unlucky in that I have been staring at the ABFS auth code for diagnostics purposes today and so these lines of code are now...

Hi @steveloughran @mukund-thakur Requesting you to kindly review the latest changes and merge if they looks good. Thanks

> @anujmodi2021 merged to trunk. Can you do the branch-3.4 cherrypick, rerun the tests and then push up as a PR to merge there too? thanks Thanks a lot Steve....