This is the code I am looking at:

    // TODO look for walogs
    for (Entry<Key,Value> entry : metaScanner) {
      return 500;
    }

    return 0;

FindBugs complains that entry is unused. Does it make sense to replace
that code with:

  return metaScanner.iterator().hasNext() ? 500 : 0;

I've already patched that. It just hasn't been committed.
See: https://issues.apache.org/jira/browse/ACCUMULO-456
Patch: https://issues.apache.org/jira/secure/attachment/12543936/ACCUMULO-456-4.patch

On Wed, Sep 5, 2012 at 10:20 PM, David Medinets
<[EMAIL PROTECTED]> wrote:
> This is the code I am looking at:
>
>     // TODO look for walogs
>     for (Entry<Key,Value> entry : metaScanner) {
>       return 500;
>     }
>
>     return 0;
>
> FindBugs complains that entry is unused. Does it make sense to replace
> that code with:
>
>   return metaScanner.iterator().hasNext() ? 500 : 0;