refactor: move database client to test (#3820)

* remove `Database` in `cli export` * move `Database` to tests-integration * fix clippy * move `DatabaseClient` along with `Database` * `cli export` now use http api * add TODO for `tests-integration` dependencies * cleanup code * 'cli export' test use http api * remove unsed dependencies in cmd crate * apply review comments and clean code * remove unused methods Signed-off-by: tison <[email protected]> --------- Signed-off-by: tison <[email protected]> Co-authored-by: tison <[email protected]>
GreptimeTeam · May 1, 2024 · 3b89b9d · 3b89b9d
1 parent 6957461
commit 3b89b9d
Show file tree

Hide file tree

Showing 23 changed files with 240 additions and 790 deletions.
diff --git a/Cargo.lock b/Cargo.lock
diff --git a/Cargo.toml b/Cargo.toml
@@ -225,6 +225,8 @@ sql = { path = "src/sql" }
 store-api = { path = "src/store-api" }
 substrait = { path = "src/common/substrait" }
 table = { path = "src/table" }
+# TODO some code depends on this
+tests-integration = { path = "tests-integration" }
 
 [workspace.dependencies.meter-macros]
 git = "https://github.com/GreptimeTeam/greptime-meter.git"

diff --git a/benchmarks/Cargo.toml b/benchmarks/Cargo.toml
@@ -33,6 +33,8 @@ rand.workspace = true
 rskafka.workspace = true
 serde.workspace = true
 store-api.workspace = true
+# TODO depend `Database` client
+tests-integration.workspace = true
 tokio.workspace = true
 toml.workspace = true
 uuid.workspace = true
diff --git a/benchmarks/src/bin/nyc-taxi.rs b/benchmarks/src/bin/nyc-taxi.rs
@@ -29,10 +29,11 @@ use client::api::v1::column::Values;
 use client::api::v1::{
     Column, ColumnDataType, ColumnDef, CreateTableExpr, InsertRequest, InsertRequests, SemanticType,
 };
-use client::{Client, Database, OutputData, DEFAULT_CATALOG_NAME, DEFAULT_SCHEMA_NAME};
+use client::{Client, OutputData, DEFAULT_CATALOG_NAME, DEFAULT_SCHEMA_NAME};
 use futures_util::TryStreamExt;
 use indicatif::{MultiProgress, ProgressBar, ProgressStyle};
 use parquet::arrow::arrow_reader::ParquetRecordBatchReaderBuilder;
+use tests_integration::database::Database;
 use tokio::task::JoinSet;
 
 const CATALOG_NAME: &str = "greptime";

diff --git a/src/client/examples/logical.rs b/src/client/examples/logical.rs