[Performance] Using x2 to x3 more cache memory and bandwidth than apollo-datasource-rest about apollo-datasource-http

[kdybicz]

Discussion created: #44

[StarpTech]

This is to be expected because we make use of two caches with different lifetimes. If you don't need it feel free to create a PR to make this optional.

[kdybicz]

That's the simplest implementation possible. I believe there might be better way of addressing the issue than simply on/off switch for the second key. I would be curious to see if anyone could come up with more optimised caching strategy.

[StarpTech]

Is there anything left here? You can create a discussion here https://github.com/StarpTech/apollo-datasource-http/discussions

[kdybicz]

Creating a discussion might be good idea, but I think it's worth adding a "disclaimer" informing devs that this implementation will double the amount of memory used for cache, as this is not that obvious atm.

[StarpTech]

This highly depends on the use case. I preferred high availability over space efficiency.

[kdybicz]

But to do that you need to start throwing more and more money to keep buy of the tier AWS machines for cache and crazy bandwidth, which ie. in our case is less than ideal :)