fapifta commented on pull request #3579: URL: https://github.com/apache/hadoop/pull/3579#issuecomment-961311596
@sodonnel I still owe this PR with some kind of test, the prior test failure should be fixed now, but it has revealed that there were some behaviour change caused by my prior changes, in the case the fallbackToSimpleAuth boolean is null. I also realised - while I was evaluating the problem with the TestSASLRPC - that the addition of setFallbackToSimpleAuth at the beginning of setupIOStreams changed behaviour if the authProtocol is not SASL. I had a plan to add a MiniCluster based test for this, but TestSASLRPC code gave me some other ideas, so I am exploring the possibilities there as well. -- 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. To unsubscribe, e-mail: [email protected] For queries about this service, please contact Infrastructure at: [email protected] --------------------------------------------------------------------- To unsubscribe, e-mail: [email protected] For additional commands, e-mail: [email protected]
