[kudu-CR] [c++ client] implemented session operations stats

2017-01-10 Thread Alexey Serbin (Code Review)
Alexey Serbin has abandoned this change. Change subject: [c++ client] implemented session operations stats .. Abandoned It seems this is not needed in this form -- Impala client could maintain these metrics itself, if needed.

[kudu-CR] [c++ client] implemented session operations stats

2016-11-08 Thread David Ribeiro Alves (Code Review)
David Ribeiro Alves has posted comments on this change. Change subject: [c++ client] implemented session operations stats .. Patch Set 3: (7 comments) http://gerrit.cloudera.org:8080/#/c/4974/3//COMMIT_MSG Commit Message:

[kudu-CR] [c++ client] implemented session operations stats

2016-11-08 Thread Alexey Serbin (Code Review)
Alexey Serbin has posted comments on this change. Change subject: [c++ client] implemented session operations stats .. Patch Set 3: As it turned out, this change is not very useful if not adding additional fine-grained stats

[kudu-CR] [c++ client] implemented session operations stats

2016-11-08 Thread Alexey Serbin (Code Review)
Hello Dinesh Bhat, Kudu Jenkins, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/4974 to look at the new patch set (#3). Change subject: [c++ client] implemented session operations stats

[kudu-CR] [c++ client] implemented session operations stats

2016-11-07 Thread Alexey Serbin (Code Review)
Alexey Serbin has posted comments on this change. Change subject: [c++ client] implemented session operations stats .. Patch Set 2: > Alexey, thanks for doing this. As we discussed, one of the issues > Impala faces is that

[kudu-CR] [c++ client] implemented session operations stats

2016-11-07 Thread Alexey Serbin (Code Review)
Alexey Serbin has posted comments on this change. Change subject: [c++ client] implemented session operations stats .. Patch Set 2: (3 comments) http://gerrit.cloudera.org:8080/#/c/4974/2//COMMIT_MSG Commit Message: PS2,

[kudu-CR] [c++ client] implemented session operations stats

2016-11-07 Thread Matthew Jacobs (Code Review)
Matthew Jacobs has posted comments on this change. Change subject: [c++ client] implemented session operations stats .. Patch Set 2: Alexey, thanks for doing this. As we discussed, one of the issues Impala faces is that the

[kudu-CR] [c++ client] implemented session operations stats

2016-11-07 Thread Jean-Daniel Cryans (Code Review)
Jean-Daniel Cryans has posted comments on this change. Change subject: [c++ client] implemented session operations stats .. Patch Set 2: (3 comments) http://gerrit.cloudera.org:8080/#/c/4974/2//COMMIT_MSG Commit Message:

[kudu-CR] [c++ client] implemented session operations stats

2016-11-07 Thread Alexey Serbin (Code Review)
Alexey Serbin has posted comments on this change. Change subject: [c++ client] implemented session operations stats .. Patch Set 2: (3 comments) http://gerrit.cloudera.org:8080/#/c/4974/2//COMMIT_MSG Commit Message: PS2,

[kudu-CR] [c++ client] implemented session operations stats

2016-11-06 Thread Alexey Serbin (Code Review)
Hello Kudu Jenkins, I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/4974 to look at the new patch set (#2). Change subject: [c++ client] implemented session operations stats .. [c++

[kudu-CR] [c++ client] implemented session operations stats

2016-11-06 Thread Alexey Serbin (Code Review)
Alexey Serbin has uploaded a new change for review. http://gerrit.cloudera.org:8080/4974 Change subject: [c++ client] implemented session operations stats .. [c++ client] implemented session operations stats The C++ client