diff options
author | Norris Lee <norrisl@simba.com> | 2015-02-24 17:37:42 -0800 |
---|---|---|
committer | Parth Chandra <pchandra@maprtech.com> | 2015-03-24 12:12:19 -0700 |
commit | c7cf88391f0bdae891f491c83ab96bfbedac9f97 (patch) | |
tree | 16c41ad19bd507b4abd1fd4e249f6244cd028d93 /contrib/native/client/src | |
parent | bebfe8ae92669dab0891e216d391297d6187a745 (diff) |
DRILL-2307: Detect DNS name resolution failure for better error messages
Diffstat (limited to 'contrib/native/client/src')
-rw-r--r-- | contrib/native/client/src/clientlib/drillClientImpl.cpp | 4 | ||||
-rw-r--r-- | contrib/native/client/src/include/drill/common.hpp | 3 |
2 files changed, 6 insertions, 1 deletions
diff --git a/contrib/native/client/src/clientlib/drillClientImpl.cpp b/contrib/native/client/src/clientlib/drillClientImpl.cpp index d14119d3a..71f960e0f 100644 --- a/contrib/native/client/src/clientlib/drillClientImpl.cpp +++ b/contrib/native/client/src/clientlib/drillClientImpl.cpp @@ -141,6 +141,10 @@ connectionStatus_t DrillClientImpl::connect(const char* host, const char* port){ } }catch(std::exception e){ + // Handle case when the hostname cannot be resolved. "resolve" is hard-coded in boost asio resolver.resolve + if (!strcmp(e.what(), "resolve")) { + return handleConnError(CONN_HOSTNAME_RESOLUTION_ERROR, getMessage(ERR_CONN_EXCEPT, e.what())); + } return handleConnError(CONN_FAILURE, getMessage(ERR_CONN_EXCEPT, e.what())); } diff --git a/contrib/native/client/src/include/drill/common.hpp b/contrib/native/client/src/include/drill/common.hpp index dbfa6fec6..656069271 100644 --- a/contrib/native/client/src/include/drill/common.hpp +++ b/contrib/native/client/src/include/drill/common.hpp @@ -106,7 +106,8 @@ typedef enum{ CONN_HANDSHAKE_FAILED=2, CONN_INVALID_INPUT=3, CONN_ZOOKEEPER_ERROR=4, - CONN_HANDSHAKE_TIMEOUT=5 + CONN_HANDSHAKE_TIMEOUT=5, + CONN_HOSTNAME_RESOLUTION_ERROR=6 } connectionStatus_t; typedef enum{ |