| 19 Sep 2025 |
John Ericson | and the view is just splatting those together | 16:16:46 |
fzakaria | addToStore is unsupported | 16:16:57 |
John Ericson | simply because I didn't do the other methods yet | 16:16:58 |
fzakaria | oh | 16:17:06 |
fzakaria | ok | 16:17:07 |
fzakaria | sorry | 16:17:08 |
John Ericson | I think we need Sergei Zimmerman (xokdvium)'s help to fix the type error 😅 but after that, you are welcome to do the rest | 16:17:28 |
fzakaria | I think i see what you mean:
ref<MemorySourceAccessor> wholeStoreView = make_ref<MemorySourceAccessor>();
That is probably a pattern (Delegating MemorySource Accessor or Composite accessor)
| 16:18:05 |
John Ericson | I think that line is fine? | 16:18:20 |
John Ericson | I am not sure though | 16:18:23 |
John Ericson | will try changing it | 16:18:24 |
fzakaria | i meant having 1 accessor but have it hide away the fact it's multiple; but maybe that doesn't matter once you do whatever subsequent refactor you meant | 16:19:00 |
John Ericson | oh ok | 16:19:21 |
fzakaria | i might epxlore after your work; the sqlite variant.... it's got a worm in my brain about it.
I like that sqlite can do in-memory or also persist to a file. | 16:19:50 |
John Ericson | do you think that can be deduplicated with local store? | 16:24:57 |