RFC: TiDB storage backend - #142
Conversation
|
Thank you for submitting the RFC for TiDB storage backend. We're reviewing your submission and debating on certain feature sets in the RFC. We will circle back with detailed response by 7/9. |
|
Thanks for the thorough RFC, @ngaut this is well thought through and we're happy to accept TiDB as an optional backend along these lines. The backend-additive, backend-owned-implementation approach is the right call. A few DynamoDB-parity points to fold in before/with implementation, split into what must match vs. what's fine to differ as long as it's documented: Must match DynamoDB (observable-behavior breaks otherwise)
Fine to differ, just call it out
Also please keep sparse-index/index-key type rules and projection (KEYS_ONLY/INCLUDE/ALL) enforced at read time, since items are stored once. Overall: accepted in principle, let's nail down the TTL-stream path and key-encoding details, then move to implementation behind the opt-in feature. Thanks again for the careful write-up! |
What
Adds
docs/rfcs/0000-tidb-storage-backend.md, a draft RFC for adding TiDB as a first-class optional ExtendDB storage backend.Why
The implementation prototype in #140 was marked deferred pending an RFC. This RFC tracks the design discussion requested by the maintainer and links to tracking issue #141.
Related: #140
Tracks: #141
Testing done
git diff --checkperl -ne 'print "$ARGV:$.:$_" if /[^\\x00-\\x7F]/' docs/rfcs/0000-tidb-storage-backend.md/tmp/extenddb-docs-venv/bin/python docs/build-docs.pyChecklist
cargo fmt --check) - documentation-only PRADR / RFC: this PR