Todd Lipcon has submitted this change and it was merged.

Change subject: C++ client: fix on KuduSession::GetPendingErrors()
......................................................................


C++ client: fix on KuduSession::GetPendingErrors()

If re-using the same std::vector instance as a placeholder
for repeated calls to KuduSession::GetPendingErrors(), irrelevant
error information can be pushed into the session error container.
Clean the placeholder container before swapping it with current
error container of the KuduSession.

Also, make it possible to pass NULL for the 'overflowed' out parameter.

Change-Id: Iafee46a6135ac2096d1f8b690da5b084a15eb73e
Reviewed-on: http://gerrit.cloudera.org:8080/3677
Reviewed-by: Todd Lipcon <t...@apache.org>
Tested-by: Kudu Jenkins
---
M src/kudu/client/error_collector.cc
1 file changed, 7 insertions(+), 2 deletions(-)

Approvals:
  Todd Lipcon: Looks good to me, approved
  Kudu Jenkins: Verified



-- 
To view, visit http://gerrit.cloudera.org:8080/3677
To unsubscribe, visit http://gerrit.cloudera.org:8080/settings

Gerrit-MessageType: merged
Gerrit-Change-Id: Iafee46a6135ac2096d1f8b690da5b084a15eb73e
Gerrit-PatchSet: 2
Gerrit-Project: kudu
Gerrit-Branch: master
Gerrit-Owner: Alexey Serbin <aser...@cloudera.com>
Gerrit-Reviewer: Kudu Jenkins
Gerrit-Reviewer: Todd Lipcon <t...@apache.org>

Reply via email to