search

ehcache / ehcache3

Ehcache 3.x line
http://www.ehcache.org
Apache License 2.0
2.02k stars 579 forks source link

Fixes #2796 Rename package Store -> store #3068

Open chrisdennis opened 2 years ago

AbfrmBlr commented 2 years ago

This PR is daggy, want to merge in some other branch as well?

chrisdennis commented 2 years ago

Pretty sure the merge in this is busted... I need to come back and look at it again later today.

cljohnso commented 1 year ago

The failed test, org.ehcache.clustered.replication.BasicClusteredCacheOpsReplicationMultiThreadedTest > testCRUD[consistency=EVENTUAL], appears to be a bit racy. It looks like it could use some additional coordination to ensure things actually get started before the ACTIVE is clobbered. May be some additional coordination needed between the "put" threads and the "get" thread. And are there enough executor threads to permit this test to run reliably -- 21 tasks vs 10 threads.

Definitely not related to your changes.