hadoop-yetus commented on pull request #2513:
URL: https://github.com/apache/hadoop/pull/2513#issuecomment-844163227


   :broken_heart: **-1 overall**
   
   
   
   
   
   
   | Vote | Subsystem | Runtime |  Logfile | Comment |
   |:----:|----------:|--------:|:--------:|:-------:|
   | +0 :ok: |  reexec  |   0m 36s |  |  Docker mode activated.  |
   |||| _ Prechecks _ |
   | +1 :green_heart: |  dupname  |   0m  4s |  |  No case conflicting files 
found.  |
   | +0 :ok: |  codespell  |   0m  4s |  |  codespell was not available.  |
   | +1 :green_heart: |  @author  |   0m  0s |  |  The patch does not contain 
any @author tags.  |
   | +1 :green_heart: |  test4tests  |   0m  0s |  |  The patch appears to 
include 69 new or modified test files.  |
   |||| _ trunk Compile Tests _ |
   | +1 :green_heart: |  mvninstall  |  34m 12s |  |  trunk passed  |
   | +1 :green_heart: |  compile  |   3m  6s |  |  trunk passed with JDK 
Ubuntu-11.0.10+9-Ubuntu-0ubuntu1.20.04  |
   | +1 :green_heart: |  compile  |   2m 22s |  |  trunk passed with JDK 
Private Build-1.8.0_282-8u282-b08-0ubuntu1~20.04-b08  |
   | +1 :green_heart: |  checkstyle  |   0m 47s |  |  trunk passed  |
   | +1 :green_heart: |  mvnsite  |   2m 25s |  |  trunk passed  |
   | +1 :green_heart: |  javadoc  |   1m 50s |  |  trunk passed with JDK 
Ubuntu-11.0.10+9-Ubuntu-0ubuntu1.20.04  |
   | +1 :green_heart: |  javadoc  |   1m 32s |  |  trunk passed with JDK 
Private Build-1.8.0_282-8u282-b08-0ubuntu1~20.04-b08  |
   | +1 :green_heart: |  spotbugs  |   5m 29s |  |  trunk passed  |
   | +1 :green_heart: |  shadedclient  |  18m 36s |  |  branch has no errors 
when building and testing our client artifacts.  |
   |||| _ Patch Compile Tests _ |
   | +0 :ok: |  mvndep  |   0m 28s |  |  Maven dependency ordering for patch  |
   | +1 :green_heart: |  mvninstall  |   2m 53s |  |  the patch passed  |
   | +1 :green_heart: |  compile  |   3m  4s |  |  the patch passed with JDK 
Ubuntu-11.0.10+9-Ubuntu-0ubuntu1.20.04  |
   | +1 :green_heart: |  javac  |   3m  4s |  |  the patch passed  |
   | +1 :green_heart: |  compile  |   2m 19s |  |  the patch passed with JDK 
Private Build-1.8.0_282-8u282-b08-0ubuntu1~20.04-b08  |
   | +1 :green_heart: |  javac  |   2m 19s |  |  the patch passed  |
   | +1 :green_heart: |  blanks  |   0m  0s |  |  The patch has no blanks 
issues.  |
   | +1 :green_heart: |  checkstyle  |   0m 40s |  |  the patch passed  |
   | +1 :green_heart: |  mvnsite  |   2m 46s |  |  the patch passed  |
   | +1 :green_heart: |  xml  |   0m  6s |  |  The patch has no ill-formed XML 
file.  |
   | +1 :green_heart: |  javadoc  |   2m  9s |  |  the patch passed with JDK 
Ubuntu-11.0.10+9-Ubuntu-0ubuntu1.20.04  |
   | +1 :green_heart: |  javadoc  |   1m 51s |  |  the patch passed with JDK 
Private Build-1.8.0_282-8u282-b08-0ubuntu1~20.04-b08  |
   | -1 :x: |  spotbugs  |   5m 52s | 
