Skip to content

Commit ed16b0e

Browse files
Downgrade to DF 46
1 parent 8472744 commit ed16b0e

3 files changed

Lines changed: 13 additions & 8 deletions

File tree

Cargo.toml

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -13,10 +13,10 @@ readme = "README.md"
1313
repository = "https://github.com/datafusion-contrib/datafusion-federation"
1414

1515
[workspace.dependencies]
16-
arrow-json = "55"
16+
arrow-json = "54"
1717
async-stream = "0.3.5"
1818
async-trait = "0.1.83"
19-
datafusion = "47.0.0"
19+
datafusion = "46.0.1"
2020
datafusion-federation = { path = "./datafusion-federation", version = "0.4.2" }
2121
futures = "0.3.31"
2222
tokio = { version = "1.41", features = ["full"] }

datafusion-federation/src/lib.rs

Lines changed: 10 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -17,6 +17,7 @@ use datafusion::{
1717
execution::session_state::{SessionState, SessionStateBuilder},
1818
optimizer::{
1919
analyzer::{
20+
expand_wildcard_rule::ExpandWildcardRule, inline_table_scan::InlineTableScan,
2021
resolve_grouping_function::ResolveGroupingFunction, type_coercion::TypeCoercion,
2122
},
2223
Analyzer, AnalyzerRule,
@@ -44,6 +45,8 @@ pub fn default_session_state() -> SessionState {
4445
/// This list should be kept in sync with the default rules in `Analyzer::new()`, but with the federation analyzer rule added.
4546
pub fn default_analyzer_rules() -> Vec<Arc<dyn AnalyzerRule + Send + Sync>> {
4647
vec![
48+
Arc::new(ExpandWildcardRule::new()),
49+
Arc::new(InlineTableScan::new()),
4750
Arc::new(FederationAnalyzerRule::new()),
4851
// The rest of these rules are run after the federation analyzer since they only affect internal DataFusion execution.
4952
Arc::new(ResolveGroupingFunction::new()),
@@ -100,10 +103,15 @@ mod tests {
100103
let default_rules = Analyzer::new().rules;
101104
assert_eq!(
102105
default_rules.len(),
103-
2,
106+
4,
104107
"Default analyzer rules have changed"
105108
);
106-
let expected_rule_names = vec!["resolve_grouping_function", "type_coercion"];
109+
let expected_rule_names = vec![
110+
"inline_table_scan",
111+
"expand_wildcard_rule",
112+
"resolve_grouping_function",
113+
"type_coercion",
114+
];
107115
for (rule, expected_name) in default_rules.iter().zip(expected_rule_names.into_iter()) {
108116
assert_eq!(
109117
expected_name,

datafusion-federation/src/optimize/optimize_projections/mod.rs

Lines changed: 1 addition & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -464,11 +464,8 @@ fn merge_consecutive_projections(proj: Projection) -> Result<Transformed<Project
464464
expr,
465465
relation,
466466
name,
467-
metadata,
468467
}) => rewrite_expr(*expr, &prev_projection).map(|result| {
469-
result.update_data(|expr| {
470-
Expr::Alias(Alias::new(expr, relation, name).with_metadata(metadata))
471-
})
468+
result.update_data(|expr| Expr::Alias(Alias::new(expr, relation, name)))
472469
}),
473470
e => rewrite_expr(e, &prev_projection),
474471
}

0 commit comments

Comments
 (0)