steveloughran commented on PR #5391:
URL: https://github.com/apache/hadoop/pull/5391#issuecomment-1429555546
-1 to anything exposing internal hdfs implementation methods. Sorry
People start using them and expect them to be stable and maintained. There
is also the little detail that in cloud deployments do not always have hdfs
jars on the class path; this PR would break those deployments.
What would make sense would be to use createFile() and for hdfs to add a
.opt() option for those favoured nodes, createFile() is the public api, .opt()
options can be ignorred by other filesystems, *or reimplemented*. There is a
lot more in terms of design and wiring up but the benefit is that portability
and maintainability.
--
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]
