Impala Public Jenkins has submitted this change and it was merged. ( http://gerrit.cloudera.org:8080/16341 )
Change subject: IMPALA-7714: try to avoid be test crash in statestore ...................................................................... IMPALA-7714: try to avoid be test crash in statestore We didn't get to a clear root cause for this, so I'm going to try two things. First, under the theory that the problem is somehow the destruction of the strings, convert them to char char* which does not require destruction on process teardown. Second, add some logging if the map lookup fails so we can better understand what may have happened. Change-Id: Id4363a93addb8a808d292906cac44ebd25c16889 Reviewed-on: http://gerrit.cloudera.org:8080/16341 Reviewed-by: Impala Public Jenkins <impala-public-jenk...@cloudera.com> Tested-by: Impala Public Jenkins <impala-public-jenk...@cloudera.com> --- M be/src/statestore/statestore.cc M be/src/statestore/statestore.h 2 files changed, 9 insertions(+), 5 deletions(-) Approvals: Impala Public Jenkins: Looks good to me, approved; Verified -- To view, visit http://gerrit.cloudera.org:8080/16341 To unsubscribe, visit http://gerrit.cloudera.org:8080/settings Gerrit-Project: Impala-ASF Gerrit-Branch: master Gerrit-MessageType: merged Gerrit-Change-Id: Id4363a93addb8a808d292906cac44ebd25c16889 Gerrit-Change-Number: 16341 Gerrit-PatchSet: 5 Gerrit-Owner: Tim Armstrong <tarmstr...@cloudera.com> Gerrit-Reviewer: Andrew Sherman <asher...@cloudera.com> Gerrit-Reviewer: Impala Public Jenkins <impala-public-jenk...@cloudera.com> Gerrit-Reviewer: Tim Armstrong <tarmstr...@cloudera.com>
