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

John Zhuge commented on HADOOP-14038:
-------------------------------------

[~vishwajeet.dusane]  Patch 004 has the following pieces:
* Rename {{dfs.adl}} properties, can support old properties
* Change default token provider type to ClientCredential, fixed a few broken 
unit tests
* Rename {{dfs.adl.<hostname>.hostname}} and {{dfs.adl.<hostname>.mountpoint}}, 
can not support exist properties due to the {{<hostname}} part
* Remove 2 unused properties
* Obsolete TestValidateConfiguration
* Extra update to index.md

BTW, are you ok to see {{LATENCY_TRACKER_KEY}} set to 
{{adl.enable.client.latency.tracker}}? Or you'd like 
{{adl.feature.enable.client.latency.tracker}} or 
{{adl.fs.enable.client.latency.tracker}}? Suggestion?

Are you ok to obsolete TestValidateConfiguration class?

> Rename ADLS credential properties
> ---------------------------------
>
>                 Key: HADOOP-14038
>                 URL: https://issues.apache.org/jira/browse/HADOOP-14038
>             Project: Hadoop Common
>          Issue Type: Sub-task
>          Components: fs/adl
>    Affects Versions: 3.0.0-alpha3
>            Reporter: John Zhuge
>            Assignee: John Zhuge
>            Priority: Minor
>         Attachments: HADOOP-14038.001.patch, HADOOP-14038.002.patch, 
> HADOOP-14038.003.patch, HADOOP-14038.004.patch
>
>
> Add ADLS credential configuration properties to {{core-default.xml}}. 
> Set/document the default value for 
> {{dfs.adls.oauth2.access.token.provider.type}} to {{ClientCredential}}.
> Fix {{AdlFileSystem#getAccessTokenProvider}} which implies the provider type 
> is {{Custom}}.
> Fix several unit tests that set {{dfs.adls.oauth2.access.token.provider}} but 
> does not set {{dfs.adls.oauth2.access.token.provider.type}}.



--
This message was sent by Atlassian JIRA
(v6.3.15#6346)

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

Reply via email to