[kudu-CR] Add table id to AlterTableResponsePB

2016-08-09 Thread Adar Dembo (Code Review)
Adar Dembo has posted comments on this change. Change subject: Add table id to AlterTableResponsePB .. Patch Set 4: Code-Review+2 -- To view, visit http://gerrit.cloudera.org:8080/3859 To unsubscribe, visit

[kudu-CR] Add table id to AlterTableResponsePB

2016-08-09 Thread Kudu Jenkins (Code Review)
Kudu Jenkins has posted comments on this change. Change subject: Add table id to AlterTableResponsePB .. Patch Set 4: Build Started http://104.196.14.100/job/kudu-gerrit/2781/ -- To view, visit

[kudu-CR] Add table id to AlterTableResponsePB

2016-08-09 Thread Dan Burkert (Code Review)
Dan Burkert has posted comments on this change. Change subject: Add table id to AlterTableResponsePB .. Patch Set 3: (1 comment)

[kudu-CR] Add table id to AlterTableResponsePB

2016-08-09 Thread Dan Burkert (Code Review)
Hello Adar Dembo, Kudu Jenkins, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/3859 to look at the new patch set (#4). Change subject: Add table id to AlterTableResponsePB .. Add table

[kudu-CR] Add table id to AlterTableResponsePB

2016-08-09 Thread Adar Dembo (Code Review)
Adar Dembo has posted comments on this change. Change subject: Add table id to AlterTableResponsePB .. Patch Set 3: Code-Review+2 (1 comment)

[kudu-CR] Add table id to AlterTableResponsePB

2016-08-09 Thread Dan Burkert (Code Review)
Dan Burkert has posted comments on this change. Change subject: Add table id to AlterTableResponsePB .. Patch Set 2: (4 comments)

[kudu-CR] Add table id to AlterTableResponsePB

2016-08-09 Thread Kudu Jenkins (Code Review)
Kudu Jenkins has posted comments on this change. Change subject: Add table id to AlterTableResponsePB .. Patch Set 3: Build Started http://104.196.14.100/job/kudu-gerrit/2776/ -- To view, visit

[kudu-CR] Add table id to AlterTableResponsePB

2016-08-09 Thread Dan Burkert (Code Review)
Hello Adar Dembo, Todd Lipcon, Kudu Jenkins, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/3859 to look at the new patch set (#3). Change subject: Add table id to AlterTableResponsePB ..

[kudu-CR] Add table id to AlterTableResponsePB

2016-08-09 Thread Dan Burkert (Code Review)
Hello Kudu Jenkins, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/3859 to look at the new patch set (#2). Change subject: Add table id to AlterTableResponsePB .. Add table id to

[kudu-CR] Add table id to AlterTableResponsePB

2016-08-08 Thread Dan Burkert (Code Review)
Dan Burkert has posted comments on this change. Change subject: Add table id to AlterTableResponsePB .. Patch Set 1: I'm using it in the rust client. There are a couple of cases where the field could be used in the C++ and

[kudu-CR] Add table id to AlterTableResponsePB

2016-08-08 Thread Todd Lipcon (Code Review)
Todd Lipcon has posted comments on this change. Change subject: Add table id to AlterTableResponsePB .. Patch Set 1: you don't want to use this new field in the client? seems strange to add it with no use case -- To view,

[kudu-CR] Add table id to AlterTableResponsePB

2016-08-07 Thread Kudu Jenkins (Code Review)
Kudu Jenkins has posted comments on this change. Change subject: Add table id to AlterTableResponsePB .. Patch Set 1: Build Started http://104.196.14.100/job/kudu-gerrit/2737/ -- To view, visit

[kudu-CR] Add table id to AlterTableResponsePB

2016-08-07 Thread Dan Burkert (Code Review)
Hello Adar Dembo, Todd Lipcon, I'd like you to do a code review. Please visit http://gerrit.cloudera.org:8080/3859 to review the following change. Change subject: Add table id to AlterTableResponsePB .. Add table id to