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

Edits to APQ edits #3729

Merged
merged 3 commits into from
Feb 6, 2020

Conversation

StephenBarlow
Copy link
Contributor

It's edits all the way down

@StephenBarlow
Copy link
Contributor Author

Hi @abernix ! One question on top of the above: can you specify a TTL for the in-memory cache that doesn't have one by default? Or is the in-memory cache an entirely no-TTL zone?

@abernix
Copy link
Member

abernix commented Jan 28, 2020

@StephenBarlow Specifying the ttl is possible for the in-memory LRU cache, however, its default setting (unlike the distributed cache store implementations) is to not assign a TTL, which means it would behave in a least-recently-used mode. However, there is no default maxSize (read: "max records") configured for the APQ cache, so the default behavior is to register all operations, forever.

Copy link
Member

@abernix abernix left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks for tidying this up! I figured rambling would be more useful to form a more complete picture, so I hope all the extra words I used were not too painful to extract. 😉

Just a couple thoughts/tweaks in order and then I think we're ship-shape.

docs/source/performance/apq.md Outdated Show resolved Hide resolved
docs/source/performance/apq.md Outdated Show resolved Hide resolved
@abernix abernix merged commit d2bb550 into abernix/docs-for-apq-and-specifically-pr-3623 Feb 6, 2020
@abernix abernix deleted the sb/apq-edits branch February 6, 2020 15:45
abernix added a commit that referenced this pull request Feb 6, 2020
* Adjust heading levels, prior to additional changes.

This makes room in the hierarchy for the work I'm about to add.

* Adjust indentation of code-fences to be within the appropriate bullets.

Otherwise, it was difficult to see the actual bullet-points.

* Lower configuration to below the fold, letting the default behavior shine.

We don't really demand configuration for this feature, which is enabled by
default.  Still, we have a lot more configuration that needs to be noted,
which can't be done without sacrificing some of the structure of this page
by inflating it with various implementation details that should be
documented, but aren't - YET! (Stand-by for next commit!)

* Add details about Redis, Redis Sentinel and Redis Cluster.

Since the `cache` options themselves are usable on other interfaces within
Apollo Server, these interfaces likely deserve to be documented on another
page eventually, but for now, this works okay and is better than them not
being documented at all.

* Add details about adjusting the cache time-to-live (TTL).

Specifically, this documents the behavior that was introduced by the
below-referenced PR.

Ref: #3623

* Provide instructions for how to disable persisted queries, if desired.

This wasn't mentioned before, but seems worth documenting!

* Edits to APQ edits (#3729)

* Edits to APQ doc improvements

* Update docs/source/performance/apq.md

* Update docs/source/performance/apq.md

Co-authored-by: Jesse Rosenberger <git@jro.cc>

Co-authored-by: Stephen Barlow <barlow.stephen+git@gmail.com>
@github-actions github-actions bot locked as resolved and limited conversation to collaborators Apr 21, 2023
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.

2 participants