Thomas Marshall has uploaded this change for review. ( 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 --- M fe/src/main/java/org/apache/impala/util/KuduUtil.java 1 file changed, 8 insertions(+), 5 deletions(-) git pull ssh://gerrit.cloudera.org:29418/Impala-ASF refs/changes/23/11423/1 -- 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: newchange Gerrit-Change-Id: Iec50e9b592593cea5418e7a074bdda065184086f Gerrit-Change-Number: 11423 Gerrit-PatchSet: 1 Gerrit-Owner: Thomas Marshall <[email protected]>
