[kudu-CR] [c++ client] added few deprecation notes

2016-10-03 Thread David Ribeiro Alves (Code Review)
David Ribeiro Alves has posted comments on this change. Change subject: [c++ client] added few deprecation notes .. Patch Set 3: (1 comment) http://gerrit.cloudera.org:8080/#/c/4569/3//COMMIT_MSG Commit Message: Line 18: Davi

[kudu-CR] [c++ client] added few deprecation notes

2016-10-03 Thread Alexey Serbin (Code Review)
Alexey Serbin has posted comments on this change. Change subject: [c++ client] added few deprecation notes .. Patch Set 3: (3 comments) Thank you for review! Will post updated version soon. http://gerrit.cloudera.org:8080/#/

[kudu-CR] [c++ client] added few deprecation notes

2016-10-03 Thread David Ribeiro Alves (Code Review)
David Ribeiro Alves has posted comments on this change. Change subject: [c++ client] added few deprecation notes .. Patch Set 3: (1 comment) http://gerrit.cloudera.org:8080/#/c/4569/3//COMMIT_MSG Commit Message: Line 18: Davi

[kudu-CR] [c++ client] added few deprecation notes

2016-10-03 Thread David Ribeiro Alves (Code Review)
David Ribeiro Alves has posted comments on this change. Change subject: [c++ client] added few deprecation notes .. Patch Set 3: (2 comments) http://gerrit.cloudera.org:8080/#/c/4569/3/src/kudu/client/client.h File src/kudu/cl

[kudu-CR] [c++ client] added few deprecation notes

2016-10-03 Thread Alexey Serbin (Code Review)
Hello Kudu Jenkins, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/4569 to look at the new patch set (#3). Change subject: [c++ client] added few deprecation notes .. [c++ client] added f

[kudu-CR] [c++ client] added few deprecation notes

2016-10-03 Thread David Ribeiro Alves (Code Review)
David Ribeiro Alves has posted comments on this change. Change subject: [c++ client] added few deprecation notes .. Patch Set 2: (1 comment) http://gerrit.cloudera.org:8080/#/c/4569/2/src/kudu/client/client.h File src/kudu/cli

[kudu-CR] [c++ client] added few deprecation notes

2016-09-30 Thread Alexey Serbin (Code Review)
Alexey Serbin has posted comments on this change. Change subject: [c++ client] added few deprecation notes .. Patch Set 2: (1 comment) > (1 comment) > > I think that the get/set latest timestamp APIs can be marked with > "

[kudu-CR] [c++ client] added few deprecation notes

2016-09-30 Thread David Ribeiro Alves (Code Review)
David Ribeiro Alves has posted comments on this change. Change subject: [c++ client] added few deprecation notes .. Patch Set 2: (1 comment) I think that the get/set latest timestamp APIs can be marked with "unstable" or "exp

[kudu-CR] [c++ client] added few deprecation notes

2016-09-29 Thread Alexey Serbin (Code Review)
Hello Kudu Jenkins, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/4569 to look at the new patch set (#2). Change subject: [c++ client] added few deprecation notes .. [c++ client] added f

[kudu-CR] [c++ client] added few deprecation notes

2016-09-29 Thread Alexey Serbin (Code Review)
Alexey Serbin has posted comments on this change. Change subject: [c++ client] added few deprecation notes .. Patch Set 1: (1 comment) > (1 comment) > > Shouldn't we wait until the new API lands before deprecating the > ol

[kudu-CR] [c++ client] added few deprecation notes

2016-09-29 Thread Adar Dembo (Code Review)
Adar Dembo has posted comments on this change. Change subject: [c++ client] added few deprecation notes .. Patch Set 1: (1 comment) Shouldn't we wait until the new API lands before deprecating the old one though? http://gerri

[kudu-CR] [c++ client] added few deprecation notes

2016-09-29 Thread Alexey Serbin (Code Review)
Alexey Serbin has uploaded a new change for review. http://gerrit.cloudera.org:8080/4569 Change subject: [c++ client] added few deprecation notes .. [c++ client] added few deprecation notes KUDU-1661 Mark kudu::client::KuduCl