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

Da Zhou commented on HADOOP-15954:
----------------------------------

*IdentityTransformerExceptio*n: line 33
input parameter "cause" is declared but not used.

Flag "*skipTransFormation*" is defined and used in 
"StandardSPIdentityTransformer", I think it is better to add it to 
"AzureBlobFileSystemStore" to avoid the unnecessary initialization for 
"IdentityTransformer".

There are many checkstyle violations, so I'm submitting the patch to get the 
Yetus report then you can fix them in another patch.

Tests: Tested locally, all existing tests passed. But we need new tests for 
patch like this.

> ABFS: Enable owner and group conversion for MSI and login user using OAuth
> --------------------------------------------------------------------------
>
>                 Key: HADOOP-15954
>                 URL: https://issues.apache.org/jira/browse/HADOOP-15954
>             Project: Hadoop Common
>          Issue Type: Sub-task
>          Components: fs/azure
>    Affects Versions: 3.2.0
>            Reporter: junhua gu
>            Assignee: junhua gu
>            Priority: Major
>         Attachments: HADOOP-15954-001.patch, HADOOP-15954-002.patch, 
> HADOOP-15954-003.patch
>
>
> Add support for overwriting owner and group in set/get operations to be the 
> service principal id when OAuth is used. Add support for upn short name 
> format.
>  
> Add Standard Transformer for SharedKey / Service 
> Add interface provides an extensible model for customizing the acquisition of 
> Identity Transformer.



--
This message was sent by Atlassian JIRA
(v7.6.3#76005)

---------------------------------------------------------------------
To unsubscribe, e-mail: common-issues-unsubscr...@hadoop.apache.org
For additional commands, e-mail: common-issues-h...@hadoop.apache.org

Reply via email to