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

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

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

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