[kudu-CR] KUDU-1807 (part 3): remove GetTableSchema.create table done

2017-09-18 Thread David Ribeiro Alves (Code Review)
David Ribeiro Alves has posted comments on this change. Change subject: KUDU-1807 (part 3): remove GetTableSchema.create_table_done .. Patch Set 5: Code-Review+2 -- To view, visit http://gerrit.cloudera.org:8080/8045 To

[kudu-CR] KUDU-1807 (part 3): remove GetTableSchema.create table done

2017-09-18 Thread Dan Burkert (Code Review)
Dan Burkert has posted comments on this change. Change subject: KUDU-1807 (part 3): remove GetTableSchema.create_table_done .. Patch Set 5: Code-Review+2 -- To view, visit http://gerrit.cloudera.org:8080/8045 To unsubscribe,

[kudu-CR] KUDU-1807 (part 3): remove GetTableSchema.create table done

2017-09-18 Thread David Ribeiro Alves (Code Review)
David Ribeiro Alves has posted comments on this change. Change subject: KUDU-1807 (part 3): remove GetTableSchema.create_table_done .. Patch Set 4: Code-Review+2 -- To view, visit http://gerrit.cloudera.org:8080/8045 To

[kudu-CR] KUDU-1807 (part 3): remove GetTableSchema.create table done

2017-09-17 Thread Adar Dembo (Code Review)
Adar Dembo has posted comments on this change. Change subject: KUDU-1807 (part 3): remove GetTableSchema.create_table_done .. Patch Set 4: Verified+1 All of the build failures were due to the build hosts running out of disk

[kudu-CR] KUDU-1807 (part 3): remove GetTableSchema.create table done

2017-09-15 Thread David Ribeiro Alves (Code Review)
David Ribeiro Alves has posted comments on this change. Change subject: KUDU-1807 (part 3): remove GetTableSchema.create_table_done .. Patch Set 3: Code-Review+2 -- To view, visit http://gerrit.cloudera.org:8080/8045 To

[kudu-CR] KUDU-1807 (part 3): remove GetTableSchema.create table done

2017-09-15 Thread Adar Dembo (Code Review)
Adar Dembo has posted comments on this change. Change subject: KUDU-1807 (part 3): remove GetTableSchema.create_table_done .. Patch Set 3: Verified+1 > are the build failures unrelated? (there are a few of them) The flex

[kudu-CR] KUDU-1807 (part 3): remove GetTableSchema.create table done

2017-09-15 Thread David Ribeiro Alves (Code Review)
David Ribeiro Alves has posted comments on this change. Change subject: KUDU-1807 (part 3): remove GetTableSchema.create_table_done .. Patch Set 3: are the build failures unrelated? (there are a few of them) -- To view,

[kudu-CR] KUDU-1807 (part 3): remove GetTableSchema.create table done

2017-09-15 Thread Adar Dembo (Code Review)
Hello David Ribeiro Alves, Todd Lipcon, Kudu Jenkins, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/8045 to look at the new patch set (#3). Change subject: KUDU-1807 (part 3): remove GetTableSchema.create_table_done

[kudu-CR] KUDU-1807 (part 3): remove GetTableSchema.create table done

2017-09-14 Thread Todd Lipcon (Code Review)
Todd Lipcon has posted comments on this change. Change subject: KUDU-1807 (part 3): remove GetTableSchema.create_table_done .. Patch Set 2: Code-Review+1 -- To view, visit http://gerrit.cloudera.org:8080/8045 To unsubscribe,

[kudu-CR] KUDU-1807 (part 3): remove GetTableSchema.create table done

2017-09-14 Thread David Ribeiro Alves (Code Review)
David Ribeiro Alves has posted comments on this change. Change subject: KUDU-1807 (part 3): remove GetTableSchema.create_table_done .. Patch Set 2: actually nm. my concern was that we might use done() (which is false by

[kudu-CR] KUDU-1807 (part 3): remove GetTableSchema.create table done

2017-09-14 Thread David Ribeiro Alves (Code Review)
David Ribeiro Alves has posted comments on this change. Change subject: KUDU-1807 (part 3): remove GetTableSchema.create_table_done .. Patch Set 2: Code-Review+2 -- To view, visit http://gerrit.cloudera.org:8080/8045 To

[kudu-CR] KUDU-1807 (part 3): remove GetTableSchema.create table done

2017-09-14 Thread David Ribeiro Alves (Code Review)
David Ribeiro Alves has posted comments on this change. Change subject: KUDU-1807 (part 3): remove GetTableSchema.create_table_done .. Patch Set 2: lgtm, just want to make sure that previously the java client (and elsewhere)

[kudu-CR] KUDU-1807 (part 3): remove GetTableSchema.create table done

2017-09-12 Thread Adar Dembo (Code Review)
Adar Dembo has posted comments on this change. Change subject: KUDU-1807 (part 3): remove GetTableSchema.create_table_done .. Patch Set 1: Verified+1 NTP synchronization error in one of the dist-test slaves, apparently. --

[kudu-CR] KUDU-1807 (part 3): remove GetTableSchema.create table done

2017-09-12 Thread Adar Dembo (Code Review)
Hello Dan Burkert, Todd Lipcon, I'd like you to do a code review. Please visit http://gerrit.cloudera.org:8080/8045 to review the following change. Change subject: KUDU-1807 (part 3): remove GetTableSchema.create_table_done