Skip to content

fix(build): route server UFS features through data transfer - #1458

Merged
lzjqsdd merged 1 commit into
CurvineIO:mainfrom
LJF-050:fix/server-ufs-feature-mapping
Aug 2, 2026
Merged

fix(build): route server UFS features through data transfer#1458
lzjqsdd merged 1 commit into
CurvineIO:mainfrom
LJF-050:fix/server-ufs-feature-mapping

Conversation

@LJF-050

@LJF-050 LJF-050 commented Aug 2, 2026

Copy link
Copy Markdown
Contributor

Summary

Route server-native UFS feature selection through curvine-data-transfer while preserving the existing client and CLI feature mappings.

Issue Describe / Design

The server crate split moved UFS composition for server builds into curvine-data-transfer, but the release build script continued to pass curvine-client/<backend> features to the server-only Cargo invocation. Cargo rejected those features because curvine-server no longer exposes the client composition path.

This change selects curvine-data-transfer/<backend> for the server-native scope. Test and client scopes continue to use curvine-client/<backend>, and the CLI scope continues to use curvine-cli/<backend>.

Changes

Area Change Impact
build/build.sh Map server-native OSS-HDFS and OpenDAL backend features to curvine-data-transfer Server-only release builds resolve UFS features against the crate that now owns server transfer composition
build/build.sh Preserve existing mappings for test, client, and CLI scopes Client-facing artifact behavior remains unchanged

Test verified

Test case Result
bash -n build/build.sh PASS
Cargo metadata with curvine-data-transfer/opendal-s3,curvine-common/jemalloc PASS
make format PASS
cargo build --release -p curvine-server --no-default-features --features curvine-data-transfer/opendal-s3,curvine-common/jemalloc PASS

Dependencies

@lzjqsdd
lzjqsdd merged commit 13a5d7f into CurvineIO:main Aug 2, 2026
3 checks passed
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.

2 participants