Skip to content

Commit 51065f3

Browse files
authored
chore(query): enable lib test (#15397)
* chore(query): enable lib test * chore(query): enable lib test for all crate * chore(query): enable lib test for all crate
1 parent d01fa36 commit 51065f3

File tree

62 files changed

+71
-135
lines changed

Some content is hidden

Large Commits have some content hidden by default. Use the searchbox below for content that may be hidden.

62 files changed

+71
-135
lines changed

src/common/arrow/Cargo.toml

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -8,7 +8,7 @@ description = "Arrow implementation forked from arrow2 and native format impleme
88

99
[lib]
1010
doctest = false
11-
test = false
11+
test = true
1212

1313
[features]
1414
default = ["arrow-default", "parquet-default"]

src/common/auth/Cargo.toml

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -8,7 +8,7 @@ edition = { workspace = true }
88

99
[lib]
1010
doctest = false
11-
test = false
11+
test = true
1212

1313
[dependencies]
1414
base64 = "0.21.0"

src/common/base/Cargo.toml

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -8,7 +8,7 @@ edition = { workspace = true }
88

99
[lib]
1010
doctest = false
11-
test = false
11+
test = true
1212

1313
[features]
1414
tracing = ["tokio/tracing"]

src/common/base/src/runtime/memory/mem_stat.rs

Lines changed: 11 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -98,7 +98,7 @@ impl MemStat {
9898
let mut used = self.used.fetch_add(batch_memory_used, Ordering::Relaxed);
9999

100100
used += batch_memory_used;
101-
self.peak_used.fetch_max(used, Ordering::Relaxed);
101+
let old_peak_used = self.peak_used.fetch_max(used, Ordering::Relaxed);
102102

103103
for (idx, parent_memory_stat) in self.parent_memory_stat.iter().enumerate() {
104104
if let Err(cause) = parent_memory_stat
@@ -108,6 +108,11 @@ impl MemStat {
108108
// We only roll back the memory that alloc failed
109109
self.used.fetch_sub(current_memory_alloc, Ordering::Relaxed);
110110

111+
if used > old_peak_used {
112+
self.peak_used
113+
.fetch_sub(current_memory_alloc, Ordering::Relaxed);
114+
}
115+
111116
for index in 0..idx {
112117
self.parent_memory_stat[index].rollback(current_memory_alloc);
113118
}
@@ -119,6 +124,11 @@ impl MemStat {
119124

120125
if let Err(cause) = self.check_limit(used) {
121126
if NEED_ROLLBACK {
127+
if used > old_peak_used {
128+
self.peak_used
129+
.fetch_sub(current_memory_alloc, Ordering::Relaxed);
130+
}
131+
122132
// NOTE: we cannot rollback peak_used of parent mem stat in this case
123133
// self.peak_used.store(peak_used, Ordering::Relaxed);
124134
self.rollback(current_memory_alloc);

src/common/building/Cargo.toml

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -8,7 +8,7 @@ edition = { workspace = true }
88

99
[lib]
1010
doctest = false
11-
test = false
11+
test = true
1212

1313
[dependencies]
1414
anyhow = { workspace = true }

src/common/cache/Cargo.toml

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -8,7 +8,7 @@ edition = { workspace = true }
88

99
[lib]
1010
doctest = false
11-
test = false
11+
test = true
1212

1313
[features]
1414
heapsize = ["heapsize_"]

src/common/cloud_control/Cargo.toml

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -8,7 +8,7 @@ edition = { workspace = true }
88

99
[lib]
1010
doctest = false
11-
test = false
11+
test = true
1212

1313
[dependencies]
1414
async-backtrace = { workspace = true }

src/common/exception/Cargo.toml

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -8,7 +8,7 @@ edition = { workspace = true }
88

99
[lib]
1010
doctest = false
11-
test = false
11+
test = true
1212

1313
[dependencies] # In alphabetical order
1414
databend-common-arrow = { path = "../arrow" }

src/common/grpc/Cargo.toml

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -8,7 +8,7 @@ edition = { workspace = true }
88

99
[lib]
1010
doctest = false
11-
test = false
11+
test = true
1212

1313
[dependencies] # In alphabetical order
1414
# Workspace dependencies

src/common/hashtable/Cargo.toml

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -8,7 +8,7 @@ edition = { workspace = true }
88

99
[lib]
1010
doctest = false
11-
test = false
11+
test = true
1212

1313
[dependencies] # In alphabetical order
1414
# Workspace dependencies

0 commit comments

Comments
 (0)