Skip to content

feat: re-export types, runtime-api, and version from client and server crates#90

Merged
drmingdrmer merged 1 commit intodatabendlabs:mainfrom
drmingdrmer:037-stream-append
Mar 5, 2026
Merged

feat: re-export types, runtime-api, and version from client and server crates#90
drmingdrmer merged 1 commit intodatabendlabs:mainfrom
drmingdrmer:037-stream-append

Conversation

@drmingdrmer
Copy link
Member

Changelog

feat: re-export types, runtime-api, and version from client and server crates

Applications using databend-meta-client or databend-meta can now access
types, runtime_api, and version through the main crate re-exports,
eliminating the need to add these as separate dependencies in Cargo.toml.

The server's version module is renamed to raft_version to avoid
conflicting with the re-exported databend_meta_version crate. The
redundant pub use databend_meta_version::MIN_CLIENT_VERSION is removed
since it's now available directly via crate::version.


…r crates

Applications using `databend-meta-client` or `databend-meta` can now access
`types`, `runtime_api`, and `version` through the main crate re-exports,
eliminating the need to add these as separate dependencies in `Cargo.toml`.

The server's `version` module is renamed to `raft_version` to avoid
conflicting with the re-exported `databend_meta_version` crate. The
redundant `pub use databend_meta_version::MIN_CLIENT_VERSION` is removed
since it's now available directly via `crate::version`.
Copilot AI review requested due to automatic review settings March 5, 2026 06:51
Copy link
Collaborator

@xp-trumpet xp-trumpet left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@xp-trumpet partially reviewed 3 files and all commit messages.
Reviewable status: :shipit: complete! all files reviewed, all discussions resolved (waiting on drmingdrmer).

Copy link

Copilot AI left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull request overview

This PR adds convenience re-exports of the types, runtime_api, and version crates from both the databend-meta-client and databend-meta (server) crates. This allows downstream consumers to access these dependencies through the main crate re-exports rather than adding them as separate Cargo.toml entries. The server's internal version module is renamed to raft_version to avoid a naming conflict with the new databend_meta_version re-export.

Changes:

  • Re-export databend_meta_runtime_api, databend_meta_types, and databend_meta_version as runtime_api, types, and version from both the client and server crates.
  • Rename the server's version module to raft_version to avoid conflicting with the re-exported databend_meta_version crate.
  • Remove the now-redundant pub use databend_meta_version::MIN_CLIENT_VERSION from raft_version.rs, since MIN_CLIENT_VERSION is accessible via the re-exported crate::version.

Reviewed changes

Copilot reviewed 3 out of 3 changed files in this pull request and generated no comments.

File Description
crates/client/client/src/lib.rs Adds pub extern crate re-exports for runtime_api, types, and version.
crates/server/service/src/lib.rs Adds pub extern crate re-exports for runtime_api, types, and version; renames pub mod version to pub mod raft_version.
crates/server/service/src/raft_version.rs Removes the now-redundant pub use databend_meta_version::MIN_CLIENT_VERSION re-export.

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

Merged via the queue into databendlabs:main with commit 949798e Mar 5, 2026
7 checks passed
@drmingdrmer drmingdrmer deleted the 037-stream-append branch March 5, 2026 06:59
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants