bilaharith commented on a change in pull request #2707:
URL: https://github.com/apache/hadoop/pull/2707#discussion_r609564775
##########
File path:
hadoop-tools/hadoop-azure/src/test/java/org/apache/hadoop/fs/azurebfs/services/TestAbfsClient.java
##########
@@ -103,7 +102,7 @@ public TestAbfsClient(){
}
private String getUserAgentString(AbfsConfiguration config,
- boolean includeSSLProvider) throws MalformedURLException {
+ boolean includeSSLProvider) throws IOException {
Review comment:
AbfsClient constructor could throw IOException, wiuth the new encoding
and hashing logic included
##########
File path:
hadoop-tools/hadoop-azure/src/test/java/org/apache/hadoop/fs/azurebfs/services/TestAbfsClient.java
##########
@@ -250,8 +249,7 @@ public void verifyUserAgentClusterType() throws Exception {
public static AbfsClient createTestClientFromCurrentContext(
AbfsClient baseAbfsClientInstance,
- AbfsConfiguration abfsConfig)
- throws AzureBlobFileSystemException {
+ AbfsConfiguration abfsConfig) throws IOException {
Review comment:
AbfsClient constructor could throw IOException, wiuth the new encoding
and hashing logic included
--
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:
[email protected]
---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]
