[ https://issues.apache.org/jira/browse/ACCUMULO-2061?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=13852414#comment-13852414
]
Keith Turner commented on ACCUMULO-2061:
----------------------------------------
For 1.6.0, I think we can only deprecate {{instance.dfs.dir}} if {{instance.volumes}} is changed
to not be defined using {{instance.dfs.dir}} as suggested in the ticket description. We may
be able to deprecate {{instance.dfs.uri}} now even if the current behavior in 1.6.0-SNAPSHOT
is not changed.
One other thing to consider is that these properties will be needed as long as the user has
relative paths. If a user upgrades to 1.7.0 and still have relative paths, then the props
may still be needed.
> Deprecate instance.dfs.uri and instance.dfs.dir
> -----------------------------------------------
>
> Key: ACCUMULO-2061
> URL: https://issues.apache.org/jira/browse/ACCUMULO-2061
> Project: Accumulo
> Issue Type: Sub-task
> Components: master, tserver
> Reporter: Christopher Tubbs
> Fix For: 1.6.0
>
>
> {{instance.dfs.uri}} and {{instance.dfs.dir}} are no longer needed with the {{instance.volumes}}
property.
> Together, these two fields are needed for upgrades from relative paths, but full URIs
for volumes should be specified in the {{instance.volumes}} set.
> Instead of appending {{instance.dfs.dir}} to every volume, which is a bit confusing,
they should be specified explicitly in the {{instance.volumes}}.
> Example:
> {code}
> <property>
> <name>instance.volumes</name>
> <value>hdfs://nn1/accumulo</value>
> </property>
> {code}
> should be equivalent to
> {code}
> <property>
> <name>instance.dfs.uri</name>
> <value>hdfs://nn1</value>
> </property>
> <property>
> <name>instance.dfs.dir</name>
> <value>/accumulo</value>
> </property>
> {code}
> This change simplifies the semantics of configuring volumes for Accumulo to use for storage,
and is a bit more obvious that we're logically configuring filesystem volumes, not "namenode
URIs".
--
This message was sent by Atlassian JIRA
(v6.1.4#6159)
|