Re: [PR] Fix array_has_all and array_has_any with empty array [datafusion]

2025-04-05 Thread via GitHub
alamb merged PR #15039: URL: https://github.com/apache/datafusion/pull/15039 -- 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...@datafusi

Re: [PR] Fix array_has_all and array_has_any with empty array [datafusion]

2025-03-27 Thread via GitHub
alamb commented on PR #15039: URL: https://github.com/apache/datafusion/pull/15039#issuecomment-2746381811 THanks again ! -- 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

Re: [PR] Fix array_has_all and array_has_any with empty array [datafusion]

2025-03-24 Thread via GitHub
LuQQiu commented on PR #15039: URL: https://github.com/apache/datafusion/pull/15039#issuecomment-2748650838 Thanks for all the reviews and suggestions! -- 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 g

Re: [PR] Fix array_has_all and array_has_any with empty array [datafusion]

2025-03-22 Thread via GitHub
LuQQiu commented on PR #15039: URL: https://github.com/apache/datafusion/pull/15039#issuecomment-2741622696 @alamb @Weijun-H Thanks for the review! -- 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

Re: [PR] Fix array_has_all and array_has_any with empty array [datafusion]

2025-03-17 Thread via GitHub
alamb commented on PR #15039: URL: https://github.com/apache/datafusion/pull/15039#issuecomment-2730069735 This PR appears to have some build / CI failures that are preventing it from merging so marking it as drat @LuQQiu can you please resolve the issues so it can be merged? -- Th

Re: [PR] Fix array_has_all and array_has_any with empty array [datafusion]

2025-03-07 Thread via GitHub
LuQQiu commented on code in PR #15039: URL: https://github.com/apache/datafusion/pull/15039#discussion_r1985809872 ## datafusion/sqllogictest/test_files/array.slt: ## @@ -5818,6 +5818,13 @@ false false false false false false false false false false false false +query BB Re

Re: [PR] Fix array_has_all and array_has_any with empty array [datafusion]

2025-03-07 Thread via GitHub
alamb commented on code in PR #15039: URL: https://github.com/apache/datafusion/pull/15039#discussion_r1985679028 ## datafusion/sqllogictest/test_files/array.slt: ## @@ -5818,6 +5818,13 @@ false false false false false false false false false false false false +query BB Rev

Re: [PR] Fix array_has_all and array_has_any with empty array [datafusion]

2025-03-07 Thread via GitHub
Weijun-H commented on code in PR #15039: URL: https://github.com/apache/datafusion/pull/15039#discussion_r1985179602 ## datafusion/functions-nested/src/array_has.rs: ## @@ -439,6 +439,16 @@ fn array_has_all_and_any_dispatch( ) -> Result { let haystack = as_generic_list_arr

Re: [PR] Fix array_has_all and array_has_any with empty array [datafusion]

2025-03-07 Thread via GitHub
Weijun-H commented on code in PR #15039: URL: https://github.com/apache/datafusion/pull/15039#discussion_r1985179602 ## datafusion/functions-nested/src/array_has.rs: ## @@ -439,6 +439,16 @@ fn array_has_all_and_any_dispatch( ) -> Result { let haystack = as_generic_list_arr

Re: [PR] Fix array_has_all and array_has_any with empty array [datafusion]

2025-03-06 Thread via GitHub
westonpace commented on code in PR #15039: URL: https://github.com/apache/datafusion/pull/15039#discussion_r1983700811 ## datafusion/functions-nested/src/array_has.rs: ## @@ -439,6 +439,14 @@ fn array_has_all_and_any_dispatch( ) -> Result { let haystack = as_generic_list_a

Re: [PR] Fix array_has_all and array_has_any with empty array [datafusion]

2025-03-06 Thread via GitHub
LuQQiu commented on code in PR #15039: URL: https://github.com/apache/datafusion/pull/15039#discussion_r1983838025 ## datafusion/functions-nested/src/array_has.rs: ## @@ -439,6 +439,14 @@ fn array_has_all_and_any_dispatch( ) -> Result { let haystack = as_generic_list_array

Re: [PR] Fix array_has_all and array_has_any with empty array [datafusion]

2025-03-06 Thread via GitHub
jayzhan211 commented on code in PR #15039: URL: https://github.com/apache/datafusion/pull/15039#discussion_r1983280996 ## datafusion/functions-nested/src/array_has.rs: ## @@ -439,6 +439,14 @@ fn array_has_all_and_any_dispatch( ) -> Result { let haystack = as_generic_list_a

Re: [PR] Fix array_has_all and array_has_any with empty array [datafusion]

2025-03-05 Thread via GitHub
alan910127 commented on code in PR #15039: URL: https://github.com/apache/datafusion/pull/15039#discussion_r1982608655 ## datafusion/functions-nested/src/array_has.rs: ## @@ -439,6 +439,14 @@ fn array_has_all_and_any_dispatch( ) -> Result { let haystack = as_generic_list_a

Re: [PR] Fix array_has_all and array_has_any with empty array [datafusion]

2025-03-05 Thread via GitHub
LuQQiu commented on code in PR #15039: URL: https://github.com/apache/datafusion/pull/15039#discussion_r1982505448 ## datafusion/functions-nested/src/array_has.rs: ## @@ -439,6 +439,14 @@ fn array_has_all_and_any_dispatch( ) -> Result { let haystack = as_generic_list_array

Re: [PR] Fix array_has_all and array_has_any with empty array [datafusion]

2025-03-05 Thread via GitHub
LuQQiu commented on code in PR #15039: URL: https://github.com/apache/datafusion/pull/15039#discussion_r1982505448 ## datafusion/functions-nested/src/array_has.rs: ## @@ -439,6 +439,14 @@ fn array_has_all_and_any_dispatch( ) -> Result { let haystack = as_generic_list_array

Re: [PR] Fix array_has_all and array_has_any with empty array [datafusion]

2025-03-05 Thread via GitHub
westonpace commented on code in PR #15039: URL: https://github.com/apache/datafusion/pull/15039#discussion_r1982443721 ## datafusion/functions-nested/src/array_has.rs: ## @@ -439,6 +439,14 @@ fn array_has_all_and_any_dispatch( ) -> Result { let haystack = as_generic_list_a

Re: [PR] Fix array_has_all and array_has_any with empty array [datafusion]

2025-03-05 Thread via GitHub
LuQQiu commented on PR #15039: URL: https://github.com/apache/datafusion/pull/15039#issuecomment-2702483400 @jayzhan211 Could you help approve the test workflows and review the PR? Thanks in advance -- This is an automated message from the Apache Git Service. To respond to the message, pl

[PR] Fix array_has_all and array_has_any with empty array [datafusion]

2025-03-05 Thread via GitHub
LuQQiu opened a new pull request, #15039: URL: https://github.com/apache/datafusion/pull/15039 ## Which issue does this PR close? - Closes #15038 ## What changes are included in this PR? Fix array_has_any and array_has_all with empty array input ## Are these chang