jvarenina commented on pull request #6477:
URL: https://github.com/apache/geode/pull/6477#issuecomment-845175503


   Hi @boglesby , @nabarunnag and @gesterzhou 
   
   Sorry to bother. Could you please share with me when you will have some time 
to review this PR?
   
   Also, If you have some concerns after a quick look, or some cases that might 
not work with this solution, please don't hesitate to share them in this PR. I 
would gladly create any additional test needed that will cover those cases.
   
   Also, I have one question for you. I couldn't find any test case in geode 
test suite that verify scenario when it is actually expected from server to 
store the events in bucketToTempQueue temporary queue. Do you maybe know if 
such test cases exist?


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


Reply via email to