tokoko commented on issue #13:
URL: https://github.com/apache/arrow-adbc/issues/13#issuecomment-4687421073

   fwiw, I don't think we need another API for this and we shouldn't have to 
choose between sql and substrait either. These sort of drivers should be free 
to support either just like any other queries:
   
   - for sql, only a subset of sql should be supported, something like this: 
`SELECT <columns | *> FROM [schema.]<table> [WHERE <filter>]`. see iceberg 
[example](https://github.com/tokoko/adbc-iceberg#sql-support)
   - for substrait, you would also need support for a subset of substrait 
operations, maybe a dialect that allows for only ReadRel, ProjectionRel, 
FilterRel and specific functions.
   
   basically I'm for treating these readers as mini engines that can do limited 
projection and predicates, but nothing else.


-- 
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]

Reply via email to