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


   :broken_heart: **-1 overall**
   
   
   
   
   
   
   | Vote | Subsystem | Runtime |  Logfile | Comment |
   |:----:|----------:|--------:|:--------:|:-------:|
   | +0 :ok: |  reexec  |   0m 55s |  |  Docker mode activated.  |
   |||| _ Prechecks _ |
   | +1 :green_heart: |  dupname  |   0m  2s |  |  No case conflicting files 
found.  |
   | +0 :ok: |  codespell  |   0m  0s |  |  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 41 new or modified test files.  |
   |||| _ trunk Compile Tests _ |
   | +1 :green_heart: |  mvninstall  |  31m  6s |  |  trunk passed  |
   | +1 :green_heart: |  compile  |   0m 40s |  |  trunk passed with JDK 
Ubuntu-11.0.11+9-Ubuntu-0ubuntu2.20.04  |
   | +1 :green_heart: |  compile  |   0m 37s |  |  trunk passed with JDK 
Private Build-1.8.0_292-8u292-b10-0ubuntu1~20.04-b10  |
   | +1 :green_heart: |  checkstyle  |   0m 30s |  |  trunk passed  |
   | +1 :green_heart: |  mvnsite  |   0m 41s |  |  trunk passed  |
   | +1 :green_heart: |  javadoc  |   0m 34s |  |  trunk passed with JDK 
Ubuntu-11.0.11+9-Ubuntu-0ubuntu2.20.04  |
   | +1 :green_heart: |  javadoc  |   0m 31s |  |  trunk passed with JDK 
Private Build-1.8.0_292-8u292-b10-0ubuntu1~20.04-b10  |
   | +1 :green_heart: |  spotbugs  |   1m  1s |  |  trunk passed  |
   | +1 :green_heart: |  shadedclient  |  14m 46s |  |  branch has no errors 
when building and testing our client artifacts.  |
   | -0 :warning: |  patch  |  15m  4s |  |  Used diff version of patch file. 
Binary files and potentially other changes not applied. Please rebase and 
squash commits if necessary.  |
   |||| _ Patch Compile Tests _ |
   | +1 :green_heart: |  mvninstall  |   0m 35s |  |  the patch passed  |
   | +1 :green_heart: |  compile  |   0m 33s |  |  the patch passed with JDK 
Ubuntu-11.0.11+9-Ubuntu-0ubuntu2.20.04  |
   | -1 :x: |  javac  |   0m 33s | 
