zhannngchen commented on code in PR #24240:
URL: https://github.com/apache/doris/pull/24240#discussion_r1322676715
##########
be/src/olap/memtable_flush_executor.h:
##########
@@ -87,7 +87,8 @@ class FlushToken {
// Records the current flush status of the tablet.
// Note: Once its value is set to Failed, it cannot return to SUCCESS.
- std::atomic<int> _flush_status;
+ Status _flush_status;
+ std::shared_mutex _flush_status_lock;
Review Comment:
It's normal put the lock in front of the variables it protects.
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: [email protected]
For queries about this service, please contact Infrastructure at:
[email protected]
---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]
