chenboat commented on issue #3748: Use the PinotFS instead of local file system to open a segment file t… URL: https://github.com/apache/incubator-pinot/pull/3748#issuecomment-458370806 @mcvsubbu , some context about this PR, in the umbrella issue, we talked about a phased solution to use Pinot FS (the detailed plan [here](https://docs.google.com/document/d/16ny6mVwxgCBOVsSuXQmfYGtaXbONykURWQWZYuC7pUQ/edit#heading=h.11ytvbgvo26h)) https://github.com/apache/incubator-pinot/issues/3420#issuecomment-437806568 Initially we planned to do Pinot FS refactoring only. In the last phase M4, we plan to do what you suggested and let server upload the metadata directly. I also did a code work through and found some metadata field server needs to pass through is nested object (i.e., ColumnMetadata). So the last step is a more involved API change (rather than adding a few more parameters). I plan to work on next. So can we treat this PR as an incremental bug fix on what Jennifer has refactored and committed so far?
---------------------------------------------------------------- This is an automated message from the Apache Git Service. To respond to the message, please log on GitHub and use the URL above to go to the specific comment. For queries about this service, please contact Infrastructure at: [email protected] With regards, Apache Git Services --------------------------------------------------------------------- To unsubscribe, e-mail: [email protected] For additional commands, e-mail: [email protected]
