Skip to content

Commit 6a24310

Browse files
authored
Merge pull request #69 from apify/chore/netmilk/move-todo
Move TODO to issue
2 parents 04a96c2 + ee663b6 commit 6a24310

File tree

1 file changed

+0
-13
lines changed

1 file changed

+0
-13
lines changed

pages/KEY_VALUE_STORE_SCHEMA.md

Lines changed: 0 additions & 13 deletions
Original file line numberDiff line numberDiff line change
@@ -84,16 +84,3 @@ In addition to this user will be able to list by prefix directly:
8484
```
8585
https://api.apify.com/v2/key-value-stores/storeId/keys?prefix=post-images-
8686
```
87-
88-
## TODO(@jancurn)
89-
- Finalize this text, keep `collections` for now
90-
- What is kv-store schema is used by Actor to define structure of key-value store it operates on,
91-
but the developer defines a non-compatible record group for "INPUT" prefix?
92-
Maybe the default kv-stores should be created with a default record group to cover the "INPUT" prefixes
93-
and give them JSON types. Then, we'd never need to worry about existing records.
94-
But it's a breaking change for some Actors... maybe we can only do this for V2 Actors with Actor file...
95-
... it's getting quite complicated.
96-
- What if there's a conflict between record groups?
97-
Shall we consider the first one matching as if the file is valid for schema?
98-
99-
... just add a note that these conflicting situations are unspecified behavior

0 commit comments

Comments
 (0)