Home | About | Sematext search-lucene.com search-hadoop.com
 Search Hadoop and all its subprojects:

Switch to Plain View
Accumulo >> mail # dev >> ExportTable.isReady - alternative to for loop?


+
David Medinets 2012-09-06, 02:20
Copy link to this message
-
Re: ExportTable.isReady - alternative to for loop?
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;