[
https://issues.apache.org/jira/browse/RYA-112?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=15376028#comment-15376028
]
David W. Lotts edited comment on RYA-112 at 7/18/16 7:24 PM:
-------------------------------------------------------------
Started the processs of Deprecating class ConfigUtils.
I recomend that when working on this, set @Deprecated on this class so that
it's usage will be highlighted.
Eliminated table key strings and get methods that use them.
Each caller changed to call it's own object's method, overriding
RyaSecondaryIndexer.getTableName().
So now the indexes manage there own table names and are not set-able.
was (Author: dlotts):
Deprecated class ConfigUtils so that it's usage will be highlighted.
Eliminated table key strings and get methods that use them.
Each caller changed to call it's own object's method, overriding
RyaSecondaryIndexer.getTableName().
So now the indexes manage there own table names and are not set-able.
> Integrate RyaDetails with secondary indexers
> --------------------------------------------
>
> Key: RYA-112
> URL: https://issues.apache.org/jira/browse/RYA-112
> Project: Rya
> Issue Type: Sub-task
> Components: clients
> Affects Versions: 3.2.10
> Reporter: Andrew Smith
> Assignee: David W. Lotts
> Fix For: 3.2.10
>
>
> The secondary indexers may or may not use ConfigUtils to get their table
> names from the client Configuration. We want to make sure that the user can
> not specifying their own secondary indexer table names and instead rely on
> default table names assigned by ConfigUtils.
> Remove the table name keys from ConfigUtils and update the getter methods to
> always return the default values.
> Class that needs updates:
> https://github.com/kchilton2/incubator-rya/blob/RYA-81/extras/indexing/src/main/java/mvm/rya/indexing/accumulo/ConfigUtils.java
> PCJ Indexer is a little different. It needs to look up the updater strategy
> from Rya Details.
--
This message was sent by Atlassian JIRA
(v6.3.4#6332)
