accumulo-notifications mailing list archives

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


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:

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:
>             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

View raw message