Michael Ho has posted comments on this change. ( 
http://gerrit.cloudera.org:8080/8751 )

Change subject: IMPALA-6238: Enhance TErrorCode::DATASTREAM_SENDER_TIMEOUT 
message
......................................................................


Patch Set 2:

(1 comment)

http://gerrit.cloudera.org:8080/#/c/8751/1/be/src/runtime/krpc-data-stream-mgr.cc
File be/src/runtime/krpc-data-stream-mgr.cc:

http://gerrit.cloudera.org:8080/#/c/8751/1/be/src/runtime/krpc-data-stream-mgr.cc@338
PS1, Line 338: ErrorMsg msg(TErrorCode::DATASTREAM_SENDER_TIMEOUT, remote_add
> how about Substitute()?
Done. Removed the parenthesis and replaced with a blank space.



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

Gerrit-Project: Impala-ASF
Gerrit-Branch: master
Gerrit-MessageType: comment
Gerrit-Change-Id: Ie3e83773fe6feda057296e7d5544690aa9271fa0
Gerrit-Change-Number: 8751
Gerrit-PatchSet: 2
Gerrit-Owner: Michael Ho <k...@cloudera.com>
Gerrit-Reviewer: Dan Hecht <dhe...@cloudera.com>
Gerrit-Reviewer: Michael Ho <k...@cloudera.com>
Gerrit-Reviewer: Mostafa Mokhtar <mmokh...@cloudera.com>
Gerrit-Comment-Date: Mon, 04 Dec 2017 22:33:39 +0000
Gerrit-HasComments: Yes

Reply via email to