siddharthteotia commented on PR #8917: URL: https://github.com/apache/pinot/pull/8917#issuecomment-1178330385
Closing this PR since all the split PRs with tests have been merged in https://github.com/apache/pinot/pull/8953 and https://github.com/apache/pinot/pull/8993 respectively to provide query support for MV raw columns issue https://github.com/apache/pinot/issues/8875 The only remaining TODO is to add the support for mutable segment for var width since it needs to write a new writer. Part 3 https://github.com/apache/pinot/pull/9004 which was also done as part of this is not necessarily related to MV raw query support. It was a side-effect / cleanup change we wanted to make as part of this work. Based on the changes in 9004 PR, it has more implications and changes and discussion is needed from `ForwardIndexReader` interface perspective regardless of SV or MV imo. cc @somandal @Jackie-Jiang @walterddr -- 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-unsubscr...@pinot.apache.org For queries about this service, please contact Infrastructure at: us...@infra.apache.org --------------------------------------------------------------------- To unsubscribe, e-mail: commits-unsubscr...@pinot.apache.org For additional commands, e-mail: commits-h...@pinot.apache.org
