Tim Armstrong has submitted this change and it was merged. Change subject: IMPALA-2982: lazy initialization of HBase connection ......................................................................
IMPALA-2982: lazy initialization of HBase connection We shouldn't initialize a HBase connection if Impala will be run on a cluster without Hbase. This started to bite us with a change to ZK or HBase that resulted in periodic log messages when a bad HBase connection was created. This patch changes the logic to only initialize the question on demand, e.g. when a query is run against a HBase table. Change-Id: I405f5cbfc62dd4a4aeb6f4019cac358376478884 Reviewed-on: http://gerrit.cloudera.org:8080/2345 Tested-by: Internal Jenkins Reviewed-by: Tim Armstrong <[email protected]> --- M be/src/runtime/hbase-table-factory.cc M be/src/runtime/hbase-table-factory.h M be/src/service/impalad-main.cc M be/src/testutil/mini-impala-cluster.cc 4 files changed, 40 insertions(+), 32 deletions(-) Approvals: Internal Jenkins: Verified Tim Armstrong: Looks good to me, approved -- To view, visit http://gerrit.cloudera.org:8080/2345 To unsubscribe, visit http://gerrit.cloudera.org:8080/settings Gerrit-MessageType: merged Gerrit-Change-Id: I405f5cbfc62dd4a4aeb6f4019cac358376478884 Gerrit-PatchSet: 8 Gerrit-Project: Impala Gerrit-Branch: cdh5-2.5.0_5.7.0 Gerrit-Owner: Tim Armstrong <[email protected]> Gerrit-Reviewer: Internal Jenkins Gerrit-Reviewer: Marcel Kornacker <[email protected]> Gerrit-Reviewer: Sailesh Mukil <[email protected]> Gerrit-Reviewer: Silvius Rus <[email protected]> Gerrit-Reviewer: Tim Armstrong <[email protected]>
