You are viewing a plain text version of this content. The canonical link for it is here.
Posted to dev@thrift.apache.org by "Michael Leinartas (Jira)" <ji...@apache.org> on 2022/06/21 16:12:00 UTC

[jira] [Created] (THRIFT-5597) contrib/fb303 does not compile after C++ concurrency code refactor

Michael Leinartas created THRIFT-5597:
-----------------------------------------

             Summary: contrib/fb303 does not compile after C++ concurrency code refactor
                 Key: THRIFT-5597
                 URL: https://issues.apache.org/jira/browse/THRIFT-5597
             Project: Thrift
          Issue Type: Bug
          Components: C++ - Library, Contributed
            Reporter: Michael Leinartas


contrib/fb303 no longer compiles after THRIFT-4730 removed ReadWriteMutex.

The ReadWriteMutex used by fb303 is protecting counter statistics. These are very write-heavy and read-light (these are read only when the Service methods getCounters() or getCounter() are called). Because of this, retaining a read-write mutex to protect these counters is unjustified and we can safely swap this out for a normal Mutex.

Compilation also fails after THRIFT-5168 removed generation of *_constants.cpp and *_constants.h generation because the fb303 Makefile explicitly depends on these unnecessary files



--
This message was sent by Atlassian Jira
(v8.20.7#820007)