Re: [PR] Migrate Optimizer tests to insta, part6 [datafusion]

2025-05-09 Thread via GitHub
qstommyshu commented on code in PR #15984: URL: https://github.com/apache/datafusion/pull/15984#discussion_r2081559659 ## datafusion/optimizer/src/analyzer/type_coercion.rs: ## @@ -1569,37 +1810,60 @@ mod test { let empty = empty_with_type(DataType::Boolean); l

Re: [PR] Migrate Optimizer tests to insta, part6 [datafusion]

2025-05-09 Thread via GitHub
alamb commented on PR #15984: URL: https://github.com/apache/datafusion/pull/15984#issuecomment-2867208259 🎉 -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comment. To unsubscribe

Re: [PR] Migrate Optimizer tests to insta, part6 [datafusion]

2025-05-09 Thread via GitHub
blaginin commented on code in PR #15984: URL: https://github.com/apache/datafusion/pull/15984#discussion_r208156 ## datafusion/optimizer/src/analyzer/type_coercion.rs: ## @@ -1569,37 +1810,60 @@ mod test { let empty = empty_with_type(DataType::Boolean); let

Re: [PR] Migrate Optimizer tests to insta, part6 [datafusion]

2025-05-09 Thread via GitHub
blaginin merged PR #15984: URL: https://github.com/apache/datafusion/pull/15984 -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comment. To unsubscribe, e-mail: github-unsubscr...@dataf

Re: [PR] Migrate Optimizer tests to insta, part6 [datafusion]

2025-05-09 Thread via GitHub
qstommyshu commented on code in PR #15984: URL: https://github.com/apache/datafusion/pull/15984#discussion_r2081563280 ## datafusion/optimizer/src/test/mod.rs: ## @@ -99,29 +99,20 @@ pub fn get_tpch_table_schema(table: &str) -> Schema { } } Review Comment: Looks like

Re: [PR] Migrate Optimizer tests to insta, part6 [datafusion]

2025-05-08 Thread via GitHub
Copilot commented on code in PR #15984: URL: https://github.com/apache/datafusion/pull/15984#discussion_r2080588006 ## datafusion/optimizer/src/test/mod.rs: ## @@ -99,29 +99,20 @@ pub fn get_tpch_table_schema(table: &str) -> Schema { } } Review Comment: [nitpick] Con

Re: [PR] Migrate Optimizer tests to insta, part6 [datafusion]

2025-05-08 Thread via GitHub
blaginin commented on code in PR #15984: URL: https://github.com/apache/datafusion/pull/15984#discussion_r2080589228 ## datafusion/optimizer/src/analyzer/type_coercion.rs: ## @@ -1569,37 +1810,60 @@ mod test { let empty = empty_with_type(DataType::Boolean); let

Re: [PR] Migrate Optimizer tests to insta, part6 [datafusion]

2025-05-07 Thread via GitHub
qstommyshu commented on PR #15984: URL: https://github.com/apache/datafusion/pull/15984#issuecomment-2860488095 1 more PR, then this should be done -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to

[PR] Migrate Optimizer tests to insta, part6 [datafusion]

2025-05-07 Thread via GitHub
qstommyshu opened a new pull request, #15984: URL: https://github.com/apache/datafusion/pull/15984 ## Which issue does this PR close? - Related #15396 , #15446, #15884, #15893, #15937, #15945 ## Rationale for this change ## What changes are included in thi