virajjasani commented on PR #5460: URL: https://github.com/apache/hadoop/pull/5460#issuecomment-1463012342
> I also don't understand why this style warning is appearing, as its not a new package folder, and it has plenty of other classes in it already. I think it's appearing because it was not resolved earlier or it was ignored. For instance, I see the last time the module was touched was by this PR #4252 and in the last QA result, I see checkstyle issues, but of course since the PR is quite old, we can't see what the error was (it reports 404 for the checkstyle report page). I guess it's not that important either. > This isn't working. I added what you suggested, but still the same error. Strange, I think this one should hopefully work: ``` --- a/hadoop-client-modules/hadoop-client-api/pom.xml +++ b/hadoop-client-modules/hadoop-client-api/pom.xml @@ -126,6 +126,12 @@ <exclude>org/apache/hadoop/yarn/client/api/package-info.class</exclude> </excludes> </filter> + <filter> + <artifact>org.apache.hadoop:*</artifact> + <excludes> + <exclude>org/apache/hadoop/hdfs/server/protocol/package-info.class</exclude> + </excludes> + </filter> </filters> <relocations> <relocation> ``` -- 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