This is an automated email from the ASF dual-hosted git repository.
alamb pushed a commit to branch main
in repository https://gitbox.apache.org/repos/asf/datafusion.git
The following commit(s) were added to refs/heads/main by this push:
new 80ad713693 Remove unused assert_analyzed_plan_ne test helper (#13121)
80ad713693 is described below
commit 80ad713693133532b53c5a8e1fc4202084203986
Author: Piotr Findeisen <[email protected]>
AuthorDate: Tue Oct 29 12:22:26 2024 +0100
Remove unused assert_analyzed_plan_ne test helper (#13121)
Plan textual representation is rich. Testing it's not a particular
string is difficult to make robust, that's probably why the helper is
unused.
---
datafusion/optimizer/src/test/mod.rs | 14 --------------
1 file changed, 14 deletions(-)
diff --git a/datafusion/optimizer/src/test/mod.rs
b/datafusion/optimizer/src/test/mod.rs
index cabeafd8e7..94d07a0791 100644
--- a/datafusion/optimizer/src/test/mod.rs
+++ b/datafusion/optimizer/src/test/mod.rs
@@ -133,20 +133,6 @@ pub fn assert_analyzed_plan_with_config_eq(
Ok(())
}
-pub fn assert_analyzed_plan_ne(
- rule: Arc<dyn AnalyzerRule + Send + Sync>,
- plan: LogicalPlan,
- expected: &str,
-) -> Result<()> {
- let options = ConfigOptions::default();
- let analyzed_plan =
- Analyzer::with_rules(vec![rule]).execute_and_check(plan, &options, |_,
_| {})?;
- let formatted_plan = format!("{analyzed_plan}");
- assert_ne!(formatted_plan, expected);
-
- Ok(())
-}
-
pub fn assert_analyzed_plan_eq_display_indent(
rule: Arc<dyn AnalyzerRule + Send + Sync>,
plan: LogicalPlan,
---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]
