steveloughran commented on pull request #2168: URL: https://github.com/apache/hadoop/pull/2168#issuecomment-741748543
@ThomasMarquardt -thanks for the feedback; will respond/update the PR I've been doing it as one PR as it lets me * do interface and impl together, plus contract tests * rebase on a regular basis on a patch that's had little attention from reviewers * test it downstream in a PoC fork of parquet just to see what works/doesn't. FWIW passing in the file status/length makes a big difference in code which goes from list dir -> opening every file in it. Other than that, changing the seek policy is the big one. Feel free to not review those s3a bits, except to look and see if there are things you'd want/want to avoid. open() has got a lot more complicated than it used to be. ---------------------------------------------------------------- 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. For queries about this service, please contact Infrastructure at: [email protected] --------------------------------------------------------------------- To unsubscribe, e-mail: [email protected] For additional commands, e-mail: [email protected]
