[ https://issues.apache.org/jira/browse/IMPALA-5119?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ]
Michael Ho reassigned IMPALA-5119: ---------------------------------- Assignee: Dan Hecht (was: Joe McDonnell) > Don't call CancelInternal() from Coordinator::UpdateFragmentExecStatus() > ------------------------------------------------------------------------ > > Key: IMPALA-5119 > URL: https://issues.apache.org/jira/browse/IMPALA-5119 > Project: IMPALA > Issue Type: Sub-task > Components: Distributed Exec > Affects Versions: Impala 2.9.0 > Reporter: Henry Robinson > Assignee: Dan Hecht > Priority: Major > > If it reports a bad status, {{UpdateFragmentExecStatus()}} will call > {{UpdateStatus()}}, which takes {{Coordinator::lock_}} and then calls > {{Cancel()}}. That method issues one RPC per fragment instance. > In KRPC, doing so much work from {{UpdateFragmentExecStatus()}} - which is an > RPC handler - is a bad idea, even if the RPCs are issued asynchronously. > There's still some serialization cost. > It's also a bad idea to do all this work while holding {{lock_}}. We should > address both of these to ensure scalability of the cancellation path. -- This message was sent by Atlassian JIRA (v7.6.3#76005) --------------------------------------------------------------------- To unsubscribe, e-mail: issues-all-unsubscr...@impala.apache.org For additional commands, e-mail: issues-all-h...@impala.apache.org