Will Berkeley has posted comments on this change.

Change subject: KUDU-2005: actionable error messages from webserver
......................................................................


Patch Set 1:

(1 comment)

http://gerrit.cloudera.org:8080/#/c/6848/1/src/kudu/server/webserver.cc
File src/kudu/server/webserver.cc:

PS1, Line 246: TryRunLsof(addr);
Put this after the error message below, since the below puts it in context.

It'd also be nice to do this lsof step only when we get an address-in-use 
error, or have good reason to suspect that is the error, rather than for any 
failure.


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

Gerrit-MessageType: comment
Gerrit-Change-Id: Ia2315f7ee88c8835a36e5174cb25132967429a77
Gerrit-PatchSet: 1
Gerrit-Project: kudu
Gerrit-Branch: master
Gerrit-Owner: Alexey Serbin <aser...@cloudera.com>
Gerrit-Reviewer: Adar Dembo <a...@cloudera.com>
Gerrit-Reviewer: Dan Burkert <danburk...@apache.org>
Gerrit-Reviewer: Kudu Jenkins
Gerrit-Reviewer: Tidy Bot
Gerrit-Reviewer: Will Berkeley <wdberke...@gmail.com>
Gerrit-HasComments: Yes

Reply via email to