Since this is fixed in persistit, I thought I'd just quickly go in and fix it in the server. First, just to prove that we need the return value to be correct, I replaced the if with:
if (exchange.remove() & true) {
Imagine my surprise when all ITs pass. Uh oh. FWIW, exchange.remove() & false does fail tests.
Since this is fixed in persistit, I thought I'd just quickly go in and fix it in the server. First, just to prove that we need the return value to be correct, I replaced the if with:
if (exchange.remove() & true) {
Imagine my surprise when all ITs pass. Uh oh. FWIW, exchange.remove() & false does fail tests.