You are viewing a plain text version of this content. The canonical link for it is here.

Posted to issues@impala.apache.org by "Joe McDonnell (Jira)" <ji...@apache.org> on 2022/10/31 15:58:00 UTC

[jira] [Created] (IMPALA-11694) Use C++ atomics for the implementation of gutil/atomicops.h

Joe McDonnell created IMPALA-11694:
--------------------------------------

             Summary: Use C++ atomics for the implementation of gutil/atomicops.h
                 Key: IMPALA-11694
                 URL: https://issues.apache.org/jira/browse/IMPALA-11694
             Project: IMPALA
          Issue Type: Improvement
          Components: Backend
    Affects Versions: Impala 4.2.0
            Reporter: Joe McDonnell


Chromium switched the implementation of atomicops.h and family to use the C++11 atomics library in this change:

[https://github.com/chromium/chromium/commit/57a4e4a50c673c25e9cdaab53e32f6e53aa0b574]

It allowed for a large simplification later on where most of the platform-specific logic and assembly could be removed. This seems easier to maintain than the current code.



--
This message was sent by Atlassian Jira
(v8.20.10#820010)