{"push_id":2094870770,"size":23,"distinct_size":23,"ref":"refs/heads/pr-test-1","head":"2a5be6cd3fe370248ce2d21b568bf5ba62a36430","before":"e202d955a022a501ea49bc5906c36b38bdbc50df","commits":[{"sha":"7d4d40b15b21a9050b11d85745655aaf00a0ed94","author":{"name":"Enrico Olivelli","email":"38b90b426ed2a6c146609b160d85505766819d2c@apache.org"},"message":"ISSUE #559: Fix findbugs error on SyncCallbackUtils\n\nFix FindBugs warning BC_UNCONFIRMED_CAST in  class SyncCallbackUtils\n\nAuthor: eolivelli <eolivelli@apache.org>\nAuthor: Enrico Olivelli <eolivelli@apache.org>\n\nReviewers: Ivan Kelly <ivan@ivankelly.net>, Jia Zhai <None>\n\nThis closes #560 from eolivelli/findbugs-synccallback, closes #559","distinct":true,"url":"https://api.github.com/repos/test-things-on-github/bookkeeper/commits/7d4d40b15b21a9050b11d85745655aaf00a0ed94"},{"sha":"49fafc2b958ce18758094d643a892d315d10ae60","author":{"name":"Ivan Kelly","email":"053116c27f7cefb015ca522af26b33796f5e816a@midokura.com"},"message":"Fix for BookKeeperBuildersTest\n\nPrevious findbugs fix broke it.\n\nAuthor: Ivan Kelly <ivan@midokura.com>\n\nReviewers: Enrico Olivelli <eolivelli@gmail.com>, Sijie Guo <sijie@apache.org>\n\nThis closes #638 from ivankelly/bkbuildertest-fail","distinct":true,"url":"https://api.github.com/repos/test-things-on-github/bookkeeper/commits/49fafc2b958ce18758094d643a892d315d10ae60"},{"sha":"9a8b4712f3752413befa3753194cc531650002e8","author":{"name":"Matteo Merli","email":"e95e5f3c2c97b8b8cbbfcb3127b436aaabab5eed@yahoo-inc.com"},"message":"Recycle buffers used in EntryLogger#addEntry\n\nTo avoid creating a buffer and letting it go out of scope and be GC'd\neach time.\n\nThis change was originally made by merlimat in the yahoo-4.3 branch.\n\nAuthor: Matteo Merli <mmerli@yahoo-inc.com>\n\nReviewers: Enrico Olivelli <eolivelli@gmail.com>, Sijie Guo <sijie@apache.org>, Ivan Kelly <ivank@apache.org>\n\nThis closes #639 from ivankelly/recycle-buf","distinct":true,"url":"https://api.github.com/repos/test-things-on-github/bookkeeper/commits/9a8b4712f3752413befa3753194cc531650002e8"},{"sha":"e977961f88305328f5a362db27aad3b5c597f1a7","author":{"name":"Matteo Merli","email":"e95e5f3c2c97b8b8cbbfcb3127b436aaabab5eed@yahoo-inc.com"},"message":"Use ByteBuf for entrylogger reads\n\nOriginal patch from merlimat in yahoo-4.3 branch\n\nAuthor: Matteo Merli <mmerli@yahoo-inc.com>\nAuthor: Matteo Merli <mmerli@apache.org>\n\nReviewers: Enrico Olivelli <eolivelli@gmail.com>, Ivan Kelly <ivank@apache.org>\n\nThis patch had conflicts when merged, resolved by\nCommitter: Enrico Olivelli <eolivelli@apache.org>\n\nThis closes #640 from ivankelly/refcount-for-read","distinct":true,"url":"https://api.github.com/repos/test-things-on-github/bookkeeper/commits/e977961f88305328f5a362db27aad3b5c597f1a7"},{"sha":"142dc219a2926ca4dc25214acc6ee8d212cc7d37","author":{"name":"Matteo Merli","email":"e95e5f3c2c97b8b8cbbfcb3127b436aaabab5eed@yahoo-inc.com"},"message":"Fixed ref counting release on read errors\n\nOriginal change by merlimat in yahoo-3.4 branch\n\nAuthor: Matteo Merli <mmerli@yahoo-inc.com>\n\nReviewers: Sijie Guo <sijie@apache.org>\n\nThis closes #644 from ivankelly/fix-ref-read","distinct":true,"url":"https://api.github.com/repos/test-things-on-github/bookkeeper/commits/142dc219a2926ca4dc25214acc6ee8d212cc7d37"},{"sha":"c45f8166bca97503d8937e6da112f98925c78056","author":{"name":"Matteo Merli","email":"e95e5f3c2c97b8b8cbbfcb3127b436aaabab5eed@yahoo-inc.com"},"message":"Fix concurrent v2 reads on the same ledger/entry\n\nOriginal fix from by merlimat in the yahoo-4.3 branch\n\nAuthor: Matteo Merli <mmerli@yahoo-inc.com>\n\nReviewers: Enrico Olivelli <eolivelli@gmail.com>, Sijie Guo <sijie@apache.org>\n\nThis closes #645 from ivankelly/con-read-v2-fix","distinct":true,"url":"https://api.github.com/repos/test-things-on-github/bookkeeper/commits/c45f8166bca97503d8937e6da112f98925c78056"},{"sha":"cfdc2b1ff5affe4271db5cd0f2f0f4ec4262e378","author":{"name":"Ivan Kelly","email":"a28a20f281c8780c2e8734549b9a1af30a13d954@apache.org"},"message":"Increase timeouts in TestDisableEnsembleChange\n\nIn some narrow cases, if we manage to connect to the bookie, but the bookie\nhasn't fully initialized, an add entry message will be dropped, and the\nclient will wait for the timeout to complete. The timeout is 5 seconds\nby default, which was 5x the timeout in the test. This patch increases\nthe test timeout to 10s, which should give enough time for the addEntry\nto fail and be retried.\n\nAuthor: Ivan Kelly <ivan@midokura.com>\n\nReviewers: Enrico Olivelli <eolivelli@gmail.com>, Sijie Guo <sijie@apache.org>\n\nThis closes #646 from ivankelly/ensemble-change-test","distinct":true,"url":"https://api.github.com/repos/test-things-on-github/bookkeeper/commits/cfdc2b1ff5affe4271db5cd0f2f0f4ec4262e378"},{"sha":"d60cad5da1ea3eaedad7d10fc49fc912646f7e96","author":{"name":"Ivan Kelly","email":"053116c27f7cefb015ca522af26b33796f5e816a@midokura.com"},"message":"Fix a typo in PCBC logging\n\nAll ops had writeLac in their logged text, which is confusing\n\nAuthor: Ivan Kelly <ivan@midokura.com>\n\nReviewers: Enrico Olivelli <eolivelli@gmail.com>, Sijie Guo <sijie@apache.org>\n\nThis closes #647 from ivankelly/pcbc-typo","distinct":true,"url":"https://api.github.com/repos/test-things-on-github/bookkeeper/commits/d60cad5da1ea3eaedad7d10fc49fc912646f7e96"},{"sha":"e9e994f8964e1017210fe15ac863f6f9996cf3ed","author":{"name":"Ivan Kelly","email":"053116c27f7cefb015ca522af26b33796f5e816a@midokura.com"},"message":"Pool the V2 keys\n\nTo generate less garbage in the fast path.\n\nAuthor: Ivan Kelly <ivan@midokura.com>\nAuthor: Matteo Merli <mmerli@yahoo-inc.com>\n\nReviewers: Sijie Guo <sijie@apache.org>\n\nThis closes #648 from ivankelly/pool-keys","distinct":true,"url":"https://api.github.com/repos/test-things-on-github/bookkeeper/commits/e9e994f8964e1017210fe15ac863f6f9996cf3ed"},{"sha":"8acdbe196e1e4116ca4d4303a9bbd8952a509ffa","author":{"name":"Ivan Kelly","email":"053116c27f7cefb015ca522af26b33796f5e816a@midokura.com"},"message":"Pool AddCompletions\n\nTo avoid generating garbage in the fastpath.\n\nAuthor: Ivan Kelly <ivan@midokura.com>\n\nReviewers: Enrico Olivelli <eolivelli@apache.org>\n\nThis closes #649 from ivankelly/pool-completions","distinct":true,"url":"https://api.github.com/repos/test-things-on-github/bookkeeper/commits/8acdbe196e1e4116ca4d4303a9bbd8952a509ffa"},{"sha":"91f7cb83c226ad9bb36e35018147d990e3103aa2","author":{"name":"Enrico Olivelli","email":"38b90b426ed2a6c146609b160d85505766819d2c@apache.org"},"message":"ISSUE #606: Enhancements in developer environment\n\nImprovements in maven configuration in order to speed up development\n\nAuthor: Enrico Olivelli <eolivelli@apache.org>\n\nReviewers: Ivan Kelly <ivank@apache.org>\n\nThis closes #661 from eolivelli/plugins-upgrade, closes #606","distinct":true,"url":"https://api.github.com/repos/test-things-on-github/bookkeeper/commits/91f7cb83c226ad9bb36e35018147d990e3103aa2"},{"sha":"6fc34a5bc14f4542fb8b627e8d50a89068ceb7f8","author":{"name":"Aaron Gresch","email":"71bf8deceb8ab7763e9c1f005fdd206e93e2f6d7@yahoo-inc.com"},"message":"Fail earlier when encountering DNS issues\n\nWe had some DNS issues today and various TestRegionAwareEnsemblePlacementPolicy tests were failing for me.\n\nThe LOG warning in RackawareEnsemblePlacementPolicyIml was not printing the right variable, which would have given a clue.\n\nAdditionally, the repp.myRegion was set to UNKNOWN, causing incorrect code flow.  Added an assert to kill the test as soon as this occurs.\n\nAuthor: Aaron Gresch <agresch@yahoo-inc.com>\n\nReviewers: Ivan Kelly <ivan@ivankelly.net>, Enrico Olivelli <eolivelli@gmail.com>, Sijie Guo <sijie@apache.org>\n\nThis closes #658 from agresch/agresch_dns_failure","distinct":true,"url":"https://api.github.com/repos/test-things-on-github/bookkeeper/commits/6fc34a5bc14f4542fb8b627e8d50a89068ceb7f8"},{"sha":"988a7b63d98aa58c6973bdfafdf9a12bba9f44f3","author":{"name":"Aaron Gresch","email":"71bf8deceb8ab7763e9c1f005fdd206e93e2f6d7@yahoo-inc.com"},"message":"Recycle instances of WriteEntryProcessor and ReadEntryProcessor\n\nmerge of some changes from Yahoo's repo\n\nAuthor: Aaron Gresch <agresch@yahoo-inc.com>\n\nReviewers: Sijie Guo <sijie@apache.org>, Matteo Merli <mmerli@apache.org>\n\nThis closes #669 from agresch/agresch_recycle_entryprocessor","distinct":true,"url":"https://api.github.com/repos/test-things-on-github/bookkeeper/commits/988a7b63d98aa58c6973bdfafdf9a12bba9f44f3"},{"sha":"621844e4343d8cef37af437be226e1e351539a99","author":{"name":"Jia Zhai","email":"f9b3ee6d811907feda533a85ea698e15dfe82f6d@apache.org"},"message":"ISSUE #662: Introduce Bookie Registration Manager for bookie server\n\nDescriptions of the changes in this PR:\nWhile in BOOKKEEPER-628, It is trying to improve/generalize the bookie registration process.\nThis PR is for bookie side, it tries to introduce Bookie registration manager for bookie server.\n\nAuthor: Jia Zhai <zhaijia@apache.org>\n\nReviewers: Enrico Olivelli <eolivelli@gmail.com>, Sijie Guo <sijie@apache.org>\n\nThis closes #663 from zhaijack/bookie_registration, closes #662","distinct":true,"url":"https://api.github.com/repos/test-things-on-github/bookkeeper/commits/621844e4343d8cef37af437be226e1e351539a99"},{"sha":"1ea27189c73869d0279f1b0d7e737f65933bbf21","author":{"name":"Sijie Guo","email":"b145f0fd70c46ca4e90736efcc8fe3a047c41511@apache.org"},"message":"ISSUE #596 ISSUE #583: Auto replication should honor ensemble placement policy\n\nDescriptions of the changes in this PR:\n\nThis pull request ports the changes from [twitter/bookkeeperfc7e171](https://github.com/twitter/bookkeeper/commit/fc7e171135a58cddb9e91f5f614b3ceb6f9f9fee)\n\nThe changes include:\n\n1. when bookkeeper admin re-replicates a ledger, it will pick a bookie from the available bookies in the cluster to satisfy the placement constraint. (#596)\n2. hence, remove `targetBookie` from ReplicationWorker, because the parameter will volatile the placement constraint. (#583)\n3. at the same time, change `LedgerFragement` to represent the number of bookies that need to be check and replicate. a) the ledger checker can use the correct bookie index for verifying the existence of entries in a bookie (for stripping case) b) only read entries one time when need to replicate them to multiple bookies.\n\nAuthor: Sijie Guo <sijie@apache.org>\n\nReviewers: Ivan Kelly <ivank@apache.org>\n\nThis closes #641 from sijie/twitter_autorecovery_fixes, closes #596, closes #583","distinct":true,"url":"https://api.github.com/repos/test-things-on-github/bookkeeper/commits/1ea27189c73869d0279f1b0d7e737f65933bbf21"},{"sha":"3c493223ea2ebd43b1a871c66ca36d91bfb79fdf","author":{"name":"Sijie Guo","email":"b145f0fd70c46ca4e90736efcc8fe3a047c41511@apache.org"},"message":"ISSUE #265: Add persistable bookie status\n\nDescriptions of the changes in this PR:\n\n- Add support for persisting bookie status\n- Add configuration to enable/disable this feature\n- Add test cases\n- Improve bookie status stat\n\nIt also includes changes for BOOKKEEPER-754 (flush ledger storage after replaying journal)\n\nAuthor: Sijie Guo <sijie@apache.org>\nAuthor: Yiming Zang <yzang@twitter.com>\n\nReviewers: Yiming Zang <yzang2016@gmail.com>, Enrico Olivelli <eolivelli@gmail.com>\n\nThis closes #266 from sijie/server_side_crc32, closes #265","distinct":true,"url":"https://api.github.com/repos/test-things-on-github/bookkeeper/commits/3c493223ea2ebd43b1a871c66ca36d91bfb79fdf"},{"sha":"6fb5b5f668a49589c0736d606392269ca8355870","author":{"name":"Sijie Guo","email":"b145f0fd70c46ca4e90736efcc8fe3a047c41511@apache.org"},"message":"ISSUE #675: Enable checkstyle in a few packages\n\nThese packages include:\n\n- feature\n- processor\n- shims\n- stats\n- streaming\n- versioning\n- zookeeper\n\nnot fully enabled in conf package.\n\nAuthor: Sijie Guo <sijie@apache.org>\n\nReviewers: Jia Zhai <None>\n\nThis closes #676 from sijie/fix_checkstyles, closes #675","distinct":true,"url":"https://api.github.com/repos/test-things-on-github/bookkeeper/commits/6fb5b5f668a49589c0736d606392269ca8355870"},{"sha":"f09835e5675611005f25cb8930bf54d90471f98d","author":{"name":"Ivan Kelly","email":"a28a20f281c8780c2e8734549b9a1af30a13d954@apache.org"},"message":"DistributionSchedule uses custom wrapped int[] rather than HashSet\n\nThis avoids the autoboxing on Integers and allocations of cells for\nthe hashset.\n\nThis also implies using wrapped int[] object for the write set in the\nDistributionSchedules and PlacementPolicies.\n\nThis patch was originally submitted as dc7933b on the yahoo-4.3\nbranch, though this has been modified extensively to remove the\ndependency on carrotsearch hppc and to allow it to work with placement\npolicies which didn't exist at the time of the original patch.\n\nAuthor: Ivan Kelly <ivank@apache.org>\n\nReviewers: Enrico Olivelli <eolivelli@gmail.com>, Sijie Guo <sijie@apache.org>\n\nThis closes #657 from ivankelly/yahoo-backports","distinct":true,"url":"https://api.github.com/repos/test-things-on-github/bookkeeper/commits/f09835e5675611005f25cb8930bf54d90471f98d"},{"sha":"19825a63f21566927da153498f4680f4d89d45cd","author":{"name":"Matteo Merli","email":"e95e5f3c2c97b8b8cbbfcb3127b436aaabab5eed@yahoo-inc.com"},"message":"Recycle instances of callback to obtain a PerChannelBookie client\n\nThis change was originally commit e2e77863 in the yahoo-4.3 branch.\n\nAuthor: Matteo Merli <mmerli@yahoo-inc.com>\n\nReviewers: Sijie Guo <sijie@apache.org>\n\nThis closes #679 from ivankelly/yahoo-bp-3","distinct":true,"url":"https://api.github.com/repos/test-things-on-github/bookkeeper/commits/19825a63f21566927da153498f4680f4d89d45cd"},{"sha":"55f568b337d2864063eb02101e621e6abba64c48","author":{"name":"eolivelli","email":"38b90b426ed2a6c146609b160d85505766819d2c@apache.org"},"message":"Copy site from 4.5.0 to 4.5.1\n\nCopy site directory in order to start new release notes for 4.5.1\n\nAuthor: eolivelli <eolivelli@apache.org>\n\nReviewers: Sijie Guo <sijie@apache.org>\n\nThis closes #680 from eolivelli/release-notes-451-copy","distinct":true,"url":"https://api.github.com/repos/test-things-on-github/bookkeeper/commits/55f568b337d2864063eb02101e621e6abba64c48"}]}
{"actor":{"display_login":"test-things-on-github"}}