Skip to content
This repository was archived by the owner on Apr 26, 2024. It is now read-only.

Add option for timed /sync cache#9484

Closed
ShadowJonathan wants to merge 2 commits into
matrix-org:developfrom
ShadowJonathan:fix/3880
Closed

Add option for timed /sync cache#9484
ShadowJonathan wants to merge 2 commits into
matrix-org:developfrom
ShadowJonathan:fix/3880

Conversation

@ShadowJonathan
Copy link
Copy Markdown
Contributor

@ShadowJonathan ShadowJonathan commented Feb 24, 2021

This adds the experimental_features.sync_cache_timeout_ms config value. Thus this fixes #3880.

Stabilisation for general recommended use is left for a future PR or decision, after this has been tested enough (in my opinion).

Pull Request Checklist

  • Pull request is based on the develop branch
  • Pull request includes a changelog file. The entry should:
    • Be a short description of your change which makes sense to users. "Fixed a bug that prevented receiving messages from other servers." instead of "Moved X method from EventStore to EventWorkerStore.".
    • Use markdown where necessary, mostly for code blocks.
    • End with either a period (.) or an exclamation mark (!).
    • Start with a capital letter.
  • Pull request includes a sign off
  • Code style is correct (run the linters)

Signed-off-by: Jonathan de Jong <jonathan@automatia.nl>

@clokep clokep changed the title Fix #3880 (add option for timed /sync cache) Add option for timed /sync cache Feb 24, 2021
@clokep clokep linked an issue Feb 24, 2021 that may be closed by this pull request
@ShadowJonathan ShadowJonathan marked this pull request as draft March 2, 2021 15:01
@ShadowJonathan
Copy link
Copy Markdown
Contributor Author

Folded into #9739

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.

cache sync results for a while after the request completes

1 participant