dongxiaoman commented on PR #8813: URL: https://github.com/apache/pinot/pull/8813#issuecomment-1159154066
@xiangfu0 My main concern is the broker query performance hit or CPU waste because Query String is parsed. It may be small cost for some queries but may also be a lot for some cases. Another possible (extra) approach: 1. allow the user to specify table name in extra header `x-pinot-tables: airlinestats, meetupRSVP` 2. trust the user's intent, forward to the right tenant before parsing query 3. if query is already forwarded, and tenant is wrong, reject the query. This step will check whether user is honest enough This way if the user really cares about extra overhead, they can manually go through the trouble to set the table name in header. If they don't care, they can just let us parse 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: [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]
