Tim Armstrong has uploaded a new change for review.

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

Change subject: IMPALA-5291: avoid calling global destructors in statestore-test
......................................................................

IMPALA-5291: avoid calling global destructors in statestore-test

The workaround implemented by this patch is to exit the process using
_exit(), which does not run global destructors.

The "proper" solution would be either to change Thrift so that the
destructors are not run unnecessarily on process teardown, or to extend
the statestore services and clients to allow stopping them. It seems the
complexity of neither is justified.

Testing:
Was able to reproduce the bug by running statestore-test in a loop.
After the fix it was not reproducible.

Change-Id: Ic185825cf262311bdc05784ebd541cf53a13cdd6
---
M be/src/statestore/statestore-test.cc
1 file changed, 9 insertions(+), 1 deletion(-)


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

Gerrit-MessageType: newchange
Gerrit-Change-Id: Ic185825cf262311bdc05784ebd541cf53a13cdd6
Gerrit-PatchSet: 1
Gerrit-Project: Impala-ASF
Gerrit-Branch: master
Gerrit-Owner: Tim Armstrong <[email protected]>

Reply via email to