[kudu-CR] c++ client: remove unnecessary code

2016-08-03 Thread Todd Lipcon (Code Review)
Todd Lipcon has posted comments on this change. Change subject: c++ client: remove unnecessary code .. Patch Set 3: Code-Review+2 -- To view, visit http://gerrit.cloudera.org:8080/3809 To unsubscribe, visit

[kudu-CR] c++ client: remove unnecessary code

2016-08-03 Thread Todd Lipcon (Code Review)
Todd Lipcon has submitted this change and it was merged. Change subject: c++ client: remove unnecessary code .. c++ client: remove unnecessary code 1. GetTableSchema() was implemented using its own RPC instead of a much

[kudu-CR] c++ client: remove unnecessary code

2016-08-02 Thread Adar Dembo (Code Review)
Adar Dembo has posted comments on this change. Change subject: c++ client: remove unnecessary code .. Patch Set 3: (2 comments) http://gerrit.cloudera.org:8080/#/c/3809/3/src/kudu/client/client-internal.cc File

[kudu-CR] c++ client: remove unnecessary code

2016-08-02 Thread Alexey Serbin (Code Review)
Alexey Serbin has posted comments on this change. Change subject: c++ client: remove unnecessary code .. Patch Set 3: Code-Review+1 (2 comments) http://gerrit.cloudera.org:8080/#/c/3809/3/src/kudu/client/client-internal.cc

[kudu-CR] c++ client: remove unnecessary code

2016-08-02 Thread Adar Dembo (Code Review)
Hello Todd Lipcon, Kudu Jenkins, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/3809 to look at the new patch set (#3). Change subject: c++ client: remove unnecessary code .. c++ client:

[kudu-CR] c++ client: remove unnecessary code

2016-08-02 Thread Kudu Jenkins (Code Review)
Kudu Jenkins has posted comments on this change. Change subject: c++ client: remove unnecessary code .. Patch Set 3: Build Started http://104.196.14.100/job/kudu-gerrit/2703/ -- To view, visit

[kudu-CR] c++ client: remove unnecessary code

2016-08-02 Thread Todd Lipcon (Code Review)
Todd Lipcon has posted comments on this change. Change subject: c++ client: remove unnecessary code .. Patch Set 2: Code-Review+1 -- To view, visit http://gerrit.cloudera.org:8080/3809 To unsubscribe, visit

[kudu-CR] c++ client: remove unnecessary code

2016-08-02 Thread Dan Burkert (Code Review)
Dan Burkert has posted comments on this change. Change subject: c++ client: remove unnecessary code .. Patch Set 2: (1 comment) http://gerrit.cloudera.org:8080/#/c/3809/2/src/kudu/client/client-internal.cc File

[kudu-CR] c++ client: remove unnecessary code

2016-07-28 Thread Kudu Jenkins (Code Review)
Kudu Jenkins has posted comments on this change. Change subject: c++ client: remove unnecessary code .. Patch Set 2: Build Started http://104.196.14.100/job/kudu-gerrit/2682/ -- To view, visit