You are viewing a plain text version of this content. The canonical link for it is here.
Posted to issues@impala.apache.org by "Michael Ho (JIRA)" <ji...@apache.org> on 2017/10/03 00:38:03 UTC
[jira] [Resolved] (IMPALA-5795)
BackendConfig::LookUpBackendDescriptor() may fail to look up non-executor
coordinator
[ https://issues.apache.org/jira/browse/IMPALA-5795?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ]
Michael Ho resolved IMPALA-5795.
--------------------------------
Resolution: Fixed
Fixed at the following commit:
https://github.com/apache/incubator-impala/commit/7b6ad283f46d31aa953d4d44e142b0bc6b0afbb4
> BackendConfig::LookUpBackendDescriptor() may fail to look up non-executor coordinator
> -------------------------------------------------------------------------------------
>
> Key: IMPALA-5795
> URL: https://issues.apache.org/jira/browse/IMPALA-5795
> Project: IMPALA
> Issue Type: Sub-task
> Components: Distributed Exec
> Reporter: Sailesh Mukil
> Assignee: Michael Ho
> Priority: Critical
> Labels: crash
> Fix For: Impala 2.10.0
>
>
> When we call LookUpBackendDescriptor() with a specific address, we always expect that we've stored the backend list for that host in the backend_map_:
> {code:java}
> /// Return the list of backends on a particular host. The caller must make sure that the
> /// host is actually contained in backend_map_.
> const BackendList& GetBackendListForHost(const IpAddr& ip) const;
> {code}
> However, for configuration with dedicated coordinator, a coordinator can be non-executor so it will not be found in {{backend_map_}}
> If that is the case, on debug binaries, this would trip up this DCHECK in GetBackendListForHost():
> https://github.com/michaelhkw/incubator-impala/blob/krpc/be/src/scheduling/backend-config.cc#L25
> On release binaries, it would hit a SEGFAULT here:
> https://github.com/michaelhkw/incubator-impala/blob/krpc/be/src/scheduling/backend-config.cc#L79
> This bug only exists on KRPC branch.
--
This message was sent by Atlassian JIRA
(v6.4.14#64029)