[/results-compile-javac-hadoop-tools_hadoop-azure-jdkUbuntu-11.0.11+9-Ubuntu-0ubuntu2.20.04.txt](https://ci-hadoop.apache.org/job/hadoop-multibranch/job/PR-3309/4/artifact/out/results-compile-javac-hadoop-tools_hadoop-azure-jdkUbuntu-11.0.11+9-Ubuntu-0ubuntu2.20.04.txt)
 |  hadoop-tools_hadoop-azure-jdkUbuntu-11.0.11+9-Ubuntu-0ubuntu2.20.04 with 
JDK Ubuntu-11.0.11+9-Ubuntu-0ubuntu2.20.04 generated 3 new + 24 unchanged - 2 
fixed = 27 total (was 26)  |
   | +1 :green_heart: |  compile  |   0m 29s |  |  the patch passed with JDK 
Private Build-1.8.0_292-8u292-b10-0ubuntu1~20.04-b10  |
   | -1 :x: |  javac  |   0m 29s | 
[/results-compile-javac-hadoop-tools_hadoop-azure-jdkPrivateBuild-1.8.0_292-8u292-b10-0ubuntu1~20.04-b10.txt](https://ci-hadoop.apache.org/job/hadoop-multibranch/job/PR-3309/4/artifact/out/results-compile-javac-hadoop-tools_hadoop-azure-jdkPrivateBuild-1.8.0_292-8u292-b10-0ubuntu1~20.04-b10.txt)
 |  
hadoop-tools_hadoop-azure-jdkPrivateBuild-1.8.0_292-8u292-b10-0ubuntu1~20.04-b10
 with JDK Private Build-1.8.0_292-8u292-b10-0ubuntu1~20.04-b10 generated 2 new 
+ 19 unchanged - 1 fixed = 21 total (was 20)  |
   | +1 :green_heart: |  blanks  |   0m  0s |  |  The patch has no blanks 
issues.  |
   | -0 :warning: |  checkstyle  |   0m 20s | 
[/results-checkstyle-hadoop-tools_hadoop-azure.txt](https://ci-hadoop.apache.org/job/hadoop-multibranch/job/PR-3309/4/artifact/out/results-checkstyle-hadoop-tools_hadoop-azure.txt)
 |  hadoop-tools/hadoop-azure: The patch generated 240 new + 15 unchanged - 4 
fixed = 255 total (was 19)  |
   | +1 :green_heart: |  mvnsite  |   0m 32s |  |  the patch passed  |
   | +1 :green_heart: |  xml  |   0m  1s |  |  The patch has no ill-formed XML 
file.  |
   | -1 :x: |  javadoc  |   0m 24s | 
[/results-javadoc-javadoc-hadoop-tools_hadoop-azure-jdkUbuntu-11.0.11+9-Ubuntu-0ubuntu2.20.04.txt](https://ci-hadoop.apache.org/job/hadoop-multibranch/job/PR-3309/4/artifact/out/results-javadoc-javadoc-hadoop-tools_hadoop-azure-jdkUbuntu-11.0.11+9-Ubuntu-0ubuntu2.20.04.txt)
 |  hadoop-tools_hadoop-azure-jdkUbuntu-11.0.11+9-Ubuntu-0ubuntu2.20.04 with 
JDK Ubuntu-11.0.11+9-Ubuntu-0ubuntu2.20.04 generated 3 new + 15 unchanged - 0 
fixed = 18 total (was 15)  |
   | -1 :x: |  javadoc  |   0m 21s | 
[/results-javadoc-javadoc-hadoop-tools_hadoop-azure-jdkPrivateBuild-1.8.0_292-8u292-b10-0ubuntu1~20.04-b10.txt](https://ci-hadoop.apache.org/job/hadoop-multibranch/job/PR-3309/4/artifact/out/results-javadoc-javadoc-hadoop-tools_hadoop-azure-jdkPrivateBuild-1.8.0_292-8u292-b10-0ubuntu1~20.04-b10.txt)
 |  
hadoop-tools_hadoop-azure-jdkPrivateBuild-1.8.0_292-8u292-b10-0ubuntu1~20.04-b10
 with JDK Private Build-1.8.0_292-8u292-b10-0ubuntu1~20.04-b10 generated 3 new 
+ 15 unchanged - 0 fixed = 18 total (was 15)  |
   | -1 :x: |  spotbugs  |   1m  9s | 
[/new-spotbugs-hadoop-tools_hadoop-azure.html](https://ci-hadoop.apache.org/job/hadoop-multibranch/job/PR-3309/4/artifact/out/new-spotbugs-hadoop-tools_hadoop-azure.html)
 |  hadoop-tools/hadoop-azure generated 14 new + 0 unchanged - 0 fixed = 14 
total (was 0)  |
   | +1 :green_heart: |  shadedclient  |  14m  4s |  |  patch has no errors 
when building and testing our client artifacts.  |
   |||| _ Other Tests _ |
   | +1 :green_heart: |  unit  |   1m 55s |  |  hadoop-azure in the patch 
passed.  |
   | +1 :green_heart: |  asflicense  |   0m 32s |  |  The patch does not 
generate ASF License warnings.  |
   |  |   |  73m 46s |  |  |
   
   
   | Reason | Tests |
   |-------:|:------|
   | SpotBugs | module:hadoop-tools/hadoop-azure |
   |  |  Dead store to sasTokenForReuse in 
org.apache.hadoop.fs.azurebfs.services.AbfsClient.fastPathClose(String, String, 
AbfsFastpathSessionInfo, TracingContext)  At 
AbfsClient.java:org.apache.hadoop.fs.azurebfs.services.AbfsClient.fastPathClose(String,
 String, AbfsFastpathSessionInfo, TracingContext)  At AbfsClient.java:[line 
802] |
   |  |  Dead store to sasTokenForReuse in 
org.apache.hadoop.fs.azurebfs.services.AbfsClient.fastPathOpen(String, String, 
AbfsFastpathSessionInfo, TracingContext)  At 
AbfsClient.java:org.apache.hadoop.fs.azurebfs.services.AbfsClient.fastPathOpen(String,
 String, AbfsFastpathSessionInfo, TracingContext)  At AbfsClient.java:[line 
785] |
   |  |  Boxing/unboxing to parse a primitive new 
org.apache.hadoop.fs.azurebfs.services.AbfsFastpathConnection(AbfsRestOperationType,
 URL, String, AuthType, String, List, AbfsFastpathSessionInfo)  At 
AbfsFastpathConnection.java:new 
org.apache.hadoop.fs.azurebfs.services.AbfsFastpathConnection(AbfsRestOperationType,
 URL, String, AuthType, String, List, AbfsFastpathSessionInfo)  At 
AbfsFastpathConnection.java:[line 94] |
   |  |  Nullcheck of closeResponse at line 271 of value previously 
dereferenced in 
org.apache.hadoop.fs.azurebfs.services.AbfsFastpathConnection.processFastpathCloseResponse()
  At AbfsFastpathConnection.java:271 of value previously dereferenced in 
org.apache.hadoop.fs.azurebfs.services.AbfsFastpathConnection.processFastpathCloseResponse()
  At AbfsFastpathConnection.java:[line 271] |
   |  |  Nullcheck of openResponse at line 212 of value previously dereferenced 
in 
org.apache.hadoop.fs.azurebfs.services.AbfsFastpathConnection.processFastpathOpenResponse()
  At AbfsFastpathConnection.java:212 of value previously dereferenced in 
org.apache.hadoop.fs.azurebfs.services.AbfsFastpathConnection.processFastpathOpenResponse()
  At AbfsFastpathConnection.java:[line 212] |
   |  |  Nullcheck of readResponse at line 241 of value previously dereferenced 
in 
org.apache.hadoop.fs.azurebfs.services.AbfsFastpathConnection.processFastpathReadResponse(byte[],
 int, int)  At AbfsFastpathConnection.java:241 of value previously dereferenced 
in 
org.apache.hadoop.fs.azurebfs.services.AbfsFastpathConnection.processFastpathReadResponse(byte[],
 int, int)  At AbfsFastpathConnection.java:[line 241] |
   |  |  Dead store to scheduledFuture in 
org.apache.hadoop.fs.azurebfs.services.AbfsFastpathSession.updateAbfsFastpathSessionToken(String,
 OffsetDateTime)  At 
AbfsFastpathSession.java:org.apache.hadoop.fs.azurebfs.services.AbfsFastpathSession.updateAbfsFastpathSessionToken(String,
 OffsetDateTime)  At AbfsFastpathSession.java:[line 163] |
   |  |  
org.apache.hadoop.fs.azurebfs.services.AbfsHttpConnection.processResponse(byte[],
 int, int) may fail to close stream  At AbfsHttpConnection.java:to close stream 
 At AbfsHttpConnection.java:[line 224] |
   |  |  
org.apache.hadoop.fs.azurebfs.services.AbfsHttpOperation.getResponseContentBuffer()
 may expose internal representation by returning 
AbfsHttpOperation.responseContentBuffer  At AbfsHttpOperation.java:by returning 
AbfsHttpOperation.responseContentBuffer  At AbfsHttpOperation.java:[line 177] |
   |  |  
org.apache.hadoop.fs.azurebfs.services.AbfsRestIODataParameters.getBuffer() may 
expose internal representation by returning AbfsRestIODataParameters.buffer  At 
AbfsRestIODataParameters.java:by returning AbfsRestIODataParameters.buffer  At 
AbfsRestIODataParameters.java:[line 48] |
   |  |  new 
org.apache.hadoop.fs.azurebfs.services.AbfsRestIODataParameters(byte[], int, 
int) may expose internal representation by storing an externally mutable object 
into AbfsRestIODataParameters.buffer  At AbfsRestIODataParameters.java:expose 
internal representation by storing an externally mutable object into 
AbfsRestIODataParameters.buffer  At AbfsRestIODataParameters.java:[line 41] |
   |  |  new 
org.apache.hadoop.fs.azurebfs.services.AbfsRestIODataParameters(byte[], int, 
int, String) may expose internal representation by storing an externally 
mutable object into AbfsRestIODataParameters.buffer  At 
AbfsRestIODataParameters.java:may expose internal representation by storing an 
externally mutable object into AbfsRestIODataParameters.buffer  At 
AbfsRestIODataParameters.java:[line 32] |
   |  |  Possible null pointer dereference of httpOperation in 
org.apache.hadoop.fs.azurebfs.services.AbfsRestOperation.executeHttpOperation(int,
 TracingContext) on exception path  Dereferenced at 
AbfsRestOperation.java:httpOperation in 
org.apache.hadoop.fs.azurebfs.services.AbfsRestOperation.executeHttpOperation(int,
 TracingContext) on exception path  Dereferenced at 
AbfsRestOperation.java:[line 350] |
   |  |  Nullcheck of httpOperation at line 385 of value previously 
dereferenced in 
org.apache.hadoop.fs.azurebfs.services.AbfsRestOperation.executeHttpOperation(int,
 TracingContext)  At AbfsRestOperation.java:385 of value previously 
dereferenced in 
org.apache.hadoop.fs.azurebfs.services.AbfsRestOperation.executeHttpOperation(int,
 TracingContext)  At AbfsRestOperation.java:[line 350] |
   
   
   | Subsystem | Report/Notes |
   |----------:|:-------------|
   | Docker | ClientAPI=1.41 ServerAPI=1.41 base: 
https://ci-hadoop.apache.org/job/hadoop-multibranch/job/PR-3309/4/artifact/out/Dockerfile
 |
   | GITHUB PR | https://github.com/apache/hadoop/pull/3309 |
   | Optional Tests | dupname asflicense compile javac javadoc mvninstall 
mvnsite unit shadedclient codespell xml spotbugs checkstyle |
   | uname | Linux b339fb9d8c31 4.15.0-151-generic #157-Ubuntu SMP Fri Jul 9 
23:07:57 UTC 2021 x86_64 x86_64 x86_64 GNU/Linux |
   | Build tool | maven |
   | Personality | dev-support/bin/hadoop.sh |
   | git revision | trunk / 19fdcda3d0b7052911cf983fc6a0a6429e287386 |
   | Default Java | Private Build-1.8.0_292-8u292-b10-0ubuntu1~20.04-b10 |
   | Multi-JDK versions | 
/usr/lib/jvm/java-11-openjdk-amd64:Ubuntu-11.0.11+9-Ubuntu-0ubuntu2.20.04 
/usr/lib/jvm/java-8-openjdk-amd64:Private 
Build-1.8.0_292-8u292-b10-0ubuntu1~20.04-b10 |
   |  Test Results | 
https://ci-hadoop.apache.org/job/hadoop-multibranch/job/PR-3309/4/testReport/ |
   | Max. process+thread count | 692 (vs. ulimit of 5500) |
   | modules | C: hadoop-tools/hadoop-azure U: hadoop-tools/hadoop-azure |
   | Console output | 
https://ci-hadoop.apache.org/job/hadoop-multibranch/job/PR-3309/4/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.

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