zhztheplayer commented on PR #8143:
URL: 
https://github.com/apache/incubator-gluten/pull/8143#issuecomment-2519635299

   > > Non-goals:
   > > This doesn't tend to make Velox backend and CH backend work together.
   > 
   > This is for clarifying the non-goal of this pr or mixed backend work. 
According to #6920, it seems making different backends co-work at runtime is 
the long-term goal. Just a bit confused.
   
   Good question, thanks.
   
   Velox / CH may be possible to work together but it may require for 
considerable effort on refactoring. I am not sure whether any user is 
interested with this kind of combination now so I didn't intentionally put it 
as a goal for this PR (will make a note in issue #6920 too). I think we can 
choose to have another individual thread for VL+CH working together when it's 
proved to be worth to have our investment.


-- 
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: [email protected]

For queries about this service, please contact Infrastructure at:
[email protected]


---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]

Reply via email to