You are viewing a plain text version of this content. The canonical link for it is here.
Posted to issues@tez.apache.org by GitBox <gi...@apache.org> on 2022/05/30 16:10:50 UTC

[GitHub] [tez] tez-yetus commented on pull request #212: TEZ-4038: Add a /prof profiler endpoint like HiveServer2 has

tez-yetus commented on PR #212:
URL: https://github.com/apache/tez/pull/212#issuecomment-1141316590

   :broken_heart: **-1 overall**
   
   
   
   
   
   
   | Vote | Subsystem | Runtime | Comment |
   |:----:|----------:|--------:|:--------|
   | +0 :ok: |  reexec  |  12m 56s |  Docker mode activated.  |
   ||| _ Prechecks _ |
   | +1 :green_heart: |  dupname  |   0m  0s |  No case conflicting files found.  |
   | +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 1 new or modified test files.  |
   ||| _ master Compile Tests _ |
   | +0 :ok: |  mvndep  |   5m 56s |  Maven dependency ordering for branch  |
   | +1 :green_heart: |  mvninstall  |  10m 26s |  master passed  |
   | +1 :green_heart: |  compile  |   1m 48s |  master passed with JDK Private Build-11.0.15+10-Ubuntu-0ubuntu0.20.04.1  |
   | +1 :green_heart: |  compile  |   1m 36s |  master passed with JDK Private Build-1.8.0_312-8u312-b07-0ubuntu1~20.04-b07  |
   | +1 :green_heart: |  checkstyle  |   1m 29s |  master passed  |
   | +1 :green_heart: |  javadoc  |   1m 46s |  master passed with JDK Private Build-11.0.15+10-Ubuntu-0ubuntu0.20.04.1  |
   | +1 :green_heart: |  javadoc  |   1m 23s |  master passed with JDK Private Build-1.8.0_312-8u312-b07-0ubuntu1~20.04-b07  |
   | +0 :ok: |  spotbugs  |   0m 45s |  Used deprecated FindBugs config; considering switching to SpotBugs.  |
   | +1 :green_heart: |  findbugs  |   2m 58s |  master passed  |
   | -0 :warning: |  patch  |   1m  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 _ |
   | +0 :ok: |  mvndep  |   0m  9s |  Maven dependency ordering for patch  |
   | +1 :green_heart: |  mvninstall  |   1m  1s |  the patch passed  |
   | +1 :green_heart: |  compile  |   1m  9s |  the patch passed with JDK Private Build-11.0.15+10-Ubuntu-0ubuntu0.20.04.1  |
   | +1 :green_heart: |  javac  |   1m  9s |  the patch passed  |
   | +1 :green_heart: |  compile  |   0m 57s |  the patch passed with JDK Private Build-1.8.0_312-8u312-b07-0ubuntu1~20.04-b07  |
   | +1 :green_heart: |  javac  |   0m 57s |  the patch passed  |
   | -0 :warning: |  checkstyle  |   0m 16s |  tez-common: The patch generated 285 new + 0 unchanged - 0 fixed = 285 total (was 0)  |
   | -0 :warning: |  checkstyle  |   0m 12s |  tez-tests: The patch generated 1 new + 0 unchanged - 0 fixed = 1 total (was 0)  |
   | +1 :green_heart: |  whitespace  |   0m  0s |  The patch has no whitespace issues.  |
   | +1 :green_heart: |  javadoc  |   0m 50s |  the patch passed with JDK Private Build-11.0.15+10-Ubuntu-0ubuntu0.20.04.1  |
   | +1 :green_heart: |  javadoc  |   0m 48s |  the patch passed with JDK Private Build-1.8.0_312-8u312-b07-0ubuntu1~20.04-b07  |
   | -1 :x: |  findbugs  |   0m 38s |  tez-common generated 3 new + 0 unchanged - 0 fixed = 3 total (was 0)  |
   ||| _ Other Tests _ |
   | +1 :green_heart: |  unit  |   0m 32s |  tez-common in the patch passed.  |
   | +1 :green_heart: |  unit  |   4m 32s |  tez-dag in the patch passed.  |
   | -1 :x: |  unit  |  38m 44s |  tez-tests in the patch failed.  |
   | -1 :x: |  asflicense  |   0m 35s |  The patch generated 2 ASF License warnings.  |
   |  |   |  94m 20s |   |
   
   
   | Reason | Tests |
   |-------:|:------|
   | FindBugs | module:tez-common |
   |  |  HTTP parameter directly written to HTTP header output in org.apache.tez.common.web.ProfileOutputServlet.doGet(HttpServletRequest, HttpServletResponse)  At ProfileOutputServlet.java:HTTP header output in org.apache.tez.common.web.ProfileOutputServlet.doGet(HttpServletRequest, HttpServletResponse)  At ProfileOutputServlet.java:[line 43] |
   |  |  Relative path traversal in org.apache.tez.common.web.ProfileOutputServlet.doGet(HttpServletRequest, HttpServletResponse)  At ProfileOutputServlet.java:HttpServletResponse)  At ProfileOutputServlet.java:[line 30] |
   |  |  HTTP parameter written to Servlet output in org.apache.tez.common.web.ProfileOutputServlet.doGet(HttpServletRequest, HttpServletResponse)  At ProfileOutputServlet.java:output in org.apache.tez.common.web.ProfileOutputServlet.doGet(HttpServletRequest, HttpServletResponse)  At ProfileOutputServlet.java:[line 34] |
   | Failed junit tests | tez.test.TestAM |
   
   
   | Subsystem | Report/Notes |
   |----------:|:-------------|
   | Docker | ClientAPI=1.41 ServerAPI=1.41 base: https://ci-hadoop.apache.org/job/tez-multibranch/job/PR-212/1/artifact/out/Dockerfile |
   | GITHUB PR | https://github.com/apache/tez/pull/212 |
   | JIRA Issue | TEZ-4038 |
   | Optional Tests | dupname asflicense javac javadoc unit spotbugs findbugs checkstyle compile |
   | uname | Linux fc4678634afd 4.15.0-169-generic #177-Ubuntu SMP Thu Feb 3 10:50:38 UTC 2022 x86_64 x86_64 x86_64 GNU/Linux |
   | Build tool | maven |
   | Personality | personality/tez.sh |
   | git revision | master / 24a77c93f |
   | Default Java | Private Build-1.8.0_312-8u312-b07-0ubuntu1~20.04-b07 |
   | Multi-JDK versions | /usr/lib/jvm/java-11-openjdk-amd64:Private Build-11.0.15+10-Ubuntu-0ubuntu0.20.04.1 /usr/lib/jvm/java-8-openjdk-amd64:Private Build-1.8.0_312-8u312-b07-0ubuntu1~20.04-b07 |
   | checkstyle | https://ci-hadoop.apache.org/job/tez-multibranch/job/PR-212/1/artifact/out/diff-checkstyle-tez-common.txt |
   | checkstyle | https://ci-hadoop.apache.org/job/tez-multibranch/job/PR-212/1/artifact/out/diff-checkstyle-tez-tests.txt |
   | findbugs | https://ci-hadoop.apache.org/job/tez-multibranch/job/PR-212/1/artifact/out/new-findbugs-tez-common.html |
   | unit | https://ci-hadoop.apache.org/job/tez-multibranch/job/PR-212/1/artifact/out/patch-unit-tez-tests.txt |
   |  Test Results | https://ci-hadoop.apache.org/job/tez-multibranch/job/PR-212/1/testReport/ |
   | asflicense | https://ci-hadoop.apache.org/job/tez-multibranch/job/PR-212/1/artifact/out/patch-asflicense-problems.txt |
   | Max. process+thread count | 1385 (vs. ulimit of 5500) |
   | modules | C: tez-common tez-dag tez-tests U: . |
   | Console output | https://ci-hadoop.apache.org/job/tez-multibranch/job/PR-212/1/console |
   | versions | git=2.25.1 maven=3.6.3 findbugs=3.0.1 |
   | Powered by | Apache Yetus 0.12.0 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: issues-unsubscribe@tez.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org