[kudu-CR] Add Google Breakpad support to Kudu

2017-01-06 Thread Lars Volker (Code Review)
Lars Volker has posted comments on this change. Change subject: Add Google Breakpad support to Kudu .. Patch Set 3: (3 comments) http://gerrit.cloudera.org:8080/#/c/5620/3/src/kudu/integration-tests/minidump_generation-itest.c

[kudu-CR] Kudu Jepsen Tests - Initial Commit

2017-01-06 Thread Dan Burkert (Code Review)
Dan Burkert has posted comments on this change. Change subject: Kudu Jepsen Tests - Initial Commit .. Patch Set 12: (2 comments) http://gerrit.cloudera.org:8080/#/c/5492/12/build-support/jenkins/build-and-test.sh File build-su

[kudu-CR] KuduRDD.collect fails because of NoSerializableException

2017-01-06 Thread Dan Burkert (Code Review)
Dan Burkert has posted comments on this change. Change subject: KuduRDD.collect fails because of NoSerializableException .. Patch Set 4: I've filed an issue and added some notes in https://issues.apache.org/jira/browse/KUDU-18

[kudu-CR] KuduRDD.collect fails because of NoSerializableException

2017-01-06 Thread Dan Burkert (Code Review)
Dan Burkert has posted comments on this change. Change subject: KuduRDD.collect fails because of NoSerializableException .. Patch Set 4: https://issues.apache.org/jira/browse/KUDU-1824 -- To view, visit http://gerrit.cloudera

[kudu-CR] [spark] KUDU-1631 push down StringStartsWith filters

2017-01-06 Thread Dan Burkert (Code Review)
Dan Burkert has posted comments on this change. Change subject: [spark] KUDU-1631 push down StringStartsWith filters .. Patch Set 3: Code-Review+2 -- To view, visit http://gerrit.cloudera.org:8080/5461 To unsubscribe, visit ht

[kudu-CR] [spark] KUDU-1631 push down StringStartsWith filters

2017-01-06 Thread Dan Burkert (Code Review)
Dan Burkert has submitted this change and it was merged. Change subject: [spark] KUDU-1631 push down StringStartsWith filters .. [spark] KUDU-1631 push down StringStartsWith filters This adds support for pushing down StringStar

[kudu-CR] log block manager: update list of buggy el6 versions

2017-01-06 Thread Todd Lipcon (Code Review)
Hello Dan Burkert, Adar Dembo, I'd like you to do a code review. Please visit http://gerrit.cloudera.org:8080/5632 to review the following change. Change subject: log_block_manager: update list of buggy el6 versions .. lo

[kudu-CR] Kudu Jepsen Tests - Initial Commit

2017-01-06 Thread Todd Lipcon (Code Review)
Todd Lipcon has posted comments on this change. Change subject: Kudu Jepsen Tests - Initial Commit .. Patch Set 12: Hrm, it sounds like the subsequent patch rewrites/improves a lot of what's in this one. Why not squash the pat

[kudu-CR] log block manager: update list of buggy el6 versions

2017-01-06 Thread Dan Burkert (Code Review)
Dan Burkert has posted comments on this change. Change subject: log_block_manager: update list of buggy el6 versions .. Patch Set 1: Code-Review+1 -- To view, visit http://gerrit.cloudera.org:8080/5632 To unsubscribe, visit ht

[kudu-CR] Add Google Breakpad support to Kudu

2017-01-06 Thread Alexey Serbin (Code Review)
Alexey Serbin has posted comments on this change. Change subject: Add Google Breakpad support to Kudu .. Patch Set 3: (10 comments) Does it work on OS X? Consider adding corresponding ifdefs at least at the first pass to all

[kudu-CR] KUDU-1643 Prune hash partitions based on IN-list predicates

2017-01-06 Thread Dan Burkert (Code Review)
Dan Burkert has posted comments on this change. Change subject: KUDU-1643 Prune hash partitions based on IN-list predicates .. Patch Set 12: (9 comments) Sorry this took so long, was out on holiday break for a stretch. Will t

[kudu-CR] Add some path / env related helper functions

2017-01-06 Thread Dinesh Bhat (Code Review)
Dinesh Bhat has posted comments on this change. Change subject: Add some path / env related helper functions .. Patch Set 3: Code-Review+1 (5 comments) http://gerrit.cloudera.org:8080/#/c/5618/3/src/kudu/util/env_util-test.cc

[kudu-CR] Kudu Jepsen Tests - Initial Commit

2017-01-06 Thread Alexey Serbin (Code Review)
Alexey Serbin has posted comments on this change. Change subject: Kudu Jepsen Tests - Initial Commit .. Patch Set 12: > Hrm, it sounds like the subsequent patch rewrites/improves a lot of > what's in this one. Why not squash t

[kudu-CR] Tightening ScanSpec primary bounds when range predicate exists

2017-01-06 Thread Dan Burkert (Code Review)
Dan Burkert has posted comments on this change. Change subject: Tightening ScanSpec primary bounds when range predicate exists .. Patch Set 4: Code-Review+2 -- To view, visit http://gerrit.cloudera.org:8080/5360 To unsubscribe

[kudu-CR] Tightening ScanSpec primary bounds when range predicate exists

