tree c3b697dd2b943bc6c7937d8113172848e27840d3
parent e8c6adcc2be8d4134b9329c766186f1d4632aa1f
author Thomas Tauber-Marshall <tmarshall@cloudera.com> 1552509528 +0000
committer Gabor Kaszab <gaborkaszab@cloudera.com> 1552899521 +0100

IMPALA-8299: Fix crash in GroupingAggregator on uninited hash table

If HashTable::Init() fails, in some cases we may try to call
HashTable::Close() on the table in GroupingAggregator::Close(), which
can cause a crash.

The solution is to set the hash table to a nullptr if Init() fails.

I also verified that the other use of HashTable, PhjBuilder, handles
this case correctly.

Testing:
- Manually verified that the crash no longer occurs if HashTable::Init
  fails.

Change-Id: I381657c43f93eb4871b93d54532cb886198fd0b2
Reviewed-on: http://gerrit.cloudera.org:8080/12746
Reviewed-by: Impala Public Jenkins <impala-public-jenkins@cloudera.com>
Tested-by: Impala Public Jenkins <impala-public-jenkins@cloudera.com>