steveloughran commented on PR #4996:
URL: https://github.com/apache/hadoop/pull/4996#issuecomment-1753244196

   test failure seems unrelated. 
   style
   ```
   
./hadoop-common-project/hadoop-common/src/main/java/org/apache/hadoop/ipc/protocolPB/GenericRefreshProtocolClientSideTranslatorPB.java:62:
      .setIdentifier(identifier): '.' has incorrect indentation level 6, 
expected level should be 8. [Indentation]
   
./hadoop-common-project/hadoop-common/src/main/java/org/apache/hadoop/ipc/protocolPB/GenericRefreshProtocolClientSideTranslatorPB.java:63:
      .addAllArgs(argList): '.' has incorrect indentation level 6, expected 
level should be 8. [Indentation]
   
./hadoop-common-project/hadoop-common/src/main/java/org/apache/hadoop/ipc/protocolPB/GenericRefreshProtocolClientSideTranslatorPB.java:64:
      .build();: '.' has incorrect indentation level 6, expected level should 
be 8. [Indentation]
   
./hadoop-common-project/hadoop-common/src/main/java/org/apache/hadoop/ipc/protocolPB/GenericRefreshProtocolClientSideTranslatorPB.java:66:
    GenericRefreshResponseCollectionProto resp = ipc(() -> 
rpcProxy.refresh(NULL_CONTROLLER, request));: Line is longer than 100 
characters (found 103). [LineLength]
   
./hadoop-common-project/hadoop-common/src/main/java/org/apache/hadoop/tools/protocolPB/GetUserMappingsProtocolClientSideTranslatorPB.java:27:import
 org.apache.hadoop.ipc.internal.ShadedProtobufHelper;:8: Unused import - 
org.apache.hadoop.ipc.internal.ShadedProtobufHelper. [UnusedImports]
   
./hadoop-common-project/hadoop-common/src/main/java/org/apache/hadoop/tools/protocolPB/GetUserMappingsProtocolClientSideTranslatorPB.java:33:import
 org.apache.hadoop.thirdparty.protobuf.ServiceException;:8: Unused import - 
org.apache.hadoop.thirdparty.protobuf.ServiceException. [UnusedImports]
   
./hadoop-hdfs-project/hadoop-hdfs-client/src/main/java/org/apache/hadoop/hdfs/protocolPB/ClientDatanodeProtocolTranslatorPB.java:243:
      ipc(() -> rpcProxy.shutdownDatanode(NULL_CONTROLLER, request));: 'method 
def' child has incorrect indentation level 6, expected level should be 4. 
[Indentation]
   ```
   javadocs
   ```
   [ERROR] in your comments are valid in HTML5, and remove the -html4 option.
   [ERROR] 
/home/jenkins/jenkins-home/workspace/hadoop-multibranch_PR-4996/ubuntu-focal/src/hadoop-common-project/hadoop-common/src/main/java/org/apache/hadoop/ipc/ProtobufHelper.java:39:
 error: unknown tag: Deprecated:
   [ERROR]  * @Deprecated: hadoop code MUST use {@link ShadedProtobufHelper}.
   ```
   
   maybe its the :


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

Reply via email to