[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:

[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.

[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:

[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

[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

[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 >

[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?

[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