accumulo-notifications mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From "Bill Havanki (JIRA)" <j...@apache.org>
Subject [jira] [Commented] (ACCUMULO-2992) Eliminate (use of) AccumuloConfiguration.getSiteConfiguration()
Date Tue, 15 Jul 2014 13:41:04 GMT

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

Bill Havanki commented on ACCUMULO-2992:
----------------------------------------

How about an annotation, something like {{@ServerOnly}}? It may even be possible to have the
compiler complain when it sees it used incorrectly:

http://stackoverflow.com/questions/1752607/how-to-intentionally-cause-a-custom-java-compiler-warning-message

Barring that, I'm fine with a comment. I'll suggest the class Javadoc for {{SiteConfiguration}},
but I'm open to whatever.

> Eliminate (use of) AccumuloConfiguration.getSiteConfiguration()
> ---------------------------------------------------------------
>
>                 Key: ACCUMULO-2992
>                 URL: https://issues.apache.org/jira/browse/ACCUMULO-2992
>             Project: Accumulo
>          Issue Type: Improvement
>            Reporter: Bill Havanki
>            Assignee: Bill Havanki
>            Priority: Trivial
>              Labels: configuration
>             Fix For: 1.6.1, 1.7.0
>
>
> The static method {{AccumuloConfiguration.getSiteConfiguration()}} has been deprecated
for a long time. [~ecn] got rid of a lot of uses in ACCUMULO-2401. This ticket is to get rid
of the rest. Each call can be replaced with:
> {code}
> SiteConfiguration.getInstance(DefaultConfiguration.getInstance());
> {code}
> The method itself can also be eliminated, as {{AccumuloConfiguration}} is not public
API, assuming the elimination would not cause a problem.



--
This message was sent by Atlassian JIRA
(v6.2#6252)

Mime
View raw message