[GitHub] [pinot] Jackie-Jiang commented on pull request #7894: #7714 ignore query options in commented out queries

[GitBox <gi...@apache.org>]

Jackie-Jiang commented on pull request #7894:
URL: https://github.com/apache/pinot/pull/7894#issuecomment-993084962


   @kriti-sc
   I feel it can be hard to identify identifiers and literals from the query as that basically requires parsing the query
   Calcite does support query hints which is similar to the options we have: https://calcite.apache.org/docs/reference.html#sql-hints. We can have a separate thread discussing whether it makes sense to replace the current query options with query hints, or provide query options as a separate parameter so that we don't have to extract that from the query.


-- 
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: commits-unsubscribe@pinot.apache.org

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



---------------------------------------------------------------------
To unsubscribe, e-mail: commits-unsubscribe@pinot.apache.org
For additional commands, e-mail: commits-help@pinot.apache.org