Jackie-Jiang commented on pull request #6947: URL: https://github.com/apache/incubator-pinot/pull/6947#issuecomment-844623761
@mcvsubbu `segment-spi` is supposed to be used by the `query-spi` (not introduced yet) to provide the storage layer of the segment. Since we haven't introduce the `query-spi` yet, I'm okay keeping the `SegmentDataManager` within the `segment-local`. We can move it later. -- 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: us...@infra.apache.org --------------------------------------------------------------------- To unsubscribe, e-mail: commits-unsubscr...@pinot.apache.org For additional commands, e-mail: commits-h...@pinot.apache.org
