Impala Public Jenkins has submitted this change and it was merged. ( http://gerrit.cloudera.org:8080/11423 )
Change subject: IMPALA-7348: fix race in KuduUtil.getKuduClient ...................................................................... IMPALA-7348: fix race in KuduUtil.getKuduClient We've seen a symptom in our builds where tests will fail due to missing cardinality info for Kudu tables. This is being caused by a NullPointerException during planning when running the 'compute stats' phase of data loading. The NullPointerException occurs because KuduUtil.getKuduClient is not thread safe and, depending on interleaving, may return NULL if there are two concurrent queries accessing Kudu tables right after cluster startup when the KuduClient map has not been populated yet. The solution is to make KuduUtil.getKuduClient thread safe by switching the KuduClient map to be a ConcurrentHashMap. Change-Id: Iec50e9b592593cea5418e7a074bdda065184086f Reviewed-on: http://gerrit.cloudera.org:8080/11423 Reviewed-by: Impala Public Jenkins <[email protected]> Tested-by: Impala Public Jenkins <[email protected]> --- M fe/src/main/java/org/apache/impala/util/KuduUtil.java 1 file changed, 8 insertions(+), 5 deletions(-) Approvals: Impala Public Jenkins: Looks good to me, approved; Verified -- To view, visit http://gerrit.cloudera.org:8080/11423 To unsubscribe, visit http://gerrit.cloudera.org:8080/settings Gerrit-Project: Impala-ASF Gerrit-Branch: master Gerrit-MessageType: merged Gerrit-Change-Id: Iec50e9b592593cea5418e7a074bdda065184086f Gerrit-Change-Number: 11423 Gerrit-PatchSet: 3 Gerrit-Owner: Thomas Marshall <[email protected]> Gerrit-Reviewer: Dan Hecht <[email protected]> Gerrit-Reviewer: Impala Public Jenkins <[email protected]> Gerrit-Reviewer: Philip Zeyliger <[email protected]> Gerrit-Reviewer: Vuk Ercegovac <[email protected]>
