Build #160

Build: #160 was successful Changes by Simon Baslé <sbasle@vmware.com>

Code commits

reactor-pool

  • Simon Baslé <sbasle@vmware.com>

    Simon Baslé <sbasle@vmware.com> f4271fa6115695b41e1f38b516ce003b6e82ffd4

    Merge #129 into 0.2.4
    japicmp is still baselining against 0.1.x, but now the latest release:
    0.1.9.RELEASE.

  • Simon Baslé <sbasle@vmware.com>

    Simon Baslé <sbasle@vmware.com> 50e173045bc5521f818943788f41b4c3cbe2abcf m

    Implementation review, make a few small improvements... (#129)
    This commit fixes a few low hanging fruits in the current pool
    implementation:

     - Remove PENDING_COUNT and rely on `Queue#size()` directly instead
     - isDisposed also checks IDLE_RESOURCES is null
     - Remove WIP lazySets to prevent re-entry in terminal state
     - In drain loop, return slot if no borrower and invalidate slot
       if pool disposed. These case are protective against polling
       an unexpected null in the drain loop, although the conditional
       branches are reviewing the actual queue sizes.
     - In tests, make the switch to new RaceTestUtils methods
     - Ensure japicmp compares with the latest release as baseline

    This commit also adds a few remaining TODOs.

    • gradle.properties (version 50e173045bc5521f818943788f41b4c3cbe2abcf)
    • src/jcstress/java/reactor/pool/SimpleDequePoolStressTest.java (version 50e173045bc5521f818943788f41b4c3cbe2abcf)
    • src/main/java/reactor/pool/AbstractPool.java (version 50e173045bc5521f818943788f41b4c3cbe2abcf)
    • src/main/java/reactor/pool/SimpleDequePool.java (version 50e173045bc5521f818943788f41b4c3cbe2abcf)
    • src/test/java/reactor/pool/AcquireDefaultPoolTest.java (version 50e173045bc5521f818943788f41b4c3cbe2abcf)
    • src/test/java/reactor/pool/CommonPoolTest.java (version 50e173045bc5521f818943788f41b4c3cbe2abcf)
    • src/test/java/reactor/pool/PendingAcquireLifoBehaviorTest.java (version 50e173045bc5521f818943788f41b4c3cbe2abcf)