[GitHub] [ozone] sodonnel commented on pull request #3743: HDDS-7192. EC: ReplicationManager - create handlers to perform various container checks

[GitBox <gi...@apache.org>]

sodonnel commented on PR #3743:
URL: https://github.com/apache/ozone/pull/3743#issuecomment-1246432447

   > I was wondering if there could be a case where a container should be handled by multiple handlers?
   
   There may be a case like that in the future, but I think we should try to avoid it to keep things simple. In your example, with open replicas and under replication, we cannot use open containers for replication, so its better they are closed first.
   
   We could handle such a case by letting such a handler issue some commands and return false, or we could return some sort of status object in the chain to separate "handled" and "should continue", but for now we don't need it, so probably best to keep it simple and add later only if its needed.


-- 
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: issues-unsubscribe@ozone.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org


---------------------------------------------------------------------
To unsubscribe, e-mail: issues-unsubscribe@ozone.apache.org
For additional commands, e-mail: issues-help@ozone.apache.org