[
https://issues.apache.org/jira/browse/HADOOP-15954?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=16739924#comment-16739924
]
Da Zhou commented on HADOOP-15954:
----------------------------------
Submitting patch 010.
*AbfsConfiguration:*
Fixed, "org.apache.hadoop.fs.azurebfs.oauth2.DefaultSPIdentityTransformer"
will be loaded and initialize if config is not set.
*DefaultSPIdentityTransformer:*
Removed the unnecessary changes for display name since this is not supported.
{quote}line 116 & 155: Should it be if
(shouldReplaceOwnerAndGroup(replaceOwner) || (SUPER_USER.equals(replaceOwner)
&& enableSuperUserReplacement))
{quote}
It is OK since A || (B && C) is the same as A || B&&C, for more complex
condition I would add the parentheses for readability.
*AzureBlobFileSystem:*
I think you mean "*AzureBlobFileSystemStore*", I've removed the unnecessary
check for the null for "*identityTransformer*".
> 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: Da Zhou
> Priority: Major
> Attachments: HADOOP-15954-001.patch, HADOOP-15954-002.patch,
> HADOOP-15954-003.patch, HADOOP-15954-004.patch, HADOOP-15954-005.patch,
> HADOOP-15954-006.patch, HADOOP-15954-007.patch, HADOOP-15954-008.patch,
> HADOOP-15954-009.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: [email protected]
For additional commands, e-mail: [email protected]
