mehakmeet opened a new pull request #2141: URL: https://github.com/apache/hadoop/pull/2141
Storage Keys in ABFS should be validated after the keys have been loaded. work: - Remove the previous validation of storage keys. - Validate at the correct place. Tested by: mvn -T 1C -Dparallel-tests=abfs clean verify Region: East US, West US. _Note:_ Reason for validateStorageAccountKey() at L56, have a separate try-catch due to a UT getting broken, if it is in the same try-catch above. TestAbfsConfigurationFieldsValidation#testGetAccountKeyWithNonExistingAccountName requires the try-catch at L45 to give an expected error, but adding validateStorageAccountKey(), causes a different error, thus the test gets broken. No tests are required as there are tests already present to validate the patch. ---------------------------------------------------------------- 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 --------------------------------------------------------------------- To unsubscribe, e-mail: common-issues-unsubscr...@hadoop.apache.org For additional commands, e-mail: common-issues-h...@hadoop.apache.org
