Sailesh Mukil has uploaded a new change for review. http://gerrit.cloudera.org:8080/4306
Change subject: IMPALA-4053: Address follow up comments for IMPALA-3610 ...................................................................... IMPALA-4053: Address follow up comments for IMPALA-3610 This is a follow up to the 'Account memory for global runtime filter aggregation patch'. This follow up contains: * Broadcast filter updates don't TryConsume() memory anymore and are sent immediately. (So the first one will always succeed, subsequent ones will be ignored) * 'disable' filter happens only for error cases. They are discarded otherwise. * Some style changes. Change-Id: I7499558cc9d3622a9f3b68904aa5fe92a113f579 --- M be/src/runtime/coordinator.cc 1 file changed, 62 insertions(+), 42 deletions(-) git pull ssh://gerrit.cloudera.org:29418/Impala-ASF refs/changes/06/4306/1 -- To view, visit http://gerrit.cloudera.org:8080/4306 To unsubscribe, visit http://gerrit.cloudera.org:8080/settings Gerrit-MessageType: newchange Gerrit-Change-Id: I7499558cc9d3622a9f3b68904aa5fe92a113f579 Gerrit-PatchSet: 1 Gerrit-Project: Impala-ASF Gerrit-Branch: master Gerrit-Owner: Sailesh Mukil <[email protected]>
