[ 
https://issues.apache.org/jira/browse/HADOOP-19471?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=17933808#comment-17933808
 ] 

ASF GitHub Bot commented on HADOOP-19471:
-----------------------------------------

anmolanmol1234 commented on code in PR #7461:
URL: https://github.com/apache/hadoop/pull/7461#discussion_r1987018509


##########
hadoop-tools/hadoop-azure/src/test/java/org/apache/hadoop/fs/azurebfs/ITestAzureBlobFileSystemChooseSAS.java:
##########
@@ -150,7 +150,9 @@ public void testBothProviderFixedTokenConfigured() throws 
Exception {
    * Helper method to get the Fixed SAS token value
    */
   private String getFixedSASToken(AbfsConfiguration config) throws Exception {
-    return config.getSASTokenProvider().getSASToken(this.getAccountName(), 
this.getFileSystemName(), getMethodName(), "read");
+    String readPermission = "read";

Review Comment:
   this should be a static string used as a constant.





> ABFS:  Support Fixed SAS token at container level
> -------------------------------------------------
>
>                 Key: HADOOP-19471
>                 URL: https://issues.apache.org/jira/browse/HADOOP-19471
>             Project: Hadoop Common
>          Issue Type: Task
>          Components: fs/azure
>    Affects Versions: 3.4.0, 3.4.1
>            Reporter: Manika Joshi
>            Assignee: Manika Joshi
>            Priority: Major
>              Labels: pull-request-available
>
> The ABFS driver currently lacks support for multiple SAS tokens for the same 
> storage account across different containers.
> We are now introducing this support.
> To use fixed SAS token at container level the configuration to be used is:
> {quote}fs.azure.sas.fixed.token.<container-name>.<storage-account-name>
> {quote}



--
This message was sent by Atlassian Jira
(v8.20.10#820010)

---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]

Reply via email to