Split/Hybrid Caching between InMemory and SQL review and thoughts about apollo-ios HOT 2 OPEN

Hi there @jaikumar-b. Thanks so much for starting this conversation. You're right that we don't have it on our roadmap to build chained caching right now. But we aren't against you implementing it yourself, and if there are minor, non-breaking changes we can make to the library to help facilitate that, we're open to the idea!

I'm going to look at the closed PR you made and make some inline comments and suggestions there. I think that will help us point this in the right direction and if you want to make another PR after the conversation, we'll happily review it!

I'm assuming you closed that PR simply because you were working against an older legacy version of the client. Would you be wanting to migrate to 1.0 and make the additions to the 1.0 APIs? If you're going to just modify the older client, I suggest you do that on a fork of your own, since we aren't actively adding features to those versions anymore.

from apollo-ios.

ons to the 1.0 APIs? If you're going to just modify the older client, I suggest you do that on a fork of your own, since we aren't actively adding features to those v

Hey @AnthonyMDev! Thanks for all the initial comments and great to hear that you're open to making non-breaking changes to enable this and also appreciate the quick review. We won't be migrating to 1.0 anytime soon due to the amount of effort needed although it's in our plans in the coming quarters. But willing to add these changes to 1.0 and maintain a version internally for the legacy version until we migrate over. Have started taking a look at 1.0 will try and take your feedback from the close PR and open a new on the dev repo this week

from apollo-ios.