Clone
Ali Alsuliman <ali.al.solaiman@gmail.com>
committed
on 02 May 19
[NO ISSUE][HYR][RT] Handle calls to close() properly in hash join
- user model changes: no
- storage format changes: no
- interface changes:… Show more
[NO ISSUE][HYR][RT] Handle calls to close() properly in hash join

- user model changes: no

- storage format changes: no

- interface changes: no

Details:

Hash join build activity assumes that upon calling close()

the join state must have been allocated in open() and

therefore, it will directly call "close build" using the join

state. However, if the build activity throws an exception

in open(), then it could happen that the join state has

not been initialized. Calling close() on the build activity

will lead to a NPE since it will use the join state directly.

This patch checks if the join state has been initialized. If

so, it will close the build as usual.

Change-Id: I41fff7deed5e56818bfefb4d36b8018a9f512cbb

Reviewed-on: https://asterix-gerrit.ics.uci.edu/3375

Contrib: Jenkins <jenkins@fulliautomatix.ics.uci.edu>

Tested-by: Jenkins <jenkins@fulliautomatix.ics.uci.edu>

Integration-Tests: Jenkins <jenkins@fulliautomatix.ics.uci.edu>

Reviewed-by: Murtadha Hubail <mhubail@apache.org>

Show less