Skip to content

Commit 547aa4d

Browse files
authored
Upgrade DF52 (#111)
* Upgrade DF52 * update test * use 52
1 parent 0d1aefa commit 547aa4d

File tree

3 files changed

+29
-28
lines changed

3 files changed

+29
-28
lines changed

Cargo.toml

Lines changed: 12 additions & 12 deletions
Original file line numberDiff line numberDiff line change
@@ -29,23 +29,23 @@ rust-version = "1.85.1"
2929

3030
[dependencies]
3131
aquamarine = "0.6.0"
32-
arrow = "57.0.0"
33-
arrow-schema = "57.0.0"
32+
arrow = "57.1.0"
33+
arrow-schema = "57.1.0"
3434
async-trait = "0.1.89"
3535
dashmap = "6"
36-
datafusion = "51"
37-
datafusion-common = "51"
38-
datafusion-expr = "51"
39-
datafusion-functions = "51"
40-
datafusion-functions-aggregate = "51"
41-
datafusion-optimizer = "51"
42-
datafusion-physical-expr = "51"
43-
datafusion-physical-plan = "51"
44-
datafusion-sql = "51"
36+
datafusion = "52"
37+
datafusion-common = "52"
38+
datafusion-expr = "52"
39+
datafusion-functions = "52"
40+
datafusion-functions-aggregate = "52"
41+
datafusion-optimizer = "52"
42+
datafusion-physical-expr = "52"
43+
datafusion-physical-plan = "52"
44+
datafusion-sql = "52"
4545
futures = "0.3"
4646
itertools = "0.14"
4747
log = "0.4"
48-
object_store = "0.12"
48+
object_store = "0.12.4"
4949
ordered-float = "5.0.0"
5050

5151
[dev-dependencies]

src/materialized/dependencies.rs

Lines changed: 15 additions & 14 deletions
Original file line numberDiff line numberDiff line change
@@ -860,7 +860,7 @@ fn project_dfschema(schema: &DFSchema, indices: &HashSet<usize>) -> Result<DFSch
860860
.filter_map(|i| {
861861
indices.contains(&i).then_some({
862862
let (reference, field) = schema.qualified_field(i);
863-
(reference.cloned(), Arc::new(field.clone()))
863+
(reference.cloned(), field.clone())
864864
})
865865
})
866866
.collect_vec();
@@ -1940,19 +1940,20 @@ mod test {
19401940
",
19411941
projection: &["year"],
19421942
expected_plan: vec![
1943-
"+--------------+--------------------------------------------------------------------+",
1944-
"| plan_type | plan |",
1945-
"+--------------+--------------------------------------------------------------------+",
1946-
"| logical_plan | Union |",
1947-
"| | Projection: coalesce(CAST(t1.year AS Utf8View), t2.year) AS year |",
1948-
"| | Full Join: Using CAST(t1.year AS Utf8View) = t2.year |",
1949-
"| | SubqueryAlias: t1 |",
1950-
"| | Projection: t1.column1 AS year |",
1951-
"| | TableScan: t1 projection=[column1] |",
1952-
"| | SubqueryAlias: t2 |",
1953-
"| | TableScan: t2 projection=[year] |",
1954-
"| | TableScan: t3 projection=[year] |",
1955-
"+--------------+--------------------------------------------------------------------+",
1943+
"+--------------+---------------------------------------------------------------------------------------------------+",
1944+
"| plan_type | plan |",
1945+
"+--------------+---------------------------------------------------------------------------------------------------+",
1946+
"| logical_plan | Union |",
1947+
"| | Projection: CASE WHEN __common_expr_2 IS NOT NULL THEN __common_expr_2 ELSE t2.year END AS year |",
1948+
"| | Projection: CAST(t1.year AS Utf8View) AS __common_expr_2, t2.year |",
1949+
"| | Full Join: Using CAST(t1.year AS Utf8View) = t2.year |",
1950+
"| | SubqueryAlias: t1 |",
1951+
"| | Projection: t1.column1 AS year |",
1952+
"| | TableScan: t1 projection=[column1] |",
1953+
"| | SubqueryAlias: t2 |",
1954+
"| | TableScan: t2 projection=[year] |",
1955+
"| | TableScan: t3 projection=[year] |",
1956+
"+--------------+---------------------------------------------------------------------------------------------------+",
19561957
],
19571958
expected_output: vec![
19581959
"+------+",

src/rewrite/normal_form.rs

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -718,11 +718,11 @@ impl Predicate {
718718
Some(range) => range,
719719
};
720720

721-
if other_range.contains(range)? != Interval::CERTAINLY_TRUE {
721+
if other_range.contains(range)? != Interval::TRUE {
722722
return Ok(None);
723723
}
724724

725-
if range.contains(other_range)? != Interval::CERTAINLY_TRUE {
725+
if range.contains(other_range)? != Interval::TRUE {
726726
if !(range.lower().is_null() || range.upper().is_null())
727727
&& (range.lower().eq(range.upper()))
728728
{

0 commit comments

Comments
 (0)