{"push_id":1294335594,"size":5,"distinct_size":5,"ref":"refs/heads/master","head":"708f38b2b932ef88dda2f990ac95a8de1cd226cb","before":"cf1bd0dc90940030330f6803e934da805a2eb1fb","commits":[{"sha":"455e1b7460984afb9ffcae8cae7bb85b9c0837a7","author":{"name":"Todd Lipcon","email":"dd5965a1ccaca30b2dd5053f256a6d9b88ca567f@apache.org"},"message":"linked_list-test: set longer history retention\n\nThe default 15 minute history retention was causing linked_list-test to\nGC some history in the case that -seconds_to_run was set significantly\nlonger than the default.\n\nThis sets the retention to 1 day to prevent such issues.\n\nI also removed an old unused function I noticed while making this\nchange.\n\nChange-Id: I43395eb73c99f79be1884b6a22fab06f22c58980\nReviewed-on: http://gerrit.cloudera.org:8080/4374\nReviewed-by: Mike Percy <mpercy@apache.org>\nTested-by: Mike Percy <mpercy@apache.org>","distinct":true,"url":"https://api.github.com/repos/EvilMcJerkface/kudu/commits/455e1b7460984afb9ffcae8cae7bb85b9c0837a7"},{"sha":"3a9ea63b9f210b64124b2d9d2b5f14ed1214f272","author":{"name":"Todd Lipcon","email":"dd5965a1ccaca30b2dd5053f256a6d9b88ca567f@apache.org"},"message":"tablet_history_gc-itest: set scanner_ttl_ms high\n\nThis test is currently flaky on slower builds (TSAN/ASAN) because of\nscanner expiration. The test opens scanners and then waits some time\nbefore actually reading from them. In a slow build, this might be more\nthan the default 1-minute scanner TTL, which would cause the scanner to\nexpire and the test to fail to read.\n\nChange-Id: I4bd8a5bdc77871a95b3630f1622a802967d14b84\nReviewed-on: http://gerrit.cloudera.org:8080/4375\nTested-by: Kudu Jenkins\nReviewed-by: Mike Percy <mpercy@apache.org>","distinct":true,"url":"https://api.github.com/repos/EvilMcJerkface/kudu/commits/3a9ea63b9f210b64124b2d9d2b5f14ed1214f272"},{"sha":"81ee6c3f3b3b05f8f145b823a2db5b1e7f80233e","author":{"name":"Todd Lipcon","email":"dd5965a1ccaca30b2dd5053f256a6d9b88ca567f@apache.org"},"message":"Fix one more flakiness in tablet_replacement-itest\n\nThe same as 30a8337f05a8b92faff9400298aae640b6ebbecb, there was a case\nwhere we were electing a leader and assuming it could immediately\nperform a config change, even though its original NO_OP may not have\nbeen committed yet.\n\nChange-Id: I1924e3fd001887d166866252f268354827a9da41\nReviewed-on: http://gerrit.cloudera.org:8080/4345\nTested-by: Todd Lipcon <todd@apache.org>\nReviewed-by: Mike Percy <mpercy@apache.org>","distinct":true,"url":"https://api.github.com/repos/EvilMcJerkface/kudu/commits/81ee6c3f3b3b05f8f145b823a2db5b1e7f80233e"},{"sha":"30099cf9e62bcfe9344bcd76807b080d423ef0e1","author":{"name":"Will Berkeley","email":"dbefc1fe2297210a4f3ef394acb9820ae7be4684@gmail.com"},"message":"Add RegexpKuduOperationsProducer class\n\nThis patch adds the RegexpKuduOperationsProducer class. This class\nserializes Event objects to Kudu inserts or upserts by decoding\nthe body into a string, parsing the string using a regular\nexpression, and finally mapping match groups to columns by\nmatching the name of the match group to the name of the column.\nParsed values are naively coerced to the proper type.\n\nThis provides an easy-to-use but flexible way to ingest data with\nvarying schemas into Kudu from Flume.\n\nChange-Id: Ibd64542095f0064a21635ec76c46d6a1be98b7ea\nReviewed-on: http://gerrit.cloudera.org:8080/3883\nTested-by: Kudu Jenkins\nReviewed-by: Mike Percy <mpercy@apache.org>","distinct":true,"url":"https://api.github.com/repos/EvilMcJerkface/kudu/commits/30099cf9e62bcfe9344bcd76807b080d423ef0e1"},{"sha":"708f38b2b932ef88dda2f990ac95a8de1cd226cb","author":{"name":"Todd Lipcon","email":"dd5965a1ccaca30b2dd5053f256a6d9b88ca567f@apache.org"},"message":"util: avoid redundant copies in various macros\n\nA lot of macros have the form:\n\n  #define FOO(arg) \\\n    Foo arg_ = (arg); \\\n    ... do something with arg_ ...\n\nOf course the purpose of the 'arg_' local is to avoid double-evaluation\nof the argument in the case that it's something expensive or something\nwith a side effect. However, this pattern results in an extra copy of\n'arg' in the case that it's actually a local, eg in the pattern:\n\n  Foo f = ...;\n  FOO(f);\n\nThis expands out to a redundant copy 'Foo arg_ = f;'.\n\nIt turns out that the solution is to make 'Foo arg_' a const reference\n'const Foo& arg_'. At first, this seems wrong: references to a temporary\nare often bad. However, it turns out that C++ has a rule that const\nreferences to temporaries on the stack keep that temporary alive as long as\nthe reference stays in scope. So, this is safe, and avoids an extra\ncopy.\n\nChange-Id: I6057751d5c6790b2d350e77636b58079e2093cb7\nReviewed-on: http://gerrit.cloudera.org:8080/4379\nTested-by: Kudu Jenkins\nReviewed-by: Mike Percy <mpercy@apache.org>","distinct":true,"url":"https://api.github.com/repos/EvilMcJerkface/kudu/commits/708f38b2b932ef88dda2f990ac95a8de1cd226cb"}]}
{"actor":{"display_login":"EvilMcJerkface"}}