Skip to content

Commit 3fa2468

Browse files
committed
pcr
Signed-off-by: discord9 <[email protected]>
1 parent 9612706 commit 3fa2468

File tree

3 files changed

+6
-9
lines changed

3 files changed

+6
-9
lines changed

src/promql/src/extension_plan/range_manipulate.rs

Lines changed: 3 additions & 6 deletions
Original file line numberDiff line numberDiff line change
@@ -198,10 +198,7 @@ impl RangeManipulate {
198198
.iter()
199199
.map(|name| serialize_column_index(self.input.schema(), name))
200200
.collect::<Vec<u64>>();
201-
debug!(
202-
"RangeManipulate serialize time_index_idx: {}, tag_column_indices: {:?}",
203-
time_index_idx, tag_column_indices
204-
);
201+
205202
pb::RangeManipulate {
206203
start: self.start,
207204
end: self.end,
@@ -237,11 +234,11 @@ impl RangeManipulate {
237234
end: pb_range_manipulate.end,
238235
interval: pb_range_manipulate.interval,
239236
range: pb_range_manipulate.range,
237+
time_index: String::new(),
238+
field_columns: Vec::new(),
240239
input: placeholder_plan,
241240
output_schema: empty_schema,
242241
unfix: Some(unfix),
243-
time_index: String::new(),
244-
field_columns: Vec::new(),
245242
})
246243
}
247244
}

tests/cases/standalone/common/promql/encode_substrait.result

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -8,7 +8,7 @@ create table count_total (
88

99
Affected Rows: 0
1010

11-
-- if `RangeManipulate` can be encoded/decoded correctly in substrait, the following queries should pass(for now it simply wouldn't push down)
11+
-- if `RangeManipulate` can be encoded/decoded correctly in substrait, the following queries should pass
1212
-- SQLNESS REPLACE (peers.*) REDACTED
1313
-- SQLNESS REPLACE (partitioning.*) REDACTED
1414
tql explain (0, 100, '1s')

tests/cases/standalone/common/promql/encode_substrait.sql

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -6,7 +6,7 @@ create table count_total (
66
primary key (tag_a, tag_b),
77
);
88

9-
-- if `RangeManipulate` can be encoded/decoded correctly in substrait, the following queries should pass(for now it simply wouldn't push down)
9+
-- if `RangeManipulate` can be encoded/decoded correctly in substrait, the following queries should pass
1010
-- SQLNESS REPLACE (peers.*) REDACTED
1111
-- SQLNESS REPLACE (partitioning.*) REDACTED
1212
tql explain (0, 100, '1s')
@@ -19,4 +19,4 @@ tql eval (0, 100, '1s')
1919
tag_a="ffa",
2020
}[1h])[12h:1h];
2121

22-
drop table count_total;
22+
drop table count_total;

0 commit comments

Comments
 (0)