andygrove opened a new issue, #2177:
URL: https://github.com/apache/datafusion-comet/issues/2177

   ### What is the problem the feature request solves?
   
   We probably can't do this right away, but we should start working towards 
removing the `native_comet` scan implementation.
   
   Motivation:
   
   - Reduce the amount of code to maintain by reducing the number of scan 
implementations from 3 to 2
   - This is a major step towards 
https://github.com/apache/datafusion-comet/issues/2171
   
   The first step will be to review all of the tests where we currently 
explicitly set the scan implementation to `native_comet`. For each test we need 
to consider:
   
   - Can we update the test such that it passes when using the default scan 
implementation?
   - Is the test specific to `native_comet` and can therefore just be removed?
   - Does the test fail with the default scan because there is a bug or missing 
feature? If so, can we just fall back to Spark?
   
   
   
   ### Describe the potential solution
   
   _No response_
   
   ### Additional context
   
   _No response_


-- 
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...@datafusion.apache.org.apache.org

For queries about this service, please contact Infrastructure at:
us...@infra.apache.org


---------------------------------------------------------------------
To unsubscribe, e-mail: github-unsubscr...@datafusion.apache.org
For additional commands, e-mail: github-h...@datafusion.apache.org

Reply via email to