[ 
https://issues.apache.org/jira/browse/RANGER-5439?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel
 ]

Siyao Meng updated RANGER-5439:
-------------------------------
    Description: 
`ozone.om.http-address` was used in the original implementation in RANGER-2325:

{code:java}
        ozoneClient = 
OzoneClientFactory.getRpcClient(conf.get("ozone.om.http-address"));
{code}

https://github.com/apache/ranger/commit/9b8eaa3d3179e13fceb896f677debb248b9529f0#diff-71910ea75756885fa17c4d7b2b7a412cb5ae57dd9eca2bb9aedac379f8aaf954R56

but is not used anymore nowadays:

{code:java}
            String[] serviceIds = 
conf.getTrimmedStrings("ozone.om.service.ids", "ozone1");
{code}

https://github.com/apache/ranger/blob/8046faee1d51ac20caf8c00f8db3100b32690402/plugin-ozone/src/main/java/org/apache/ranger/services/ozone/client/OzoneClient.java#L67


We should deprecate this config (or remove this if it is safe to do so) in 
order to minimize confusion:

https://github.com/apache/ranger/blob/8046faee1d51ac20caf8c00f8db3100b32690402/agents-common/src/main/resources/service-defs/ranger-servicedef-ozone.json#L97

  was:
`ozone.om.http-address` was used in the original implementation in RANGER-2325:

{code:java}
        ozoneClient = 
OzoneClientFactory.getRpcClient(conf.get("ozone.om.http-address"));
{code}

https://github.com/apache/ranger/commit/9b8eaa3d3179e13fceb896f677debb248b9529f0#diff-71910ea75756885fa17c4d7b2b7a412cb5ae57dd9eca2bb9aedac379f8aaf954R56

but is not used anymore nowadays:

{code:java}
            String[] serviceIds = 
conf.getTrimmedStrings("ozone.om.service.ids", "ozone1");
{code}

https://github.com/apache/ranger/blob/8046faee1d51ac20caf8c00f8db3100b32690402/plugin-ozone/src/main/java/org/apache/ranger/services/ozone/client/OzoneClient.java#L67


We should deprecate this config (or remove this if it is safe to do so) in 
order to minimize confusion.


> Deprecate Ozone URL (ozone.om.http-address) config in Ozone servicedef
> ----------------------------------------------------------------------
>
>                 Key: RANGER-5439
>                 URL: https://issues.apache.org/jira/browse/RANGER-5439
>             Project: Ranger
>          Issue Type: Task
>          Components: plugins
>            Reporter: Siyao Meng
>            Priority: Major
>
> `ozone.om.http-address` was used in the original implementation in 
> RANGER-2325:
> {code:java}
>         ozoneClient = 
> OzoneClientFactory.getRpcClient(conf.get("ozone.om.http-address"));
> {code}
> https://github.com/apache/ranger/commit/9b8eaa3d3179e13fceb896f677debb248b9529f0#diff-71910ea75756885fa17c4d7b2b7a412cb5ae57dd9eca2bb9aedac379f8aaf954R56
> but is not used anymore nowadays:
> {code:java}
>             String[] serviceIds = 
> conf.getTrimmedStrings("ozone.om.service.ids", "ozone1");
> {code}
> https://github.com/apache/ranger/blob/8046faee1d51ac20caf8c00f8db3100b32690402/plugin-ozone/src/main/java/org/apache/ranger/services/ozone/client/OzoneClient.java#L67
> We should deprecate this config (or remove this if it is safe to do so) in 
> order to minimize confusion:
> https://github.com/apache/ranger/blob/8046faee1d51ac20caf8c00f8db3100b32690402/agents-common/src/main/resources/service-defs/ranger-servicedef-ozone.json#L97



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

Reply via email to