Skip to content

Commit

Permalink
refactor(connector): replace protobuf-native with protox for sche…
Browse files Browse the repository at this point in the history
…ma compiling (#18543)

Signed-off-by: Bugen Zhao <[email protected]>
  • Loading branch information
BugenZhao authored Sep 18, 2024
1 parent a3c6e48 commit 25b0cc9
Show file tree
Hide file tree
Showing 6 changed files with 152 additions and 187 deletions.
179 changes: 84 additions & 95 deletions Cargo.lock

Some generated files are not rendered by default. Learn more about how customized files appear on GitHub.

3 changes: 1 addition & 2 deletions src/connector/codec/Cargo.toml
Original file line number Diff line number Diff line change
Expand Up @@ -29,7 +29,7 @@ num-bigint = "0.4"
prost = { workspace = true, features = ["no-recursion-limit"] }
prost-reflect = { version = "0.14", features = ["serde"] }
prost-types = "0.13"
protobuf-native = "0.2.2"
protox = "0.7"
risingwave_common = { workspace = true }
risingwave_pb = { workspace = true }
rust_decimal = "1"
Expand All @@ -47,7 +47,6 @@ tokio = { version = "0.2", package = "madsim-tokio" }

[build-dependencies]
prost-build = "0.12"
protobuf-src = "1"

[target.'cfg(not(madsim))'.dependencies]
workspace-hack = { path = "../../workspace-hack" }
Expand Down
6 changes: 0 additions & 6 deletions src/connector/codec/build.rs
Original file line number Diff line number Diff line change
Expand Up @@ -26,10 +26,4 @@ fn main() {
.out_dir("./tests/integration_tests/protobuf")
.compile_protos(&protos, &Vec::<String>::new())
.unwrap();

let proto_include_path = protobuf_src::include();
println!(
"cargo:rustc-env=PROTO_INCLUDE={}",
proto_include_path.to_str().unwrap()
);
}
Loading

0 comments on commit 25b0cc9

Please sign in to comment.