KimZhangJinMing opened a new issue, #8069:
URL: https://github.com/apache/rocketmq/issues/8069
### Before Creating the Enhancement Request
- [X] I have confirmed that this should be classified as an enhancement
rather than a bug/feature.
### Summary
In class ServiceThread. The variables hasNotified has been declared as
AtomicBoolean. This is ensure the visibility, whether can declared without the
volatile?
protected volatile AtomicBoolean hasNotified = new AtomicBoolean(false);
### Motivation
This maybe cause confuse when view the source code.
### Describe the Solution You'd Like
protected AtomicBoolean hasNotified = new AtomicBoolean(false);
### Describe Alternatives You've Considered
None.
### Additional Context
_No response_
--
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: commits-unsubscr...@rocketmq.apache.org.apache.org
For queries about this service, please contact Infrastructure at:
us...@infra.apache.org
