[GitHub] wohali commented on issue #554: JS stats test has inconsistent results

2017-07-07 Thread git
wohali commented on issue #554: JS stats test has inconsistent results
URL: https://github.com/apache/couchdb/issues/554#issuecomment-313824985
 
 
   I have fully disabled this test for now. I'm starting to think that the 
dynamic behaviour of the clustered setup means these stats don't make as much 
sense anymore -- and that the test needs a complete rework.
   
   We can revisit after 2.1.0.
 

This is an automated message from the Apache Git Service.
To respond to the message, please log on GitHub and use the
URL above to go to the specific comment.
 
For queries about this service, please contact Infrastructure at:
us...@infra.apache.org


With regards,
Apache Git Services


[GitHub] wohali commented on issue #554: JS stats test has inconsistent results

2017-07-03 Thread git
wohali commented on issue #554: JS stats test has inconsistent results
URL: https://github.com/apache/couchdb/issues/554#issuecomment-312728326
 
 
   We're still seeing a bunch of these:
   
   https://travis-ci.org/apache/couchdb/jobs/243780115 - stats.js mismatch line 
58
   https://travis-ci.org/apache/couchdb/jobs/243303510 - stats.js mismatch line 
58
   
   I think we have a real set of bugs here, but it may be time to re-disable 
the entire stats.js test for now. The fact that the test run is failing is a 
blocker; the fact that our stats are sometimes inconsistent, to me, isn't - 
though it may be indicative of race conditions as of yet uncovered.
   
   Thoughts?
 

This is an automated message from the Apache Git Service.
To respond to the message, please log on GitHub and use the
URL above to go to the specific comment.
 
For queries about this service, please contact Infrastructure at:
us...@infra.apache.org


With regards,
Apache Git Services


[GitHub] wohali commented on issue #554: JS stats test has inconsistent results

2017-06-02 Thread git
wohali commented on issue #554: JS stats test has inconsistent results
URL: https://github.com/apache/couchdb/issues/554#issuecomment-305900683
 
 
   restartServer() works with 2.0 via a call out to `dev/run`, see my comments 
here: https://github.com/apache/couchdb/pull/556#issuecomment-305900649
 

This is an automated message from the Apache Git Service.
To respond to the message, please log on GitHub and use the
URL above to go to the specific comment.
 
For queries about this service, please contact Infrastructure at:
us...@infra.apache.org


With regards,
Apache Git Services