You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@bookkeeper.apache.org by "thetumbled (via GitHub)" <gi...@apache.org> on 2024/03/29 10:03:28 UTC

[I] Useless BookieCheckTask. [bookkeeper]

thetumbled opened a new issue, #4255:
URL: https://github.com/apache/bookkeeper/issues/4255

   We have listen for the changes of the list of available bookies and readonly bookies. Once changed, we will call `org.apache.bookkeeper.replication.Auditor#submitAuditTask` to start audit task.
   
   But we have another perodic task for audit bookies.
   `org.apache.bookkeeper.replication.Auditor#scheduleBookieCheckTask`
   
   I doubt with the meaning of such a periodic task as we can get the notification as soon as we receive the event from zookeeper.


-- 
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-unsubscribe@bookkeeper.apache.org.apache.org

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


Re: [I] Useless BookieCheckTask. [bookkeeper]

Posted by "shoothzj (via GitHub)" <gi...@apache.org>.
shoothzj commented on issue #4255:
URL: https://github.com/apache/bookkeeper/issues/4255#issuecomment-2124097688

   what if zookeeper shutdown ungracefully?what if notification lost?I think perodic would work will in this scenario. WDYT?


-- 
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-unsubscribe@bookkeeper.apache.org

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


Re: [I] Useless BookieCheckTask. [bookkeeper]

Posted by "thetumbled (via GitHub)" <gi...@apache.org>.
thetumbled closed issue #4255: Useless BookieCheckTask.
URL: https://github.com/apache/bookkeeper/issues/4255


-- 
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-unsubscribe@bookkeeper.apache.org

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


Re: [I] Useless BookieCheckTask. [bookkeeper]

Posted by "thetumbled (via GitHub)" <gi...@apache.org>.
thetumbled commented on issue #4255:
URL: https://github.com/apache/bookkeeper/issues/4255#issuecomment-2124132139

   > what if zookeeper shutdown ungracefully?what if notification lost?I think perodic would work will in this scenario. WDYT?
   
   Make sense, thanks for reply.


-- 
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-unsubscribe@bookkeeper.apache.org

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