search

oracle / coherence

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

COH-21232: Introduce NamedMap API #13

Closed aseovic closed 4 years ago

aseovic commented 4 years ago

Initial refactoring to add NamedMap API.

There is still work to reflect the change in the gRPC client and CDI, but this is a good start for the review.

Of course, there are many other places where notion of the "cache" leaks through (configuration, events, cache loaders/stores, etc.) but this is an incremental improvement in a highly visible part of the API. One step at a time...

aseovic commented 4 years ago

I've added CDI and gRPC client refactoring, which made this PR pretty massive as far as the number of files touched goes.

It does have some major usability improvements in it, is fairly low risk, and I need the changes to unblock other work, so I'm gonna submit it to RQ as-is to see if the PR process still works as it should and we can review it later.

coherence-bot commented 4 years ago

Failed to process pull request Failed to submit RQ job for P4 changelist 78973.

coherence-bot commented 4 years ago

Failed to process pull request Failed to submit RQ job for P4 changelist 78974.

aseovic commented 4 years ago

Merged.