{"push_id":859311768,"size":1,"distinct_size":1,"ref":"refs/heads/master","head":"b81b430987f46844b400e933f53f4312e8d0743f","before":"75a8bad2abcb9932913288e47ca1adaa03de4c0a","commits":[{"sha":"b81b430987f46844b400e933f53f4312e8d0743f","author":{"email":"982811392032c5829eb905464d4d638fb6ed4f34@fb.com","name":"Nathan Bronson"},"message":"Switch to thread-local random for skiplist\n\nSummary:\nUsing a TLS random instance for skiplist makes it smaller\n(useful for hash_skiplist_rep) and prepares skiplist for concurrent\nadds.  This diff also modifies the branching factor math to avoid an\nunnecessary division.\n\nThis diff has the effect of changing the sequence of skip list node\nheight choices made by tests, so it has the potential to cause unit\ntest failures for tests that implicitly rely on the exact structure\nof the skip list.  Tests that try to exactly trigger a compaction are\nlikely suspects for this problem (these tests have always been brittle to\nchanges in the skiplist details).  I've minimizes this risk by reseeding\nthe main thread's Random at the beginning of each test, increasing the\nuniversal compaction size_ratio limit from 101% to 105% for some tests,\nand verifying that the tests pass many times.\n\nTest Plan: for i in `seq 0 9`; do make check; done\n\nReviewers: sdong, igor\n\nReviewed By: igor\n\nSubscribers: dhruba\n\nDifferential Revision: https://reviews.facebook.net/D50439","distinct":true,"url":"https://api.github.com/repos/facebook/rocksdb/commits/b81b430987f46844b400e933f53f4312e8d0743f"}]}