[GitHub] [hadoop] sreeb-msft commented on a diff in pull request #5136: HADOOP-18498. ABFS: Removing unwanted ? prefix from SAS Tokens

2022-11-28 Thread GitBox
sreeb-msft commented on code in PR #5136: URL: https://github.com/apache/hadoop/pull/5136#discussion_r1033251323 ## hadoop-tools/hadoop-azure/src/test/java/org/apache/hadoop/fs/azurebfs/extensions/MockWithPrefixSASTokenProvider.java: ## @@ -0,0 +1,46 @@ +/** + * Licensed to the

[GitHub] [hadoop] sreeb-msft commented on a diff in pull request #5136: HADOOP-18498. ABFS: Removing unwanted ? prefix from SAS Tokens

2022-11-28 Thread GitBox
sreeb-msft commented on code in PR #5136: URL: https://github.com/apache/hadoop/pull/5136#discussion_r1033250814 ## hadoop-tools/hadoop-azure/src/test/java/org/apache/hadoop/fs/azurebfs/extensions/MockWithPrefixSASTokenProvider.java: ## @@ -0,0 +1,46 @@ +/** + * Licensed to the

[GitHub] [hadoop] sreeb-msft commented on a diff in pull request #5136: HADOOP-18498. ABFS: Removing unwanted ? prefix from SAS Tokens

2022-11-17 Thread GitBox
sreeb-msft commented on code in PR #5136: URL: https://github.com/apache/hadoop/pull/5136#discussion_r1025128772 ## hadoop-tools/hadoop-azure/src/main/java/org/apache/hadoop/fs/azurebfs/services/AbfsClient.java: ## @@ -1130,6 +1130,10 @@ private String appendSASTokenToQuery(Stri