sarveshtamba commented on issue #2127: Fix for CompactTest failure
URL: https://github.com/apache/couchdb/pull/2127#issuecomment-525684289
 
 
   > @sansato I'd say `info["sizes.active"]` is the best option here, it's a 
size of "live" data in bytes, so it should go down when doc deleted and db then 
compacted.
   > 
   > But I agree with @iilyak, the fact that in your trace finale data and disk 
sizes bigger than the originals indicates that compaction not finished at the 
time of final test assertion, i.e. it's not the question of _what_ to compare, 
but more of a question _when_ to compare.
   
   @iilyak @eiri @sansato I have run this compaction test with increased 
timeouts of 99999999 ms (~27 hours), yet the compaction doesn't complete and 
the test fails/times out.
   
   > I appreciate it's not a simple task, sizes calculation, especially in 
combination with compaction is convoluted, so maybe someone more proficient 
than me in elixir suite can disable this test for time being and we'll get back 
to it in version 3.x
   
   @eiri I agree with you on this, I myself am quite new to 
couchdb/elixir/erlang and it was a challenging task, though I could figure 
above details with the limited amount of knowledge I could gather.
   Could we somehow disable this test for now and revisit it later, so that we 
can build the entire couchdb & the test suites off top of the tree/latest 
master branch without having to make any changes at the users end?

----------------------------------------------------------------
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
 
For queries about this service, please contact Infrastructure at:
[email protected]


With regards,
Apache Git Services

Reply via email to