Possibly remove `BaseSessionStore`?

[ankatiyar]

Description

I was having a think around Polishing the API surface #4473 and also somewhat related #2973 and realised since:

• we removed ShelveStore in Remove ShelveStore #4148
• And we have removed experiment tracking from Kedro-Viz Remove experiment tracking kedro-viz#2202

Maybe we don't need the session store anymore? Would be a breaking change and would be good to evaluate before Kedro 1.0.0

Context

• OmegaConfigLoader resolves all config into dict and doesn't use the DictConfig from omegaconf because DictConfig was non-serialisable and was causing issues with storing parameters for Kedro-viz but we don't have that constraint anymore so Enable access to OmegaConfigLoader DictConfig instead of dict only #2973 can be looked into.
• Would simplify the code in KedroSession
• Not sure if BaseSessionStore is used for anything else (I could be wrong)

[datajoely]

So this is where @idanov did have a vision for versioning code, config and a history of runs. Is that direction dead?

[astrojuanlu]

I don't think it's dead but having the BaseSessionStore right now with any actual use is a flaring violation of YAGNI

The moment we get back to this (and I'm sure we will soon, #2307 is one of the most commonly requested features) we can think of a fresh approach I believe

[astrojuanlu]

We decided to keep BaseSessionStore for 1.0 while we work on #4503 👍🏼