mcvsubbu 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-461229243 Thanks, @npawar for evaluating this. Here are things I will add: step 2 should be made configurable on the server side (i.e. only if a config is enabled, we should upload metadata) step 1a should be to modify LLCSegmentCompletionHandlers.commit to include checking whether two more parts are included, and if so, use them as metadataPropertiesFile and creationMetaFile) In Step 2 we need to handle both segmentCommit as well as segmentCommitEndWithMetadata under the same config mentioned before In step 3 call method depending on config
---------------------------------------------------------------- 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: us...@infra.apache.org With regards, Apache Git Services --------------------------------------------------------------------- To unsubscribe, e-mail: dev-unsubscr...@pinot.apache.org For additional commands, e-mail: dev-h...@pinot.apache.org
