Re: git commit: COUCHDB-1424 Fix etap to not consume any message

2012-10-31 Thread Paul Davis
Nice find On Wed, Oct 31, 2012 at 11:29 AM, fdman...@apache.org wrote: Updated Branches: refs/heads/1424-fix-etap-consuming-any-test-message [created] 67d531b02 COUCHDB-1424 Fix etap to not consume any message Turns out that etap consumes any message in the mailbox in some cases. This

Re: git commit: COUCHDB-1424 Fix etap to not consume any message

2012-10-31 Thread Filipe David Manana
On Wed, Oct 31, 2012 at 4:32 PM, Paul Davis paul.joseph.da...@gmail.com wrote: Nice find Say thanks to Damien :) Fixed in an older etap version, I just ported it to latest etap. Robert (Dionne), wanna test this to see if it fixes the hangs you used to have with OTP R15Bx? thanks On Wed, Oct

Re: git commit: COUCHDB-1424 Fix etap to not consume any message

2012-10-31 Thread Bob Dionne
Hi Filipe, I tested this new branch and it seems to have issues (at least on my machine) : ok 44 reduce_false ok 45 reduce_false_temp not ok 46 replication Reason: expected 'tony', got 'null' Trace back (most recent call first): 0: Error(expected '\tony\', got 'null') 46:

Re: git commit: COUCHDB-1424 Fix etap to not consume any message

2012-10-31 Thread Filipe David Manana
On Wed, Oct 31, 2012 at 8:01 PM, Bob Dionne dio...@dionne-associates.com wrote: Hi Filipe, I tested this new branch and it seems to have issues (at least on my machine) : ok 44 reduce_false ok 45 reduce_false_temp not ok 46 replication Reason: expected 'tony', got 'null' Trace back

Re: git commit: COUCHDB-1424 Fix etap to not consume any message

2012-10-31 Thread Bob Dionne
oops, sorry, wrong paste, it is irrelevant. I meant to paste: ./test/etap/120-stats-collect.t ... ok ./test/etap/121-stats-aggregates.t ok ./test/etap/130-attachments-md5.t . ok ./test/etap/140-attachment-comp.t . Failed 9/85 subtests

Re: git commit: COUCHDB-1424 Fix etap to not consume any message

2012-10-31 Thread Bob Dionne
also +1 on this patch just based on inspection, it looks like the right thing to do On Oct 31, 2012, at 3:16 PM, Bob Dionne dio...@dionne-associates.com wrote: oops, sorry, wrong paste, it is irrelevant. I meant to paste: ./test/etap/120-stats-collect.t ... ok