[/new-spotbugs-hadoop-tools.html](https://ci-hadoop.apache.org/job/hadoop-multibranch/job/PR-2513/1/artifact/out/new-spotbugs-hadoop-tools.html)
 |  hadoop-tools generated 15 new + 0 unchanged - 0 fixed = 15 total (was 0)  |
   | +1 :green_heart: |  shadedclient  |  19m  2s |  |  patch has no errors 
when building and testing our client artifacts.  |
   |||| _ Other Tests _ |
   | +1 :green_heart: |  unit  |   0m 36s |  |  hadoop-runc in the patch 
passed.  |
   | -1 :x: |  unit  |  83m 27s | 
[/patch-unit-hadoop-tools.txt](https://ci-hadoop.apache.org/job/hadoop-multibranch/job/PR-2513/1/artifact/out/patch-unit-hadoop-tools.txt)
 |  hadoop-tools in the patch passed.  |
   | +1 :green_heart: |  asflicense  |   0m 42s |  |  The patch does not 
generate ASF License warnings.  |
   |  |   | 190m 21s |  |  |
   
   
   | Reason | Tests |
   |-------:|:------|
   | SpotBugs | module:hadoop-tools |
   |  |  Class org.apache.hadoop.runc.docker.auth.BearerScheme defines 
non-transient non-serializable instance field client  In 
BearerScheme.java:instance field client  In BearerScheme.java |
   |  |  org.apache.hadoop.runc.squashfs.data.DataBlock.getData() may expose 
internal representation by returning DataBlock.data  At DataBlock.java:by 
returning DataBlock.data  At DataBlock.java:[line 28] |
   |  |  new org.apache.hadoop.runc.squashfs.data.DataBlock(byte[], int, int) 
may expose internal representation by storing an externally mutable object into 
DataBlock.data  At DataBlock.java:expose internal representation by storing an 
externally mutable object into DataBlock.data  At DataBlock.java:[line 44] |
   |  |  
org.apache.hadoop.runc.squashfs.directory.DirectoryBuilder$Entry.getName() may 
expose internal representation by returning DirectoryBuilder$Entry.name  At 
DirectoryBuilder.java:by returning DirectoryBuilder$Entry.name  At 
DirectoryBuilder.java:[line 146] |
   |  |  new 
org.apache.hadoop.runc.squashfs.directory.DirectoryBuilder$Entry(int, int, 
short, short, byte[]) may expose internal representation by storing an 
externally mutable object into DirectoryBuilder$Entry.name  At 
DirectoryBuilder.java:byte[]) may expose internal representation by storing an 
externally mutable object into DirectoryBuilder$Entry.name  At 
DirectoryBuilder.java:[line 134] |
   |  |  org.apache.hadoop.runc.squashfs.directory.DirectoryEntry.getName() may 
expose internal representation by returning DirectoryEntry.name  At 
DirectoryEntry.java:by returning DirectoryEntry.name  At 
DirectoryEntry.java:[line 80] |
   |  |  org.apache.hadoop.runc.squashfs.inode.BasicFileINode.getBlockSizes() 
may expose internal representation by returning BasicFileINode.blockSizes  At 
BasicFileINode.java:by returning BasicFileINode.blockSizes  At 
BasicFileINode.java:[line 187] |
   |  |  
org.apache.hadoop.runc.squashfs.inode.BasicFileINode.setBlockSizes(int[]) may 
expose internal representation by storing an externally mutable object into 
BasicFileINode.blockSizes  At BasicFileINode.java:by storing an externally 
mutable object into BasicFileINode.blockSizes  At BasicFileINode.java:[line 
192] |
   |  |  
org.apache.hadoop.runc.squashfs.inode.BasicSymlinkINode.getTargetPath() may 
expose internal representation by returning BasicSymlinkINode.targetPath  At 
BasicSymlinkINode.java:by returning BasicSymlinkINode.targetPath  At 
BasicSymlinkINode.java:[line 68] |
   |  |  
org.apache.hadoop.runc.squashfs.inode.ExtendedFileINode.getBlockSizes() may 
expose internal representation by returning ExtendedFileINode.blockSizes  At 
ExtendedFileINode.java:by returning ExtendedFileINode.blockSizes  At 
ExtendedFileINode.java:[line 85] |
   |  |  
org.apache.hadoop.runc.squashfs.inode.ExtendedFileINode.setBlockSizes(int[]) 
may expose internal representation by storing an externally mutable object into 
ExtendedFileINode.blockSizes  At ExtendedFileINode.java:by storing an 
externally mutable object into ExtendedFileINode.blockSizes  At 
ExtendedFileINode.java:[line 90] |
   |  |  
org.apache.hadoop.runc.squashfs.inode.ExtendedSymlinkINode.getTargetPath() may 
expose internal representation by returning ExtendedSymlinkINode.targetPath  At 
ExtendedSymlinkINode.java:by returning ExtendedSymlinkINode.targetPath  At 
ExtendedSymlinkINode.java:[line 53] |
   |  |  new 
org.apache.hadoop.runc.squashfs.metadata.MemoryMetadataBlockReader(int, 
SuperBlock, byte[], int, int) may expose internal representation by storing an 
externally mutable object into MemoryMetadataBlockReader.data  At 
MemoryMetadataBlockReader.java:int) may expose internal representation by 
storing an externally mutable object into MemoryMetadataBlockReader.data  At 
MemoryMetadataBlockReader.java:[line 45] |
   |  |  org.apache.hadoop.runc.squashfs.metadata.MetadataBlock.getData() may 
