[GitHub] [incubator-pinot] jackjlli commented on pull request #6942: Detect invalid column names from SQL query in BrokerRequestHandler

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

jackjlli commented on pull request #6942:
URL: https://github.com/apache/incubator-pinot/pull/6942#issuecomment-845358143


   > I'd strongly recommend short circuit the query on broker without sending it to servers. On server side all segments will be pruned and the result will be the same as all records are filtered out, which will cause confusion. You may refer to #6765 which short circuit the queries with wrong table name
   
   Thanks for the reference. I've updated the PR to directly return an empty response from broker side.


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

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