shameersss1 commented on PR #6631:
URL: https://github.com/apache/hadoop/pull/6631#issuecomment-2041016095
> the checkstyle is because it wants
`./hadoop-common-project/hadoop-common/src/main/java/org/apache/hadoop/hdfs/protocol/datatransfer/`
to have a package class describing it
>
> proposed
>
> * add one
>
> * tag stability as stable
>
> * tag scope as LimitedPrivate("yarn")
>
> * mention original and purpose
>
>
> then in the hdfs version of the same package, add a mention that the tuple
class has been moved to hadoop common, with a cross reference to the class
1) package class is already added `package
org.apache.hadoop.hdfs.protocol.datatransfer`; not sure why it complains.
2) Ack for adding `@InterfaceStability.Stable`
3) @steveloughran - why * tag scope as LimitedPrivate("yarn") ? I see the
class IOStreamPair being used by both Yarn and HDFS.
4) We don't have the package in HDFS module anymore.
--
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: common-issues-unsubscr...@hadoop.apache.org
For queries about this service, please contact Infrastructure at:
us...@infra.apache.org
---------------------------------------------------------------------
To unsubscribe, e-mail: common-issues-unsubscr...@hadoop.apache.org
For additional commands, e-mail: common-issues-h...@hadoop.apache.org
