Author: Dan Hecht <email@example.com>
Date: Thu Apr 27 16:03:31 2017 -0700
IMPALA-5252: Fix crash in HiveUdfCall::GetStringVal() when mem_limit exceeded
We need to check for AllocateLocal() returning NULL. CopyFrom() takes
care of that for us. Also adjust a few other places in the code base
that didn't have the check.
The new test reproduces the crash, but in order to get this test file to
execute, I had to move the xfail to be a function decorator. Apparently
xfail as a statement causes the test to not run at all. We should run
all of these queries even if they are non-determistic to at least verify
that impalad does not crash.
Reviewed-by: Dan Hecht <firstname.lastname@example.org>
Tested-by: Impala Public Jenkins