Skip to content

Commit

Permalink
forbid external commands in simulation
Browse files Browse the repository at this point in the history
Signed-off-by: Bugen Zhao <[email protected]>
  • Loading branch information
BugenZhao committed Sep 20, 2023
1 parent 4f7ea71 commit 7c3f6d1
Show file tree
Hide file tree
Showing 3 changed files with 20 additions and 3 deletions.
15 changes: 13 additions & 2 deletions Cargo.lock

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

2 changes: 1 addition & 1 deletion src/tests/simulation/Cargo.toml
Original file line number Diff line number Diff line change
Expand Up @@ -44,7 +44,7 @@ risingwave_sqlsmith = { workspace = true }
serde = "1.0.188"
serde_derive = "1.0.188"
serde_json = "1.0.107"
sqllogictest = "0.15.3"
sqllogictest = "0.17.0"
tempfile = "3"
tokio = { version = "0.2.23", package = "madsim-tokio" }
tokio-postgres = "0.7"
Expand Down
6 changes: 6 additions & 0 deletions src/tests/simulation/src/client.rs
Original file line number Diff line number Diff line change
Expand Up @@ -223,4 +223,10 @@ impl sqllogictest::AsyncDB for RisingWave {
async fn sleep(dur: Duration) {
tokio::time::sleep(dur).await
}

async fn run_command(
_command: std::process::Command,
) -> std::io::Result<std::process::ExitStatus> {
unimplemented!("spawning process is not supported in simulation mode")
}
}

0 comments on commit 7c3f6d1

Please sign in to comment.