Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

New module: Akka-HTTP-Caching #1539

Merged
merged 3 commits into from
Nov 21, 2017
Merged

New module: Akka-HTTP-Caching #1539

merged 3 commits into from
Nov 21, 2017

Conversation

raboof
Copy link
Member

@raboof raboof commented Nov 21, 2017

Squashed version of #1503

tomrf1 and others added 3 commits November 21, 2017 11:52
Rename to Lfu

Make remove return Unit

Move caffeine dependency to Dependencies.scala

Fix CachingDirectives package name

Increase delay in cache test

@ApiMayChange annotations

Remove test that sometimes fails due to race condition

Exclude akka-http-caching from MiMa previous artifacts

Remove the need for a loader on LfuCache creation

Simplify CachingDirectives - remove CacheSpecMagnet

MiMa ignoredModules

Refactor LfuCache - replace subclasses with store param

Docs and example code for caching directives

Move LfuCache params to LfuCacheSettings to support binary compatibility

Replace Keyed and ValueMagnet classes with overloaded apply and get methods

Remove default value for keyer function

Remove build.sbt

Update docs

Make cache key type generic

Fix Cache.remove comment

WIP towards JavaDSL for akka-http-caching

Use JavaMapping

formatting

copyright, docs, annotations

Make CacheJavaMapping internal

Java API markers

getStrict -> getOrCreateStrict

Remove simpleKeyer

Use LfuCacheSettings in routeCache

Example keyer for docs

Fix for sbt 1.0

Make Java DSL implement proper CachingSettings companion

=cache make settings reference.conf and mark things as internal
=doc Add server-side overview page (#1371)

Revert formatting changes back to how Scalariform formats

Mark more settings constructors private

Use self reference to implement with... methods

Override productPrefix in the setting implementations

Fix implementation of withLfuCacheSettings

Implement withLfuCacheSettings in the Java DSL and provide a
Scala-specific implementation as well.

Fix compilation error in the caching directive example

Fix places where the Java DSL leaks into the Scala DSL

For example apply methods should always take the Scala DSL instance
where as create methods should take the Java DSL instances.

Caching docs (#6)

* Fix use of fragments and IDs in directives

* Add Java caching directives examples

* Document Java caching directives

* Document the caching design and implementation

Port spray-caching documentation and update to the new caching API.

* Document the akka-http-caching reference config

* Fix reference to Java DSL cache

* Fix another Cache reference

* Reword docs related to the LFU cache
@akka-ci akka-ci added validating PR that is currently being validated by Jenkins tested PR that was successfully built and tested by Jenkins and removed validating PR that is currently being validated by Jenkins labels Nov 21, 2017
@akka-ci
Copy link

akka-ci commented Nov 21, 2017

Test PASSed.

@ktoso
Copy link
Member

ktoso commented Nov 21, 2017

L G T M !

google_image_result_for_http___images_dailydawdle_com_seal-of-approval_jpg

@ktoso ktoso changed the title Caching New module: Akka-HTTP-Caching Nov 21, 2017
@ktoso ktoso added the t:caching Issues related to the akka-http-caching module label Nov 21, 2017
@ktoso
Copy link
Member

ktoso commented Nov 21, 2017

Thanks @tomrf1 @ben-manes @jonas @jrudolph @yannick-cw et al for all the help, code and reviews. Very excited we finally got it in; I'm sorry it took us so long, but the outcome is very excellent so far :)

There's some small follow up ticktes already: t:caching Issues related to the akka-http-caching module
Feel free to pick at them or add new ones

Resolves #213

@ktoso ktoso merged commit 49baa95 into master Nov 21, 2017
@ktoso
Copy link
Member

ktoso commented Nov 21, 2017

And of course after writing "watch out with merging" I merge it using the wrong button... I'll fix it, so everyone gets attribution.

@tomrf1
Copy link
Contributor

tomrf1 commented Nov 21, 2017

🎉

@jrudolph jrudolph deleted the tomrf1-caching-more-squashing branch November 30, 2017 17:02
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
t:caching Issues related to the akka-http-caching module tested PR that was successfully built and tested by Jenkins
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants