Impala Public Jenkins has submitted this change and it was merged. ( http://gerrit.cloudera.org:8080/11096 )
Change subject: IMPALA-7376: DCHECK hit if a fragment instance fails to initialize the filter bank ...................................................................... IMPALA-7376: DCHECK hit if a fragment instance fails to initialize the filter bank While Prepare()-ing a fragment instance, if we fail to initialize the runtime filter bank, we will exit FIS::Prepare() without acquiring a thread token (AcquireThreadToken()): FIS::Finalize() is called always regardless of whether the fragment instance succeeded or failed. And FIS::Finalize() tries to ReleaseThreadToken() even though it might not have gotten acquired, causing a DCHECK to be hit. This patch fixes it by making sure that no failable code is run before acquiring the thread token, thereby ensuring that the thread token is always acquired and thus avoiding the above crash. A test is added to confirm this as well. This test crashes without the code changes in this patch. Change-Id: I1d6e7afc18fe2f0e1e29d2bd8a5f804a78f7043a Reviewed-on: http://gerrit.cloudera.org:8080/11096 Reviewed-by: Sailesh Mukil <sail...@cloudera.com> Tested-by: Impala Public Jenkins <impala-public-jenk...@cloudera.com> --- M be/src/runtime/fragment-instance-state.cc M tests/failure/test_failpoints.py 2 files changed, 16 insertions(+), 2 deletions(-) Approvals: Sailesh Mukil: Looks good to me, approved Impala Public Jenkins: Verified -- To view, visit http://gerrit.cloudera.org:8080/11096 To unsubscribe, visit http://gerrit.cloudera.org:8080/settings Gerrit-Project: Impala-ASF Gerrit-Branch: master Gerrit-MessageType: merged Gerrit-Change-Id: I1d6e7afc18fe2f0e1e29d2bd8a5f804a78f7043a Gerrit-Change-Number: 11096 Gerrit-PatchSet: 4 Gerrit-Owner: Sailesh Mukil <sail...@cloudera.com> Gerrit-Reviewer: Impala Public Jenkins <impala-public-jenk...@cloudera.com> Gerrit-Reviewer: Michael Ho <k...@cloudera.com> Gerrit-Reviewer: Sailesh Mukil <sail...@cloudera.com>
