nickva opened a new pull request, #6179: URL: https://github.com/apache/couchdb/pull/6179
When we get duplicates in the rev tree stemming repairs the tree by rebuilding it from paths. Repair used to apply the revs_limit first, and then mergedthe fragments. If the duplicates were far enough appart from each other (source dbs with different revs_limits were pushing to a target) the fragments might not be fold and the shorter one could stay around as an extra leaf. Since #6168 it would be even worse since a leaf could end up with nothing stored for it, which would be a broken rev tree. So instead we should fold the paths first then stem the result. This was noticed when fixing the property tests to avoid discarding so many generating cases and in this case we never actually tested this multi-fragment merge scenario. Move to use numbers for revisions so we don't have to generate random revs then reject them with ?IMPLIES any longer. Also increase the number of runs to 500 from a default of 100. Issue: #6170 -- 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. To unsubscribe, e-mail: [email protected] For queries about this service, please contact Infrastructure at: [email protected]
