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: [email protected] With regards, Apache Git Services --------------------------------------------------------------------- To unsubscribe, e-mail: [email protected] For additional commands, e-mail: [email protected]
