[GitHub] [geode] dschneider-pivotal commented on pull request #2388: Feature/geode 5624 Use a single thread to ensure beforeCompletion and afterCompletion are executed by the same thread.

[GitHub]

change this to "while (!condition.getAsBoolean())" so the impl match the method 
name "waitForCondition".
So you will need to change the callers to flip the logic of the expressions 
they pass in.
change: "!finished" to "finished"
change: "status == -1 && !cancelled" to "status != -1 || cancelled".


[ Full content available at: https://github.com/apache/geode/pull/2388 ]
This message was relayed via gitbox.apache.org for devnull@infra.apache.org