impala-dev mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From "Dimitris Tsirogiannis (Code Review)" <ger...@cloudera.org>
Subject [Impala-CR](cdh5-trunk) IMPALA-3502: Fix race in the coordinator while updating filter routing table
Date Thu, 12 May 2016 07:06:09 GMT
Hello Dan Hecht, Tim Armstrong,

I'd like you to reexamine a change.  Please visit

    http://gerrit.cloudera.org:8080/3018

to look at the new patch set (#3).

Change subject: IMPALA-3502: Fix race in the coordinator while updating filter routing table
......................................................................

IMPALA-3502: Fix race in the coordinator while updating filter routing
table

This commit fixes an issue where a fragment may send an
UpdateFilter message to the coordinator while the latter is still
updating the runtime filter routing table. The fix is to decouple the
update of the filter routing table from starting the fragments. With
this fix, the coordinator will finish populating the filter routing
table before it starts any remote fragments.

Change-Id: Iecc737106fd38aa4af0c72959a577adfb413728d
---
M be/src/runtime/coordinator.cc
1 file changed, 33 insertions(+), 17 deletions(-)


  git pull ssh://gerrit.cloudera.org:29418/Impala refs/changes/18/3018/3
-- 
To view, visit http://gerrit.cloudera.org:8080/3018
To unsubscribe, visit http://gerrit.cloudera.org:8080/settings

Gerrit-MessageType: newpatchset
Gerrit-Change-Id: Iecc737106fd38aa4af0c72959a577adfb413728d
Gerrit-PatchSet: 3
Gerrit-Project: Impala
Gerrit-Branch: cdh5-trunk
Gerrit-Owner: Dimitris Tsirogiannis <dtsirogiannis@cloudera.com>
Gerrit-Reviewer: Dan Hecht <dhecht@cloudera.com>
Gerrit-Reviewer: Dimitris Tsirogiannis <dtsirogiannis@cloudera.com>
Gerrit-Reviewer: Tim Armstrong <tarmstrong@cloudera.com>

Mime
View raw message