jiacheliu3 commented on PR #733:
URL: https://github.com/apache/ratis/pull/733#issuecomment-1245161113

   > @jiacheliu3 , thanks for the update! This change becomes big and involves 
many files. Could you split it into two pull requests?
   > 
   > 1. Add a Daemon.Builder.
   > 2. Support UncaughtExceptionHandler.
   > 
   > Some comments inlined but I have not checked all the changes.
   
   Sure will do. My biggest concern at this moment is the backward 
compatibility, could you shed some light on this thread? 
   https://github.com/apache/ratis/pull/733#discussion_r967057138
   Thx!


-- 
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]

Reply via email to