Skip to content

Commit d2d14be

Browse files
committed
fix test
1 parent 4556716 commit d2d14be

File tree

3 files changed

+13
-14
lines changed

3 files changed

+13
-14
lines changed

src/query/service/tests/it/storages/fuse/operations/table_analyze.rs

Lines changed: 3 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -24,6 +24,7 @@ use databend_common_expression::ColumnId;
2424
use databend_common_expression::Scalar;
2525
use databend_common_io::prelude::borsh_deserialize_from_slice;
2626
use databend_common_storage::MetaHLL;
27+
use databend_common_storage::MetaHLL12;
2728
use databend_common_storages_fuse::io::MetaReaders;
2829
use databend_common_storages_fuse::io::MetaWriter;
2930
use databend_common_storages_fuse::statistics::reducers::merge_statistics_mut;
@@ -216,7 +217,8 @@ async fn test_table_analyze_without_prev_table_seq() -> Result<()> {
216217

217218
// generate table statistics.
218219
let col: Vec<u8> = vec![1, 3, 0, 0, 0, 118, 5, 1, 21, 6, 3, 229, 13, 3];
219-
let hll: HashMap<ColumnId, MetaHLL> = HashMap::from([(0, borsh_deserialize_from_slice(&col)?)]);
220+
let meta: MetaHLL12 = borsh_deserialize_from_slice(&col)?;
221+
let hll: HashMap<ColumnId, MetaHLL> = HashMap::from([(0, meta.into())]);
220222
let table_statistics =
221223
TableSnapshotStatistics::new(hll, HashMap::new(), snapshot_1.snapshot_id, 14);
222224
let table_statistics_location = location_gen.snapshot_statistics_location_from_uuid(

tests/sqllogictests/suites/base/09_fuse_engine/09_0020_analyze.test

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -129,7 +129,7 @@ analyze table `t`
129129
query TI
130130
select * from fuse_statistic('db_09_0020', 't')
131131
----
132-
a 3 (empty)
132+
a 2 (empty)
133133

134134
statement ok
135135
create or replace table t1 as select number as a from numbers(10);

tests/suites/1_stateful/09_http_handler/09_0008_forward.py

Lines changed: 9 additions & 12 deletions
Original file line numberDiff line numberDiff line change
@@ -29,9 +29,12 @@ def do_query(query, port=8000, session=None, node_id=None, wait=100):
2929

3030

3131
def get_txn_state(resp):
32-
return (resp.get("state") == "Succeeded",
33-
resp.get("session").get("need_sticky"),
34-
resp.get("session").get("txn_state"))
32+
return (
33+
resp.get("state") == "Succeeded",
34+
resp.get("session").get("need_sticky"),
35+
resp.get("session").get("txn_state"),
36+
)
37+
3538

3639
def test_txn_success():
3740
resp = do_query("create or replace table t1(a int)").json()
@@ -63,22 +66,16 @@ def test_txn_fail():
6366
session = resp.get("session")
6467

6568
# fail
66-
resp = do_query(
67-
"select 1/0", session=session
68-
).json()
69+
resp = do_query("select 1/0", session=session).json()
6970
assert get_txn_state(resp) == (False, False, "Fail"), resp
7071

7172
# fail because wrong node
72-
resp = do_query(
73-
"select 1", port=8002, session=session
74-
).json()
73+
resp = do_query("select 1", port=8002, session=session).json()
7574
session = resp.get("session")
7675
assert get_txn_state(resp) == (False, False, "Fail"), resp
7776

7877
# keep fail state until commit/abort
79-
resp = do_query(
80-
"select 1", session=session, node_id=node_id
81-
).json()
78+
resp = do_query("select 1", session=session, node_id=node_id).json()
8279
assert get_txn_state(resp) == (False, False, "Fail"), resp
8380
session = resp.get("session")
8481

0 commit comments

Comments
 (0)