Re: [PR] fix: Refactor CometScanRule and fix bugs [datafusion-comet]

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

Re: [PR] fix: Refactor CometScanRule and fix bugs [datafusion-comet]

2025-03-27 Thread via GitHub
andygrove commented on PR #1483: URL: https://github.com/apache/datafusion-comet/pull/1483#issuecomment-2738004391 > This change looks good to me. It makes sense to not run these tests for `native_datafusion` if we're falling back anyway. Do you have a good example of a test or query where

Re: [PR] fix: Refactor CometScanRule and fix bugs [datafusion-comet]

2025-03-26 Thread via GitHub
andygrove commented on PR #1483: URL: https://github.com/apache/datafusion-comet/pull/1483#issuecomment-2738005266 Thanks for the review @parthchandra and @mbutrovich -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use t

Re: [PR] fix: Refactor CometScanRule and fix bugs [datafusion-comet]

2025-03-19 Thread via GitHub
andygrove commented on PR #1483: URL: https://github.com/apache/datafusion-comet/pull/1483#issuecomment-2737541255 @mbutrovich Could you take a look as well since this disables some tests for `native_datafusion` exec. -- This is an automated message from the Apache Git Service. To respon

Re: [PR] fix: Refactor CometScanRule and fix bugs [datafusion-comet]

2025-03-17 Thread via GitHub
parthchandra commented on code in PR #1483: URL: https://github.com/apache/datafusion-comet/pull/1483#discussion_r1999846503 ## spark/src/main/scala/org/apache/comet/CometSparkSessionExtensions.scala: ## @@ -188,69 +185,62 @@ class CometSparkSessionExtensions sc

Re: [PR] fix: Refactor CometScanRule and fix bugs [datafusion-comet]

2025-03-17 Thread via GitHub
andygrove commented on code in PR #1483: URL: https://github.com/apache/datafusion-comet/pull/1483#discussion_r1999438466 ## spark/src/main/scala/org/apache/comet/CometSparkSessionExtensions.scala: ## @@ -188,69 +185,62 @@ class CometSparkSessionExtensions scanE

Re: [PR] fix: Refactor CometScanRule and fix bugs [datafusion-comet]

2025-03-17 Thread via GitHub
andygrove commented on code in PR #1483: URL: https://github.com/apache/datafusion-comet/pull/1483#discussion_r1999378679 ## spark/src/main/scala/org/apache/comet/CometSparkSessionExtensions.scala: ## @@ -188,69 +185,62 @@ class CometSparkSessionExtensions scanE

Re: [PR] fix: Refactor CometScanRule and fix bugs [datafusion-comet]

2025-03-17 Thread via GitHub
parthchandra commented on code in PR #1483: URL: https://github.com/apache/datafusion-comet/pull/1483#discussion_r1999359044 ## spark/src/main/scala/org/apache/comet/CometSparkSessionExtensions.scala: ## @@ -188,69 +185,62 @@ class CometSparkSessionExtensions sc

Re: [PR] fix: Refactor CometScanRule and fix bugs [datafusion-comet]

2025-03-17 Thread via GitHub
andygrove commented on PR #1483: URL: https://github.com/apache/datafusion-comet/pull/1483#issuecomment-2729501003 @mbutrovich @parthchandra This is now ready for review -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and us