Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

chore: update doc about compatibility changes #14260

Merged
Merged
Show file tree
Hide file tree
Changes from all commits
Commits
File filter

Filter by extension

Filter by extension

Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
17 changes: 9 additions & 8 deletions src/meta/README.md
Original file line number Diff line number Diff line change
Expand Up @@ -77,14 +77,15 @@ To add a new feature(add new type or update an type), the developer should do:

The following is an illustration of current query-meta compatibility:

| `Meta\Query` | [0.7.59, 0.8.80) | [0.8.80, 0.9.41) | [0.9.41, 1.1.34) | [1.1.34, +∞) |
|:-----------------|:-----------------|:-----------------|:-----------------|:-------------|
| [0.8.30, 0.8.35) | ✅ | ❌ | ❌ | ❌ |
| [0.8.35, 0.9.23) | ✅ | ✅ | ✅ | ❌ |
| [0.9.23, 0.9.42) | ❌ | ✅ | ✅ | ❌ |
| [0.9.42, 1.1.32) | ❌ | ❌ | ✅ | ❌ |
| [1.1.32, 1.2.63) | ❌ | ❌ | ✅ | ✅ |
| [1.2.63, +∞) | ❌ | ❌ | ✅ | ✅ |
| `Meta\Query` | [0.7.59, 0.8.80) | [0.8.80, 0.9.41) | [0.9.41, 1.1.34) | [1.1.34, 1.2.287) | [1.2.287, +∞) |
|:------------------|:-----------------|:-----------------|:-----------------|:---------------|:-----------|
| [0.8.30, 0.8.35) | ✅ | ❌ | ❌ | ❌ | ❌ |
| [0.8.35, 0.9.23) | ✅ | ✅ | ✅ | ❌ | ❌ |
| [0.9.23, 0.9.42) | ❌ | ✅ | ✅ | ❌ | ❌ |
| [0.9.42, 1.1.32) | ❌ | ❌ | ✅ | ❌ | ❌ |
| [1.1.32, 1.2.63) | ❌ | ❌ | ✅ | ✅ | ❌ |
| [1.2.63, 1.2.226) | ❌ | ❌ | ✅ | ✅ | ❌ |
| [1.2.226, +∞) | ❌ | ❌ | ✅ | ✅ | ✅ |


## Compatibility between databend-meta
Expand Down
2 changes: 1 addition & 1 deletion src/meta/client/src/lib.rs
Original file line number Diff line number Diff line change
Expand Up @@ -85,7 +85,7 @@ pub static METACLI_COMMIT_SEMVER: LazyLock<Version> = LazyLock::new(|| {
/// - 2024-01-02: since 1.2.279:
/// Meta client: remove `Compatible` for KVAppError and MetaAPIError, added in `2023-02-16: since 0.9.41`
///
/// - 2024-01-07: since TODO
/// - 2024-01-07: since 1.2.287:
/// client: remove calling RPC kv_api() with MetaGrpcReq::GetKV/MGetKV/ListKV, kv_api only accept Upsert;
/// client: remove using MetaGrpcReq::GetKV/MGetKV/ListKV;
/// client: remove falling back kv_read_v1(Streamed(List)) to kv_api(List), added in `2023-10-20: since 1.2.176`;
Expand Down
4 changes: 3 additions & 1 deletion src/meta/types/proto/meta.proto
Original file line number Diff line number Diff line change
Expand Up @@ -218,7 +218,7 @@ service MetaService {
// General KV API for get, mget, list, and upsert;
// It is introduced to replace `WriteMsg` and `ReadMsg`.
//
// Since: 0.8.35; 2022-09-14
// 2022-09-14: since: 0.8.35
rpc KvApi(RaftRequest) returns (RaftReply);

// Handle application read request.
Expand All @@ -228,6 +228,8 @@ service MetaService {
// It returns a stream of `StreamItem`.
// - For single-reply request, the stream contains only one item, e.g. `Get`.
// - For multi-reply request, the stream contains multiple items, e.g. `MGet` and `List`.
//
// 2023-10-17: since 1.2.163
rpc KvReadV1(RaftRequest) returns (stream StreamItem);

// Export all meta data.
Expand Down
Loading