[
https://issues.apache.org/jira/browse/THRIFT-5598?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel
]
Sylwester Lachiewicz resolved THRIFT-5598.
------------------------------------------
Resolution: Duplicate
Closing as a duplicate of THRIFT-5597 - same title, same reporter, same day, so
this looks like a double submit.
To be clear, this closes the duplicate ticket and not the defect: the
contrib/fb303 build failure is unresolved and stays open on THRIFT-5597, which
is the copy carrying the discussion.
_This comment was created with AI assistance._
> contrib/fb303 does not compile after C++ concurrency code refactor
> ------------------------------------------------------------------
>
> Key: THRIFT-5598
> URL: https://issues.apache.org/jira/browse/THRIFT-5598
> Project: Thrift
> Issue Type: Bug
> Components: C++ - Library, Contributed
> Reporter: Michael Leinartas
> Priority: Minor
>
> 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.10#820010)