2017-01-06 Thread Dan Burkert (Code Review)
Dan Burkert has submitted this change and it was merged. Change subject: Tightening ScanSpec primary bounds when range predicate exists .. Tightening ScanSpec primary bounds when range predicate exists Currently, push upper bou

[kudu-CR] Initial draft of release notes and doc updates for 1.2

2017-01-06 Thread Jean-Daniel Cryans (Code Review)
Jean-Daniel Cryans has posted comments on this change. Change subject: Initial draft of release notes and doc updates for 1.2 .. Patch Set 2: (1 comment) Got my own rendering and saw one issue. http://gerrit.cloudera.org:8080

[kudu-CR] Restructure release notes in preparation for 1.2 release

2017-01-06 Thread Jean-Daniel Cryans (Code Review)
Jean-Daniel Cryans has posted comments on this change. Change subject: Restructure release notes in preparation for 1.2 release .. Patch Set 2: Code-Review+2 -- To view, visit http://gerrit.cloudera.org:8080/5602 To unsubscrib

[kudu-CR] KUDU-1741: Make MiniCluster and ExternalMiniCluster follow one semantic for Restart

2017-01-06 Thread Jean-Daniel Cryans (Code Review)
Jean-Daniel Cryans has posted comments on this change. Change subject: KUDU-1741: Make MiniCluster and ExternalMiniCluster follow one semantic for Restart .. Patch Set 1: Intuitively restart means "stop then start" for me, and

[kudu-CR] Restructure release notes in preparation for 1.2 release

2017-01-06 Thread Todd Lipcon (Code Review)
Todd Lipcon has submitted this change and it was merged. Change subject: Restructure release notes in preparation for 1.2 release .. Restructure release notes in preparation for 1.2 release * Moved the 1.1 release notes to the

[kudu-CR] Initial draft of release notes and doc updates for 1.2

2017-01-06 Thread Todd Lipcon (Code Review)
Todd Lipcon has posted comments on this change. Change subject: Initial draft of release notes and doc updates for 1.2 .. Patch Set 2: (2 comments) http://gerrit.cloudera.org:8080/#/c/5604/2/docs/release_notes.adoc File docs/r

[kudu-CR] Initial draft of release notes and doc updates for 1.2

2017-01-06 Thread Todd Lipcon (Code Review)
Hello Dinesh Bhat, Kudu Jenkins, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/5604 to look at the new patch set (#3). Change subject: Initial draft of release notes and doc updates for 1.2 .

[kudu-CR] KUDU-1824. KuduRDD.collect fails because of NoSerializableException

2017-01-06 Thread Dan Burkert (Code Review)
Hello Jean-Daniel Cryans, Todd Lipcon, I'd like you to do a code review. Please visit http://gerrit.cloudera.org:8080/5636 to review the following change. Change subject: KUDU-1824. KuduRDD.collect fails because of NoSerializableException ..

[kudu-CR] KUDU-1824. KuduRDD.collect fails because of NoSerializableException

2017-01-06 Thread Dan Burkert (Code Review)
Dan Burkert has uploaded a new patch set (#2). Change subject: KUDU-1824. KuduRDD.collect fails because of NoSerializableException .. KUDU-1824. KuduRDD.collect fails because of NoSerializableException This also fixes a few sty

[kudu-CR] KuduRDD.collect fails because of NoSerializableException

2017-01-06 Thread Dan Burkert (Code Review)
Dan Burkert has posted comments on this change. Change subject: KuduRDD.collect fails because of NoSerializableException .. Patch Set 4: I've put up a new gerrit based on Kousuke's suggestion: https://gerrit.cloudera.org/#/c/5

[kudu-CR] [kudu-jepsen] Kudu Jepsen tests

2017-01-06 Thread Alexey Serbin (Code Review)
Hello Kudu Jenkins, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/5492 to look at the new patch set (#13). Change subject: [kudu-jepsen] Kudu Jepsen tests .. [kudu-jepsen] Kudu Jepsen te

[kudu-CR] [kudu-jepsen] Kudu Jepsen tests

2017-01-06 Thread Alexey Serbin (Code Review)
Alexey Serbin has posted comments on this change. Change subject: [kudu-jepsen] Kudu Jepsen tests .. Patch Set 12: (13 comments) http://gerrit.cloudera.org:8080/#/c/5492/12/java/kudu-jepsen/src/main/clojure/jepsen/kudu.clj Fil

[kudu-CR] [kudu-jepsen] Kudu Jepsen tests

2017-01-06 Thread Alexey Serbin (Code Review)
Hello Kudu Jenkins, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/5492 to look at the new patch set (#14). Change subject: [kudu-jepsen] Kudu Jepsen tests .. [kudu-jepsen] Kudu Jepsen te

[kudu-CR] [kudu-jepsen] added Jenkins script

2017-01-06 Thread Alexey Serbin (Code Review)
Hello Kudu Jenkins, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/5624 to look at the new patch set (#4). Change subject: [kudu-jepsen] added Jenkins script .. [kudu-jepsen] added Jenkin

[kudu-CR] KUDU-1821. Noisy warning from catalog manager

2017-01-06 Thread Jun He (Code Review)
Jun He has uploaded a new change for review. http://gerrit.cloudera.org:8080/5637 Change subject: KUDU-1821. Noisy warning from catalog manager .. KUDU-1821. Noisy warning from catalog manager This commit changes the returned