[GitHub] [pinot] Jackie-Jiang commented on issue #10633: Make Timestamp index an actual index

["Jackie-Jiang (via GitHub)" <gi...@apache.org>]

Jackie-Jiang commented on issue #10633:
URL: https://github.com/apache/pinot/issues/10633#issuecomment-1514032795

   For the context, the query override logic is in `BaseBrokerRequestHandler.handleTimestampIndexOverride()`.
   I'll vote for option 2 - making timestamp an actual index type. The current approach works, but is very hard to manage because it requires modifying table config, schema and query transparently.


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