steveloughran commented on PR #6877: URL: https://github.com/apache/hadoop/pull/6877#issuecomment-2158462521
welcome to hadoop PRs! We do not need a feature that is already available. The challenge is getting it adopted at the stack. We also want to pass in more information and the status, specifically: read policy, split start and end, and maybe in future expected footer length. Because we already have the API this patch is not going to go in. However your experience working with it will help you in future and reviewing my own work, which is where your help is most critical right now: #6686 Note that whenever you go near cloud storage *which this patch implicitly does* the relevant testing policies apply, as covered in the s3a and abfs testing docs. Any patch which doesn't declare the endpoint you tested against doesn't exist. I recommend you get set up for testing against S3 as it sounds like you already have the credentials. Anything you can do to help get this up the stock will be wonderful82 that's not just a code, it is evangelisation. -- 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]
