{"push_id":854760274,"size":9,"distinct_size":9,"ref":"refs/heads/master","head":"d47119f48a645008f4050dc0506789454b90e26c","before":"14c6e1a04c4bece57ca1e4842f357325892f3208","commits":[{"sha":"d84a36dd471f72833798a81b5503f51b54ffe0ae","author":{"email":"1280782e6fb517b5a25f52258612ff12f647014c@fb.com","name":"Satnam Singh"},"message":"Add RocksDb/GeoDb Iterator interface\n\nSummary:\nThis diff is a first step towards an iterator based interface for the\nSearchRadial method which replaces a vector of GeoObjects with an\niterator for GeoObjects. This diff works by just wrapping the iterator\nfor the encapsulated vector of GeoObjects. A future diff could extend\nthis approach by defining an interator in terms of the underlying\niteration in SearchRadial which would then remove the need to have\nan in-memory representation for all the matching GeoObjects.\nFixes T8421387\n\nTest Plan:\nThe existing tests have been modified to work with the new\ninterface.\n\nReviewers: IslamAbdelRahman, kradhakrishnan, dhruba, igor\n\nReviewed By: igor\n\nSubscribers: igor, dhruba, leveldb\n\nDifferential Revision: https://reviews.facebook.net/D50031","distinct":true,"url":"https://api.github.com/repos/facebook/rocksdb/commits/d84a36dd471f72833798a81b5503f51b54ffe0ae"},{"sha":"71f7c0b75298d41f38e37dae0ee0315b3f502687","author":{"email":"bef395f6fd4d67db5ef1aca73823e8ea900fb75c@gmail.com","name":"Brent Garber"},"message":"Fix introduced in 2ab7065 was reverted by 18285c1.\n\nCorrects:\n\ndb/memtablerep_bench.cc:135:22: error: ‘FLAGS_env’ defined but not used [-Werror=unused-variable]\n static rocksdb::Env* FLAGS_env = rocksdb::Env::Default();\n                      ^\ncc1plus: all warnings being treated as errors\nMakefile:1147: recipe for target 'db/memtablerep_bench.o' failed","distinct":true,"url":"https://api.github.com/repos/facebook/rocksdb/commits/71f7c0b75298d41f38e37dae0ee0315b3f502687"},{"sha":"99d5dc5d82949e8204638f3b1cb7c6a7b671cca9","author":{"email":"40bc6d3c318538862cbe0610c762785daf6403c0@microsoft.com","name":"Dmitri Smirnov"},"message":"Do not disable compiler warnings:\n  C4101 'identifier' : unreferenced local variable\n  C4189 'identifier' : local variable is initialized but not referenced\n  C4100 'identifier' : unreferenced formal parameter\n  C4296 'operator' : expression is always false","distinct":true,"url":"https://api.github.com/repos/facebook/rocksdb/commits/99d5dc5d82949e8204638f3b1cb7c6a7b671cca9"},{"sha":"1b9e0202635043842f2c3e2b25da0f364e4f435c","author":{"email":"a0e79c420b85b9d5d6f43403bcc7bcfb053f36b9@gmail.com","name":"SherlockNoMad"},"message":"Add Option to Skip Flushing in TableBuilder","distinct":true,"url":"https://api.github.com/repos/facebook/rocksdb/commits/1b9e0202635043842f2c3e2b25da0f364e4f435c"},{"sha":"b932cf394ae610730a54732d3deec7453541ba2e","author":{"email":"a0e79c420b85b9d5d6f43403bcc7bcfb053f36b9@gmail.com","name":"SherlockNoMad"},"message":"Fix Travis Build Error","distinct":true,"url":"https://api.github.com/repos/facebook/rocksdb/commits/b932cf394ae610730a54732d3deec7453541ba2e"},{"sha":"a16f80e6daabe57324126545cf38b43cf8f88f12","author":{"email":"a0e79c420b85b9d5d6f43403bcc7bcfb053f36b9@gmail.com","name":"SherlockNoMad"},"message":"Move skip_table_builder_flush to BlockBasedTableOption","distinct":true,"url":"https://api.github.com/repos/facebook/rocksdb/commits/a16f80e6daabe57324126545cf38b43cf8f88f12"},{"sha":"add78850441fff6c41f06652ddbabbc1bd1be121","author":{"email":"a0e79c420b85b9d5d6f43403bcc7bcfb053f36b9@gmail.com","name":"SherlockNoMad"},"message":"Fix white space at end of line","distinct":true,"url":"https://api.github.com/repos/facebook/rocksdb/commits/add78850441fff6c41f06652ddbabbc1bd1be121"},{"sha":"8faa40f54f9d546937f3b3fb4d0cfb31664e0997","author":{"email":"1280782e6fb517b5a25f52258612ff12f647014c@fb.com","name":"Satnam Singh"},"message":"Delete test iterators\n\nSummary:\nValgrind reports an issue with the test for GeoIterator.\nThis diff explicitly deletes the two iterators used in this test.\n\nTest Plan: This diff is for a test. The test still passes.\n\nSubscribers: dhruba, leveldb\n\nDifferential Revision: https://reviews.facebook.net/D50193","distinct":true,"url":"https://api.github.com/repos/facebook/rocksdb/commits/8faa40f54f9d546937f3b3fb4d0cfb31664e0997"},{"sha":"d47119f48a645008f4050dc0506789454b90e26c","author":{"email":"1280782e6fb517b5a25f52258612ff12f647014c@fb.com","name":"Satnam Singh"},"message":"Merge branch 'master' into fixmem\n\nConflicts:\n\tutilities/geodb/geodb_test.cc","distinct":true,"url":"https://api.github.com/repos/facebook/rocksdb/commits/d47119f48a645008f4050dc0506789454b90e26c"}]}