[kudu-CR] Kudu Backup/Restore Spark Jobs

2018-06-19 Thread Grant Henke (Code Review)
Grant Henke has posted comments on this change. ( http://gerrit.cloudera.org:8080/10375 ) Change subject: Kudu Backup/Restore Spark Jobs .. Patch Set 17: (25 comments)

[kudu-CR] Kudu Backup/Restore Spark Jobs

2018-06-19 Thread Grant Henke (Code Review)
Hello Mike Percy, Kudu Jenkins, Adar Dembo, Todd Lipcon, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/10375 to look at the new patch set (#18). Change subject: Kudu Backup/Restore Spark Jobs

[kudu-CR] KUDU-2480: tsan failure of master-stress-test

2018-06-19 Thread Dan Burkert (Code Review)
Hello Hao Hao, I'd like you to do a code review. Please visit http://gerrit.cloudera.org:8080/10770 to review the following change. Change subject: KUDU-2480: tsan failure of master-stress-test .. KUDU-2480: tsan failure

[kudu-CR] KUDU-2191: Correct error handling in Metadata upgrade tool

2018-06-19 Thread Dan Burkert (Code Review)
Dan Burkert has posted comments on this change. ( http://gerrit.cloudera.org:8080/10728 ) Change subject: KUDU-2191: Correct error handling in Metadata upgrade tool .. Patch Set 2: (1 comment)

[kudu-CR] KUDU-2191: HMS Metadata Consistency Check Tool

2018-06-19 Thread Hao Hao (Code Review)
Hao Hao has posted comments on this change. ( http://gerrit.cloudera.org:8080/10217 ) Change subject: KUDU-2191: HMS Metadata Consistency Check Tool .. Patch Set 11: (10 comments)

[kudu-CR] KUDU-2191: HMS Metadata Consistency Check Tool

2018-06-19 Thread Hao Hao (Code Review)
Hello Tidy Bot, Dan Burkert, Kudu Jenkins, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/10217 to look at the new patch set (#11). Change subject: KUDU-2191: HMS Metadata Consistency Check Tool

[kudu-CR] KUDU-2480: tsan failure of master-stress-test

2018-06-19 Thread Mike Percy (Code Review)
Mike Percy has posted comments on this change. ( http://gerrit.cloudera.org:8080/10770 ) Change subject: KUDU-2480: tsan failure of master-stress-test .. Patch Set 1: Code-Review+2 I don't see why it would be a problem to

[kudu-CR] KUDU-2480: tsan failure of master-stress-test

2018-06-19 Thread Hao Hao (Code Review)
Hao Hao has posted comments on this change. ( http://gerrit.cloudera.org:8080/10770 ) Change subject: KUDU-2480: tsan failure of master-stress-test .. Patch Set 1: Code-Review+1 -- To view, visit

[kudu-CR] consensus: separate storage from ConsensusMetadata

2018-06-19 Thread Mike Percy (Code Review)
Mike Percy has posted comments on this change. ( http://gerrit.cloudera.org:8080/10649 ) Change subject: consensus: separate storage from ConsensusMetadata .. Patch Set 4: Code-Review+1 (3 comments) Looks pretty good to me.

[kudu-CR] KUDU-2480: tsan failure of master-stress-test

2018-06-19 Thread Dan Burkert (Code Review)
Dan Burkert has posted comments on this change. ( http://gerrit.cloudera.org:8080/10770 ) Change subject: KUDU-2480: tsan failure of master-stress-test .. Patch Set 1: Verified+1 Looks like an unrelated flake (non-HMS test).

[kudu-CR] KUDU-2480: tsan failure of master-stress-test

2018-06-19 Thread Dan Burkert (Code Review)
Dan Burkert has submitted this change and it was merged. ( http://gerrit.cloudera.org:8080/10770 ) Change subject: KUDU-2480: tsan failure of master-stress-test .. KUDU-2480: tsan failure of master-stress-test This is an

[kudu-CR] KUDU-2480: tsan failure of master-stress-test

2018-06-19 Thread Dan Burkert (Code Review)
Dan Burkert has removed a vote on this change. Change subject: KUDU-2480: tsan failure of master-stress-test .. Removed Verified-1 by Kudu Jenkins (120) -- To view, visit http://gerrit.cloudera.org:8080/10770 To unsubscribe,

[kudu-CR] [tools] add 'kudu tablet set attributes' sub-command

2018-06-19 Thread Mike Percy (Code Review)
Mike Percy has posted comments on this change. ( http://gerrit.cloudera.org:8080/10536 ) Change subject: [tools] add 'kudu tablet set_attributes' sub-command .. Patch Set 5: (1 comment) looks good overall

[kudu-CR] KUDU-2479 Flume sink should properly shade its dependencies that are incompatible with flume

2018-06-19 Thread Grant Henke (Code Review)
Grant Henke has posted comments on this change. ( http://gerrit.cloudera.org:8080/10753 ) Change subject: KUDU-2479 Flume sink should properly shade its dependencies that are incompatible with flume .. Patch Set 1:

[kudu-CR] consensus: separate storage from ConsensusMetadata

2018-06-19 Thread Andrew Wong (Code Review)
Andrew Wong has posted comments on this change. ( http://gerrit.cloudera.org:8080/10649 ) Change subject: consensus: separate storage from ConsensusMetadata .. Patch Set 4: (3 comments)

[kudu-CR] KUDU-2479 Flume sink should properly shade its dependencies that are incompatible with flume

2018-06-19 Thread Mike Percy (Code Review)
Mike Percy has posted comments on this change. ( http://gerrit.cloudera.org:8080/10753 ) Change subject: KUDU-2479 Flume sink should properly shade its dependencies that are incompatible with flume .. Patch Set 1:

[kudu-CR] KUDU-2479 Flume sink should properly shade its dependencies that are incompatible with flume

2018-06-19 Thread Mike Percy (Code Review)
Mike Percy has submitted this change and it was merged. ( http://gerrit.cloudera.org:8080/10753 ) Change subject: KUDU-2479 Flume sink should properly shade its dependencies that are incompatible with flume .. KUDU-2479 Flume

[kudu-CR] Kudu Backup/Restore Spark Jobs

2018-06-19 Thread Mike Percy (Code Review)
Mike Percy has posted comments on this change. ( http://gerrit.cloudera.org:8080/10375 ) Change subject: Kudu Backup/Restore Spark Jobs .. Patch Set 18: (6 comments)

[kudu-CR] KUDU-2479 Flume sink should properly shade its dependencies that are incompatible with flume

2018-06-19 Thread Ferenc Szabo (Code Review)
Ferenc Szabo has uploaded this change for review. ( http://gerrit.cloudera.org:8080/10753 Change subject: KUDU-2479 Flume sink should properly shade its dependencies that are incompatible with flume .. KUDU-2479 Flume sink

[kudu-CR] KUDU-2437 Split a tablet into primary key ranges by size

2018-06-19 Thread Yao Xu (Code Review)
Hello Kudu Jenkins, Todd Lipcon, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/10406 to look at the new patch set (#7). Change subject: KUDU-2437 Split a tablet into primary key ranges by size

[kudu-CR] Kudu Backup/Restore Spark Jobs

2018-06-19 Thread Grant Henke (Code Review)
Hello Mike Percy, Kudu Jenkins, Adar Dembo, Todd Lipcon, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/10375 to look at the new patch set (#19). Change subject: Kudu Backup/Restore Spark Jobs

[kudu-CR] KUDU-2437 Split a tablet into primary key ranges by size

2018-06-19 Thread Yao Xu (Code Review)
Hello Kudu Jenkins, Todd Lipcon, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/10406 to look at the new patch set (#5). Change subject: KUDU-2437 Split a tablet into primary key ranges by size

[kudu-CR] KUDU-2437 Split a tablet into primary key ranges by size

2018-06-19 Thread Yao Xu (Code Review)
Hello Kudu Jenkins, Todd Lipcon, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/10406 to look at the new patch set (#8). Change subject: KUDU-2437 Split a tablet into primary key ranges by size

[kudu-CR] KUDU-2437 Split a tablet into primary key ranges by size

2018-06-19 Thread Yao Xu (Code Review)
Hello Kudu Jenkins, Todd Lipcon, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/10406 to look at the new patch set (#9). Change subject: KUDU-2437 Split a tablet into primary key ranges by size

[kudu-CR] KUDU-2438 Class relocation in the maven build should be

2018-06-19 Thread Ferenc Szabo (Code Review)
Ferenc Szabo has abandoned this change. ( http://gerrit.cloudera.org:8080/10524 ) Change subject: KUDU-2438 Class relocation in the maven build should be .. Abandoned see a more specific solution at