search

oracle / coherence

Oracle Coherence Community Edition
https://coherence.community
Universal Permissive License v1.0
426 stars 71 forks source link

Updated BDB version to one available in maven central #20

Closed smillidge closed 4 years ago

smillidge commented 4 years ago

BDB version referenced in the poms is not available in maven central so build fails.

Signed-off-by: smillidge steve.millidge@payara.fish

harveyraja commented 4 years ago

Hi @smillidge,

Thanks for bringing this to our attention. Unfortunately, it is slightly trickier than simply bumping the version.

With Coherence we offer backwards compatibility across major versions. Even though this is the first Coherence CE release, we do support migrating from a Coherence commercial release to Coherence CE. Moreover, we also support rolling upgrades / mixed version clusters in 20.06 and 20.06.01 and 20.12 (upcoming release).

I believe there are some breaking changes in BDB that allow you to migrate a DB from version n to n+1 but to not go back. As Coherence uses BDB as a means to store data on disk, there is a potential that a node that can read version n of BDB needs to read an n+1 format. This will be problematic.

Could you please raise a GH Issue for the BDB version not being available on Maven central, thus your inability to build? I thought we did validate this, but clearly we had a cached version of the dependency or something of that ilk.

Thanks, -Harvey

harveyraja commented 4 years ago

Also @smillidge, I just heard from @tmiddlet2666 who mentioned that BDB should get pulled from https://download.oracle.com/maven which is in the pom.xml. Can you confirm that this is not the case for you or if you can identify some other reason why https://download.oracle.com/maven is not being consulted?

harveyraja commented 4 years ago

Also please make sure you read the contributing markdown prior to submitting a PR.

smillidge commented 4 years ago

It didn't download from the oracle maven repo. As that repo is not browsable I can't tell if it is there. I will close the PR.