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: [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]
