Matthew Jacobs has uploaded a new change for review.

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

Change subject: IMPALA-3500: GetEffectiveUser() segfault using -enable_rm
......................................................................

IMPALA-3500: GetEffectiveUser() segfault using -enable_rm

A previous change to improve admission control broke Llama
integration. The Coordinator's runtime_state has a partial
TQueryCtx, it doesn't have the TSessionState which is used
by RuntimeState::effective_user(), so calling that in the
SimpleScheduler caused the process to crash.

This is easily fixed by using the TSessionState from the
schedule (which is what the Admission Control code was
doing).

Llama integration is no longer supported, so this was only
tested manually.

Change-Id: Ia04b6aec35ae794d7062fd32104f3964f397f00c
---
M be/src/scheduling/simple-scheduler.cc
1 file changed, 1 insertion(+), 1 deletion(-)


  git pull ssh://gerrit.cloudera.org:29418/Impala refs/changes/16/3016/1
-- 
To view, visit http://gerrit.cloudera.org:8080/3016
To unsubscribe, visit http://gerrit.cloudera.org:8080/settings

Gerrit-MessageType: newchange
Gerrit-Change-Id: Ia04b6aec35ae794d7062fd32104f3964f397f00c
Gerrit-PatchSet: 1
Gerrit-Project: Impala
Gerrit-Branch: cdh5-trunk
Gerrit-Owner: Matthew Jacobs <[email protected]>

Reply via email to