You are viewing a plain text version of this content. The canonical link for it is here.
Posted to dev@impala.apache.org by "Thomas Tauber-Marshall (Code Review)" <ge...@cloudera.org> on 2016/05/13 22:25:13 UTC

[hs2client-CR] Fix hs2client EXPECT_OK macro.

Thomas Tauber-Marshall has uploaded a new change for review.

  http://gerrit.cloudera.org:8080/3068

Change subject: Fix hs2client EXPECT_OK macro.
......................................................................

Fix hs2client EXPECT_OK macro.

The EXPECT_OK macro used in tests references its parameters twice,
leading to duplicate execution. Fix this by updating it to store
the parameter's value to a local veriable so it is only referenced
once.

Change-Id: If44402809dc4e76ec40597caa65e40a202b241c8
---
M src/hs2client/test-util.h
1 file changed, 5 insertions(+), 1 deletion(-)


  git pull ssh://gerrit.cloudera.org:29418/hs2client refs/changes/68/3068/1
-- 
To view, visit http://gerrit.cloudera.org:8080/3068
To unsubscribe, visit http://gerrit.cloudera.org:8080/settings

Gerrit-MessageType: newchange
Gerrit-Change-Id: If44402809dc4e76ec40597caa65e40a202b241c8
Gerrit-PatchSet: 1
Gerrit-Project: hs2client
Gerrit-Branch: master
Gerrit-Owner: Thomas Tauber-Marshall <tm...@cloudera.com>