Skip to content
This repository has been archived by the owner on Jul 27, 2023. It is now read-only.

Override default backoff delay for Cache #184

Merged
merged 11 commits into from
Nov 4, 2016
Merged

Override default backoff delay for Cache #184

merged 11 commits into from
Nov 4, 2016

Conversation

yfouquet
Copy link
Collaborator

@yfouquet yfouquet commented Nov 3, 2016

Default backoff delay for caches is equal to 10s.
Enable this value to be overriden from properties if such a property exists.
For instance, "-Dcom.orbitz.consul.cache.backOffDelay=500".

I chose to use properties as the code of caches starts to grow fast, some
caches already have 4 constructors and i think it could be better to avoid
the creation of another one.

yfouquet and others added 11 commits October 10, 2016 17:23
Commit 3714976 changes the way the prefix are removed from the key.
It works fine for key/subkey pattern but fails when we are looking for the
exact key (for intance, "a/b").
* Fix indentation and renumber examples

In example 4 there was an extra level of indentation, and there was no example 5, but examples 6, and 7, 8. I fixed the indentation and the numbering gap.

* add slight documentation to example 4

I added a single line of documentation to example 4 as per #174.

* Revert "Fix indentation and renumber examples"
In example 4 there was an extra level of indentation, and there was no example 5, but examples 6, and 7, 8. I fixed the indentation and the numbering gap.
* Add eclipse project files to .gitignore

* Add support for Check-And-Set option to DELETE operation.
Default backoff delay for caches is equal to 10s.
Enable this value to be overriden from properties if such a property exists.
For instance, "-Dcom.orbitz.consul.cache.backOffDelay=500".

I chose to use properties as the code of caches starts to grow fast, some
caches already have 4 constructors and i think it could be better to avoid
the creation of another one.
@rickfast rickfast merged commit 1b4e3a2 into rickfast:master Nov 4, 2016
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants