siddhantsangwan commented on PR #4072:
URL: https://github.com/apache/ozone/pull/4072#issuecomment-1347905067
Do we have multiple threads adding and getting commands from the command
queue?
Checking the usages of `getNextCommand()`, it seems we only have a single
thread getting commands right now:
```
/**
* Task that periodically checks if we have any outstanding commands.
* It is assumed that commands can be processed slowly and in order.
* This assumption might change in future. Right now due to this
assumption
* we have single command queue process thread.
*/
```
Even if we have multiple threads, I don't think there are any potential
problems related to multithreaded inconsistency in the new code. I'm asking for
better understanding.
--
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]
