ifndef-SleePy commented on issue #9269: [FLINK-9900][tests] Fix unstable 
ZooKeeperHighAvailabilityITCase
URL: https://github.com/apache/flink/pull/9269#issuecomment-516479435
 
 
   Hi @tillrohrmann , thanks for reviewing.
   
   Actually I implemented the solution at first as same as your suggestion. 
However it fails sometime when I test the solution by slowing the job failing a 
lot. There are some unexpected restorations. The assertion of `illegalRestores` 
fails because it is greater than zero. I believe the reason is also the race 
condition of `CheckpointFailureManager`.
   
   It seems to need more changes than I expected, so I abandoned the 
implementation. I could give a deeper analysis later about this. 

----------------------------------------------------------------
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.
 
For queries about this service, please contact Infrastructure at:
us...@infra.apache.org


With regards,
Apache Git Services

Reply via email to