Yeah, prefer an infinite loop on failure to an intermittent test fail due to a timing hiccup.  Ditto for all the other potentially infinite test loops.

On May 6, 2013, at 4:33 PM, Stefano Santoro <notifications@github.com> wrote:

> In tests/frontend/org/voltdb/regressionsuites/TestSystemProcedureSuite.java:
> 
> >  
> >              int foundItem = 0;
> > +            VoltTable results[] = client.callProcedure("@Statistics", "table", 0).getResults();
> > +            while (foundItem != 6) {
> any chance this could go infinite?
> 
> —
> Reply to this email directly or view it on GitHub.