expose internal representation by returning MetadataBlock.data  At 
MetadataBlock.java:by returning MetadataBlock.data  At MetadataBlock.java:[line 
63] |
   |  |  new 
org.apache.hadoop.runc.squashfs.table.MemoryTableReader(SuperBlock, byte[], 
int, int) may expose internal representation by storing an externally mutable 
object into MemoryTableReader.data  At MemoryTableReader.java:may expose 
internal representation by storing an externally mutable object into 
MemoryTableReader.data  At MemoryTableReader.java:[line 41] |
   | Failed junit tests | hadoop.tools.fedbalance.TestDistCpProcedure |
   |   | hadoop.tools.fedbalance.procedure.TestBalanceProcedureScheduler |
   
   
   | Subsystem | Report/Notes |
   |----------:|:-------------|
   | Docker | ClientAPI=1.41 ServerAPI=1.41 base: 
https://ci-hadoop.apache.org/job/hadoop-multibranch/job/PR-2513/1/artifact/out/Dockerfile
 |
   | GITHUB PR | https://github.com/apache/hadoop/pull/2513 |
   | Optional Tests | dupname asflicense codespell xml compile javac javadoc 
mvninstall mvnsite unit shadedclient spotbugs checkstyle |
   | uname | Linux 6ea630679121 4.15.0-60-generic #67-Ubuntu SMP Thu Aug 22 
16:55:30 UTC 2019 x86_64 x86_64 x86_64 GNU/Linux |
   | Build tool | maven |
   | Personality | dev-support/bin/hadoop.sh |
   | git revision | trunk / 24f004b2f1627224b9c57c0a80649633d9e6da88 |
   | Default Java | Private Build-1.8.0_282-8u282-b08-0ubuntu1~20.04-b08 |
   | Multi-JDK versions | 
/usr/lib/jvm/java-11-openjdk-amd64:Ubuntu-11.0.10+9-Ubuntu-0ubuntu1.20.04 
/usr/lib/jvm/java-8-openjdk-amd64:Private 
Build-1.8.0_282-8u282-b08-0ubuntu1~20.04-b08 |
   |  Test Results | 
https://ci-hadoop.apache.org/job/hadoop-multibranch/job/PR-2513/1/testReport/ |
   | Max. process+thread count | 1096 (vs. ulimit of 5500) |
   | modules | C: hadoop-tools/hadoop-runc hadoop-tools U: hadoop-tools |
   | Console output | 
https://ci-hadoop.apache.org/job/hadoop-multibranch/job/PR-2513/1/console |
   | versions | git=2.25.1 maven=3.6.3 spotbugs=4.2.2 |
   | Powered by | Apache Yetus 0.14.0-SNAPSHOT https://yetus.apache.org |
   
   
   This message was automatically generated.
   
   


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

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