Details
-
Type:
Improvement
-
Status: Closed
-
Priority:
Minor
-
Resolution: Fixed
-
Affects Version/s: 2.6.3
-
Fix Version/s: 2.8.0, 2.7.3, 2.6.5, 3.0.0-alpha1
-
Component/s: util
-
Labels:None
-
Hadoop Flags:Reviewed
Description
Currently ApplicationClassLoader does not log the classpath at the INFO level although the system classes are logged at that level. Knowing exactly what classpath ApplicationClassLoader has is a critical piece of information for troubleshooting. We should log it at the INFO level.
Activity
- All
- Comments
- Work Log
- History
- Activity
- Transitions
 -1 overall |
| Vote | Subsystem | Runtime | Comment |
|---|---|---|---|
| 0 | reexec | 0m 0s | Docker mode activated. |
| -1 | docker | 0m 4s | Docker failed to build yetus/hadoop:0ca8df7. |
| Subsystem | Report/Notes |
|---|---|
| JIRA Patch URL | https://issues.apache.org/jira/secure/attachment/12787372/HADOOP-12789.01.patch |
| JIRA Issue | |
| Console output | https://builds.apache.org/job/PreCommit-HADOOP-Build/8592/console |
| Powered by | Apache Yetus 0.2.0-SNAPSHOT http://yetus.apache.org |
This message was automatically generated.
| -1 overall |
| Vote | Subsystem | Runtime | Comment |
|---|---|---|---|
| 0 | reexec | 0m 11s | Docker mode activated. |
| +1 | @author | 0m 0s | The patch does not contain any @author tags. |
| -1 | test4tests | 0m 0s | The patch doesn't appear to include any new or modified tests. Please justify why no new tests are needed for this patch. Also please list what manual steps were performed to verify this patch. |
| +1 | mvninstall | 6m 53s | trunk passed |
| +1 | compile | 5m 55s | trunk passed with JDK v1.8.0_72 |
| +1 | compile | 6m 45s | trunk passed with JDK v1.7.0_95 |
| +1 | checkstyle | 0m 22s | trunk passed |
| +1 | mvnsite | 1m 6s | trunk passed |
| +1 | mvneclipse | 0m 14s | trunk passed |
| +1 | findbugs | 1m 35s | trunk passed |
| +1 | javadoc | 0m 53s | trunk passed with JDK v1.8.0_72 |
| +1 | javadoc | 1m 3s | trunk passed with JDK v1.7.0_95 |
| +1 | mvninstall | 0m 42s | the patch passed |
| +1 | compile | 5m 51s | the patch passed with JDK v1.8.0_72 |
| +1 | javac | 5m 51s | the patch passed |
| +1 | compile | 6m 45s | the patch passed with JDK v1.7.0_95 |
| +1 | javac | 6m 45s | the patch passed |
| +1 | checkstyle | 0m 23s | the patch passed |
| +1 | mvnsite | 1m 3s | the patch passed |
| +1 | mvneclipse | 0m 15s | the patch passed |
| +1 | whitespace | 0m 0s | Patch has no whitespace issues. |
| +1 | findbugs | 1m 48s | the patch passed |
| +1 | javadoc | 0m 52s | the patch passed with JDK v1.8.0_72 |
| +1 | javadoc | 1m 4s | the patch passed with JDK v1.7.0_95 |
| -1 | unit | 19m 22s | hadoop-common in the patch failed with JDK v1.8.0_72. |
| +1 | unit | 7m 25s | hadoop-common in the patch passed with JDK v1.7.0_95. |
| +1 | asflicense | 0m 20s | Patch does not generate ASF License warnings. |
| 71m 54s |
| Reason | Tests |
|---|---|
| JDK v1.8.0_72 Timed out junit tests | org.apache.hadoop.http.TestHttpServerLifecycle |
| Subsystem | Report/Notes |
|---|---|
| Docker | Image:yetus/hadoop:0ca8df7 |
| JIRA Patch URL | https://issues.apache.org/jira/secure/attachment/12787372/HADOOP-12789.01.patch |
| JIRA Issue | |
| Optional Tests | asflicense compile javac javadoc mvninstall mvnsite unit findbugs checkstyle |
| uname | Linux 4fe79d63860d 3.13.0-36-lowlatency #63-Ubuntu SMP PREEMPT Wed Sep 3 21:56:12 UTC 2014 x86_64 x86_64 x86_64 GNU/Linux |
| Build tool | maven |
| Personality | /testptch/hadoop/patchprocess/precommit/personality/provided.sh |
| git revision | trunk / aeb13ef |
| Default Java | 1.7.0_95 |
| Multi-JDK versions | /usr/lib/jvm/java-8-oracle:1.8.0_72 /usr/lib/jvm/java-7-openjdk-amd64:1.7.0_95 |
| findbugs | v3.0.0 |
| unit | https://builds.apache.org/job/PreCommit-HADOOP-Build/8591/artifact/patchprocess/patch-unit-hadoop-common-project_hadoop-common-jdk1.8.0_72.txt |
| unit test logs | https://builds.apache.org/job/PreCommit-HADOOP-Build/8591/artifact/patchprocess/patch-unit-hadoop-common-project_hadoop-common-jdk1.8.0_72.txt |
| JDK v1.7.0_95 Test Results | https://builds.apache.org/job/PreCommit-HADOOP-Build/8591/testReport/ |
| modules | C: hadoop-common-project/hadoop-common U: hadoop-common-project/hadoop-common |
| Max memory used | 77MB |
| Console output | https://builds.apache.org/job/PreCommit-HADOOP-Build/8591/console |
| Powered by | Apache Yetus 0.2.0-SNAPSHOT http://yetus.apache.org |
This message was automatically generated.
| -1 overall |
| Vote | Subsystem | Runtime | Comment |
|---|---|---|---|
| 0 | reexec | 0m 15s | Docker mode activated. |
| +1 | @author | 0m 0s | The patch does not contain any @author tags. |
| -1 | test4tests | 0m 0s | The patch doesn't appear to include any new or modified tests. Please justify why no new tests are needed for this patch. Also please list what manual steps were performed to verify this patch. |
| +1 | mvninstall | 7m 5s | trunk passed |
| +1 | compile | 5m 53s | trunk passed with JDK v1.8.0_72 |
| +1 | compile | 6m 47s | trunk passed with JDK v1.7.0_95 |
| +1 | checkstyle | 0m 21s | trunk passed |
| +1 | mvnsite | 1m 5s | trunk passed |
| +1 | mvneclipse | 0m 13s | trunk passed |
| +1 | findbugs | 1m 39s | trunk passed |
| +1 | javadoc | 0m 55s | trunk passed with JDK v1.8.0_72 |
| +1 | javadoc | 1m 6s | trunk passed with JDK v1.7.0_95 |
| +1 | mvninstall | 0m 41s | the patch passed |
| +1 | compile | 5m 50s | the patch passed with JDK v1.8.0_72 |
| +1 | javac | 5m 50s | the patch passed |
| +1 | compile | 6m 50s | the patch passed with JDK v1.7.0_95 |
| +1 | javac | 6m 50s | the patch passed |
| +1 | checkstyle | 0m 21s | the patch passed |
| +1 | mvnsite | 1m 3s | the patch passed |
| +1 | mvneclipse | 0m 14s | the patch passed |
| +1 | whitespace | 0m 0s | Patch has no whitespace issues. |
| +1 | findbugs | 1m 49s | the patch passed |
| +1 | javadoc | 0m 55s | the patch passed with JDK v1.8.0_72 |
| +1 | javadoc | 1m 2s | the patch passed with JDK v1.7.0_95 |
| +1 | unit | 7m 1s | hadoop-common in the patch passed with JDK v1.8.0_72. |
| +1 | unit | 7m 17s | hadoop-common in the patch passed with JDK v1.7.0_95. |
| +1 | asflicense | 0m 24s | Patch does not generate ASF License warnings. |
| 59m 59s |
| Subsystem | Report/Notes |
|---|---|
| Docker | Image:yetus/hadoop:0ca8df7 |
| JIRA Patch URL | https://issues.apache.org/jira/secure/attachment/12787372/HADOOP-12789.01.patch |
| JIRA Issue | |
| Optional Tests | asflicense compile javac javadoc mvninstall mvnsite unit findbugs checkstyle |
| uname | Linux c1c5747c4619 3.13.0-36-lowlatency #63-Ubuntu SMP PREEMPT Wed Sep 3 21:56:12 UTC 2014 x86_64 x86_64 x86_64 GNU/Linux |
| Build tool | maven |
| Personality | /testptch/hadoop/patchprocess/precommit/personality/provided.sh |
| git revision | trunk / aeb13ef |
| Default Java | 1.7.0_95 |
| Multi-JDK versions | /usr/lib/jvm/java-8-oracle:1.8.0_72 /usr/lib/jvm/java-7-openjdk-amd64:1.7.0_95 |
| findbugs | v3.0.0 |
| JDK v1.7.0_95 Test Results | https://builds.apache.org/job/PreCommit-HADOOP-Build/8594/testReport/ |
| modules | C: hadoop-common-project/hadoop-common U: hadoop-common-project/hadoop-common |
| Max memory used | 77MB |
| Console output | https://builds.apache.org/job/PreCommit-HADOOP-Build/8594/console |
| Powered by | Apache Yetus 0.2.0-SNAPSHOT http://yetus.apache.org |
This message was automatically generated.
Any reviews? This shouldn't take too long to review... Thanks.
Any takers? This is a pretty trivial change.
+1. Committed to trunk, branch-2, branch-2.8, branch-2.7 and branch-2.6. Thanks Sangjin Lee for the contribution.
SUCCESS: Integrated in Hadoop-trunk-Commit #9441 (See https://builds.apache.org/job/Hadoop-trunk-Commit/9441/)
HADOOP-12789. log classpath of ApplicationClassLoader at INFO level. (mingma: rev 49eedc7ff02ea61764f416f0e2ddf81370aec5fb)
- hadoop-common-project/hadoop-common/src/main/java/org/apache/hadoop/util/ApplicationClassLoader.java
Thanks Ming Ma!
Closing the JIRA as part of 2.7.3 release.
Patch posted.