I like your comment for this function. I shows the 4 distinct steps of this 
method.

Maybe each step can be pulled out into it's own function since this function is 
already over 100 lines. It might help with readability

[ Full content available at: https://github.com/apache/geode/pull/3044 ]
This message was relayed via gitbox.apache.org for 
[email protected]

Reply via email to