[GitHub] [couchdb] sarveshtamba edited a comment on issue #2099: CompactTest fails while executing CouchDB elixir test suite.

[GitBox <gi...@apache.org>]

sarveshtamba edited a comment on issue #2099: CompactTest fails while executing CouchDB elixir test suite.
URL: https://github.com/apache/couchdb/issues/2099#issuecomment-522952604
 
 
   Hi @kocolosk ,
   
   I debugged the failing `CompactTest` test case and have managed to find the root cause of the failure.
   
   After understanding the logic of the test case and tracing the code flow, I realised that the failure happened due to the incorrect assert check at the following location:-
   	https://github.com/apache/couchdb/blob/master/test/elixir/test/compact_test.exs#L46
   
   This is because the final data size after deletion & further compaction is more than the deleted data size after only deletion, but not compaction.
   The opposite was being checked due to which the test case was failing consistently.
   
   Following are the values of the variables in question that I managed to trace:-
   
   		CompactTest
   		  * test compaction reduces size of deleted docs
   
   		Value of orig_data_size = 4436.
   
   		Value of orig_disk_size = 103907.
   
   		Value of deleted_data_size = 7455.
   
   		Value of final_data_size = 11924.
   
   		Value of final_disk_size = 218681.
   
   		  * test compaction reduces size of deleted docs (18819.2ms)
   
   I have made the necessary changes and submitted a PR for the same.
   	https://github.com/apache/couchdb/pull/2127

----------------------------------------------------------------
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:
users@infra.apache.org


With regards,
Apache Git Services