[IMPALA-4540] Scheduler calls function in DCHECK, leading to crash

• Type: Bug
• Status: Resolved
• Priority: Blocker
• Resolution: Fixed
• Affects Version/s: Impala 2.8.0
• Fix Version/s: Impala 2.8.0
• Component/s: Backend
• Labels: broken-build crash scheduler
• Target Version: Impala 2.8.0
• Assignee: Lars Volker
• Reporter: Lars Volker
• Created: 24/Nov/16 20:59
• Updated: 30/Nov/16 13:26
• Resolved: 25/Nov/16 17:55

Description

The fix for IMPALA-4494 wrapped the call to backend_config_.LookUpBackendIp in simple-scheduler.cc:922 in a DCHECK, effectively disabling it in release builds. This leads to a crash when backend_ip is empty.

921   IpAddr backend_ip;
922   DCHECK(backend_config_.LookUpBackendIp(backend.address.hostname, &backend_ip));
923   DCHECK(!backend_ip.empty());
924   assignment_heap_.InsertOrUpdate(backend_ip, scan_range_length,
925       GetBackendRank(backend_ip));

Issue Links

• blocks: IMPALA-4494 Crash in SimpleScheduler when restarting under load