Henry Robinson has uploaded a new change for review. http://gerrit.cloudera.org:8080/7457
Change subject: IMPALA-4905: Don't send empty insert status to coordinator ...................................................................... IMPALA-4905: Don't send empty insert status to coordinator If the coordinator, in UpdateBackendExecStatus(), receives a report that includes a TInsertExecStatus, it will call UpdateInsertExecStatus() which takes the coordinator-wide lock. Avoid doing this for fragment instances that would only send an empty TInsertExecStatus (including instances that belong to SELECT queries, not DML queries). Future changes should fix the locking around the UpdateBackendExecStatus() path to remove dependencies on Coordinator::lock_, but this fix is simple and addresses one point of needless contention. Change-Id: I314dd8d96922d273c6329266970d249ec8c5c608 --- M be/src/runtime/query-state.cc 1 file changed, 10 insertions(+), 9 deletions(-) git pull ssh://gerrit.cloudera.org:29418/Impala-ASF refs/changes/57/7457/1 -- To view, visit http://gerrit.cloudera.org:8080/7457 To unsubscribe, visit http://gerrit.cloudera.org:8080/settings Gerrit-MessageType: newchange Gerrit-Change-Id: I314dd8d96922d273c6329266970d249ec8c5c608 Gerrit-PatchSet: 1 Gerrit-Project: Impala-ASF Gerrit-Branch: master Gerrit-Owner: Henry Robinson <he...@cloudera.com>
