shounakmk219 commented on PR #17713: URL: https://github.com/apache/pinot/pull/17713#issuecomment-3912500026
@jineshparakh sounds good! > Each one needs a closer look at the PinotFS contract and how other implementations handle it My take would be to stick to what the PinotFS interface docs say about the behaviour, if that is not clear fix that first and propagate to all implementations. We can address these in separate PRs, maybe create an issue with all the gaps you found above so that its tracked and fixed later. > which is an additional call to GCS to verify the blob still exists on the server Got it, in that case it should be safe to skip it in case it's following getBlob right? -- 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: [email protected] For queries about this service, please contact Infrastructure at: [email protected] --------------------------------------------------------------------- To unsubscribe, e-mail: [email protected] For additional commands, e-mail: [email protected]
