[ 
https://issues.apache.org/jira/browse/IMPALA-5904?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=17065250#comment-17065250
 ] 

ASF subversion and git services commented on IMPALA-5904:
---------------------------------------------------------

Commit 8b162ff480aa346d1d32830097586d681976e2d1 in impala's branch 
refs/heads/master from Sahil Takiar
[ https://gitbox.apache.org/repos/asf?p=impala.git;h=8b162ff ]

IMPALA-5904: (part 3) Fix more TSAN bugs

As of this commit, all backend tests are TSAN clean (or suppressions
exist for any races that are considered benign).

TSAN Fixes:
* Fixes IMPALA-9374: Possible data race in TupleDescriptor::GetLlvmStruct
    * Replaces the assignment of llvm_field_idx_ with slot_idx_, since
      they are the same; added a DCHECK to enforce this
* Data race in be/src/exprs/scalar-expr-evaluator.cc:285:26
  impala::ScalarExprEvaluator::GetValue(...)
    * This race seems specific to data-stream-test, and does not occur
      during any other tests
    * I made some changes to DataStreamTest to fix the sharing of
      TupleRowComparators between threads

Suppresions:
* Added a suppresion for IMPALA-9455: Possible data race in
  kudu::security::InitKerberosForServer
* Added suppresions for hs2-http-test since it uses the ThriftServer, which
  is known to be racey (IMPALA-9314)

Testing:
* Ran exhaustive tests
* Re-ran TSAN tests and confirmed the data races have been fixed

Change-Id: I99b7b119e256085d1ba6977e1161fc658273b242
Reviewed-on: http://gerrit.cloudera.org:8080/15363
Tested-by: Impala Public Jenkins <[email protected]>
Reviewed-by: Tim Armstrong <[email protected]>


> Enable ThreadSanitizer for Impala
> ---------------------------------
>
>                 Key: IMPALA-5904
>                 URL: https://issues.apache.org/jira/browse/IMPALA-5904
>             Project: IMPALA
>          Issue Type: Improvement
>          Components: Infrastructure
>            Reporter: Tim Armstrong
>            Assignee: Sahil Takiar
>            Priority: Major
>              Labels: ramp-up
>         Attachments: impalad.ERROR
>
>
> It would be great to be able to automatically detect data races in Impala 
> using ThreadSanitizer to avoid tricky-to-reproduce bugs. This issue tracks 
> enabling ThreadSanitizer, fixing bugs and adding suppressions to get to the 
> point where Impala runs cleanly with the sanitizer.



--
This message was sent by Atlassian Jira
(v8.3.4#803005)

---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]

Reply via email to