You are viewing a plain text version of this content. The canonical link for it is here.

Posted to commits@pinot.apache.org by GitBox <gi...@apache.org> on 2022/03/15 05:19:03 UTC

[GitHub] [pinot] kmozaid commented on pull request #8255: Added multi column partitioning for offline table

kmozaid commented on pull request #8255:
URL: https://github.com/apache/pinot/pull/8255#issuecomment-1067574379


   > Shall we add a queries test (search for `*QueriesTest` for examples) or integration test to ensure it works end-to-end?
   
   I have added a test which prune segments based on query - 
   https://github.com/apache/pinot/pull/8255/files#diff-9e8214b130b1a4a54157230e84674df2328018e7e7044a90a1f2a114ad56674bR353
   
   and Sure let me see if I can add an integration test.


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