[
https://issues.apache.org/jira/browse/THRIFT-5597?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=17557003#comment-17557003
]
Checo Zavala commented on THRIFT-5597:
--------------------------------------
[sergiozav...@yooho.com|mailto:sergiozav...@yooho.com]
> 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
> 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.7#820007)
