steveloughran commented on PR #7128:
URL: https://github.com/apache/hadoop/pull/7128#issuecomment-2457797959
close. Four little checkstyles to fix, all minor
```
./hadoop-common-project/hadoop-common/src/test/java/org/apache/hadoop/constants/ConfigConstants.java:21:public
interface ConfigConstants {: interfaces should describe a type and hence have
methods. [InterfaceIsType]
./hadoop-common-project/hadoop-common/src/test/java/org/apache/hadoop/constants/ConfigConstants.java:27:
public static final String CONFIG_AVRO_SERIALIZABLE_PACKAGES =
"org.apache.avro.SERIALIZABLE_PACKAGES";: 'member def modifier' has incorrect
indentation level 3, expected level should be 2. [Indentation]
./hadoop-common-project/hadoop-common/src/test/java/org/apache/hadoop/constants/ConfigConstants.java:27:
public static final String CONFIG_AVRO_SERIALIZABLE_PACKAGES =
"org.apache.avro.SERIALIZABLE_PACKAGES";: Line is longer than 100 characters
(found 106). [LineLength]
./hadoop-common-project/hadoop-common/src/test/java/org/apache/hadoop/constants/ConfigConstants.java:27:
public static final String CONFIG_AVRO_SERIALIZABLE_PACKAGES =
"org.apache.avro.SERIALIZABLE_PACKAGES";:4: Redundant 'public' modifier.
[RedundantModifier]
./hadoop-mapreduce-project/hadoop-mapreduce-client/hadoop-mapreduce-client-core/src/main/java/org/apache/hadoop/mapreduce/jobhistory/JobQueueChangeEvent.java:23:import
org.apache.avro.util.Utf8;:8: Unused import - org.apache.avro.util.Utf8.
[UnusedImports]
```
+1 pending these being fixed
--
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]
