yangjiandan commented on PR #7352: URL: https://github.com/apache/hadoop/pull/7352#issuecomment-2689455665
> @cnauroth I Couldn't find it -but I'm sure there was a migration, or at least a WiP one > > * recreate package in new source tree > * move the java class > * update javadocs to explain why, add package-info.java > * in original tree, update package-info.java > * same for tests > > result is very low risk, minimal diff patch @steveloughran Thanks for your feedback! Your idea is great, and I understand your concern about the scope of this patch. However, my main concern is that creating the org.apache.hadoop.yarn.util package under hadoop-common-project might introduce even greate confusion. Hadoop Common is primarily intended for shared components across the Hadoop ecosystem, and containing **YARN** in package path could mislead developers about their intended scope and usage, potentially affecting the overall structure. In constract, the current approach could ensure better modular clarity and consistency. -- 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]
