Does the SCHEMA cache actually work?

[woodardj]

I've just enabled the RedisCacheAdapter, and by default, it seems to be caching users and roles just fine, and according to the TTL that I've set (on parse-server 2.3.7).

However, the __SCHEMA cache defaults to 5 seconds, which I'm not sure would be of any value. There's this note in the definitions file for the defaults: https://github.com/ParsePlatform/parse-server/blob/2.3.7/src/cli/definitions/parse-server.js#L215

If I set this property as a config when initializing ParseServer via:
schemaCacheTTL: 60 * 1000,

I see the keys accumulating in Redis, but every request appears to generate a new …__ALL_KEYS and …__MAIN_SCHEMA key, each of which contains identical data, rather than using the previously cached value.

Is caching the SCHEMA through the redis adapter working for anyone? What have I missed?

[flovilmart]

Thanks for reporting the issue, do you have an idea how we could write a failing test for that?

[flovilmart]

@woodardj I went back to the CLI options and yes, I believe you missed the enableSingleSchemaCache; without that option, the _schema is called upon every request.

[woodardj]

Ah, thanks for circling back, @flovilmart — I must have come to a similar conclusion, as I have that enableSingleSchemaCache param in my config. I'd be happy to add this to docs somewhere and issue a PR, but I'm not sure the best place.

[flovilmart]

enableSingleSchemaCache is true in your config?

[woodardj]

It is, I must have found it digging through source; our cache appears to be behaving as expected now. I probably found it and forgot to post back on this issue.

[flovilmart]

Awesome :) we probably need to document that. Do you want to add a PR on https://github.com/parse-community/docs/blob/gh-pages/_includes/parse-server/cache-adapters.md or another section about using SchemaCache altogether?

[woodardj]

Can do!

…

-Jonathan

On Wed, May 17, 2017 at 10:07 AM, Florent Vilmart ***@***.***> wrote: Awesome :) we probably need to document that. Do you want to add a PR on https://github.com/parse-community/docs/blob/gh-pages/_ includes/parse-server/cache-adapters.md or another section about using SchemaCache altogether? — You are receiving this because you were mentioned. Reply to this email directly, view it on GitHub <#3663 (comment)>, or mute the thread <https://github.com/notifications/unsubscribe-auth/AAiDSc9YjxPrttNv3wtDpkjULV9wkumHks5r6xtYgaJpZM4MnK97> .

[flovilmart]

Thanks! Very appreciated!

[flovilmart]

Closing as not an issue with parse-server