[
https://issues.apache.org/jira/browse/IMPALA-13978?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=17951921#comment-17951921
]
ASF subversion and git services commented on IMPALA-13978:
----------------------------------------------------------
Commit 78d1c2cd3a4f2c81a9142bc80b8bba9d2e9ff292 in impala's branch
refs/heads/master from Joe McDonnell
[ https://gitbox.apache.org/repos/asf?p=impala.git;h=78d1c2cd3 ]
IMPALA-14049: Fix TSAN issue with HdrHistogram in expr-test
IMPALA-13978 switched HdrHistrogram from using a gscoped_ptr
to unique_ptr. This has been causing TSAN issues during the
teardown for expr-test. gscoped_ptr doesn't null out the
pointer when it gets destructed, but unique_ptr does. This
is a data race with the threads that are still running and
trying to access the metrics.
The full solution would be to have an orderly shutdown of
all the threads before destructing things. That is a large
project that would touch many different components. As a
short-term fix, this avoids the TSAN issue by leaking
the statestore metrics.
We should consider fixing IMPALA-9314 and implementing
orderly shutdown.
Testing:
- Ran expr-test in a loop with TSAN and didn't see this
particular issue. There are other shutdown issues with
much lower frequency that have different symptoms.
Change-Id: I73c3f4db16c6ffa272f2512e9871db5743be7a54
Reviewed-on: http://gerrit.cloudera.org:8080/22900
Reviewed-by: Riza Suminto <riza.sumi...@cloudera.com>
Tested-by: Impala Public Jenkins <impala-public-jenk...@cloudera.com>
> Remove gutil's gscoped_ptr/bind/callback functionality
> ------------------------------------------------------
>
> Key: IMPALA-13978
> URL: https://issues.apache.org/jira/browse/IMPALA-13978
> Project: IMPALA
> Issue Type: Task
> Components: Backend
> Affects Versions: Impala 5.0.0
> Reporter: Joe McDonnell
> Assignee: Joe McDonnell
> Priority: Major
> Fix For: Impala 5.0.0
>
>
> The functionality in the gutil/gscoped_ptr.h header is obsolete with the
> modern C++ standard library. Kudu removed this a while back in
> [https://github.com/apache/kudu/commit/4f7b1aaf88d848b45083ab62d50c1c457c943954]
> We should bring those modifications over and adjust any other locations in
> our code. gscoped_ptr/gscoped_array can be replaced with std::unique_ptr.
> gutil/bind* and gutil/callback* headers are also unused. Once
> gutil/gscoped_ptr.h is gone, gutil/move.h can be removed.
--
This message was sent by Atlassian Jira
(v8.20.10#820010)
---------------------------------------------------------------------
To unsubscribe, e-mail: issues-all-unsubscr...@impala.apache.org
For additional commands, e-mail: issues-all-h...@impala.apache.org
