Skip to content

Conversation

@xiaoyuyao
Copy link
Contributor

…ava8.

@xiaoyuyao xiaoyuyao self-assigned this Mar 18, 2019
@xiaoyuyao xiaoyuyao requested a review from elek March 18, 2019 05:08
@hadoop-yetus
Copy link

💔 -1 overall

Vote Subsystem Runtime Comment
0 reexec 32 Docker mode activated.
_ Prechecks _
0 yamllint 0 yamllint was not available.
+1 @author 0 The patch does not contain any @author tags.
-1 test4tests 0 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.
_ trunk Compile Tests _
+1 mvninstall 1074 trunk passed
+1 compile 61 trunk passed
+1 mvnsite 20 trunk passed
+1 shadedclient 1770 branch has no errors when building and testing our client artifacts.
+1 javadoc 15 trunk passed
_ Patch Compile Tests _
-1 mvninstall 17 dist in the patch failed.
+1 compile 20 the patch passed
+1 javac 20 the patch passed
+1 mvnsite 20 the patch passed
+1 whitespace 0 The patch has no whitespace issues.
+1 shadedclient 743 patch has no errors when building and testing our client artifacts.
+1 javadoc 17 the patch passed
_ Other Tests _
+1 unit 21 dist in the patch passed.
+1 asflicense 25 The patch does not generate ASF License warnings.
2804
Subsystem Report/Notes
Docker Client=17.05.0-ce Server=17.05.0-ce base: https://builds.apache.org/job/hadoop-multibranch/job/PR-615/1/artifact/out/Dockerfile
GITHUB PR #615
Optional Tests dupname asflicense mvnsite compile javac javadoc mvninstall unit shadedclient yamllint
uname Linux 65c4685dff7e 4.4.0-139-generic #165-Ubuntu SMP Wed Oct 24 10:58:50 UTC 2018 x86_64 x86_64 x86_64 GNU/Linux
Build tool maven
Personality personality/hadoop.sh
git revision trunk / 926d548
maven version: Apache Maven 3.3.9
Default Java 1.8.0_191
mvninstall https://builds.apache.org/job/hadoop-multibranch/job/PR-615/1/artifact/out/patch-mvninstall-hadoop-ozone_dist.txt
Test Results https://builds.apache.org/job/hadoop-multibranch/job/PR-615/1/testReport/
Max. process+thread count 453 (vs. ulimit of 5500)
modules C: hadoop-ozone/dist U: hadoop-ozone/dist
Console output https://builds.apache.org/job/hadoop-multibranch/job/PR-615/1/console
Powered by Apache Yetus 0.9.0 http://yetus.apache.org

This message was automatically generated.

Copy link
Member

@elek elek left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

+1

We can adjust the remaining docker-compose files later (ozones3/ozonperf). With updating the normal compose we will test everything with both jdk8 (ozone) and jdk11 (ozones3 + ozonesecure).

@xiaoyuyao xiaoyuyao merged commit 992489c into apache:trunk Mar 19, 2019
asfgit pushed a commit that referenced this pull request Mar 19, 2019
…ava8. Contributed by Xiaoyu Yao.

This closes #615

(cherry picked from commit 992489c)
shanthoosh pushed a commit to shanthoosh/hadoop that referenced this pull request Oct 15, 2019
…ainers

- Updated internal state that tracks "pending" containers correctly
- Refactored `YarnClusterResourceManager` for testability. Add an unit test

Author: Jagadish <[email protected]>

Reviewers: Jake Maes<[email protected]>

Closes apache#615 from vjagadish1989/container-launch-error
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants