Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

Fix getUnixPID() for jdk11+ and stop waiting for tests after 10 min #11199

Merged
merged 1 commit into from
Nov 17, 2020
Merged

Fix getUnixPID() for jdk11+ and stop waiting for tests after 10 min #11199

merged 1 commit into from
Nov 17, 2020

Conversation

pshipton
Copy link
Member

Fixes #11196
Fixes #11197

Signed-off-by: Peter Shipton Peter_Shipton@ca.ibm.com

@pshipton pshipton requested a review from llxia November 16, 2020 17:24
@llxia
Copy link
Contributor

llxia commented Nov 16, 2020

jenkins test sanity all jdk8,jdk11

@llxia
Copy link
Contributor

llxia commented Nov 17, 2020

Test compilation error:

13:53:21      [javac] /home/****/workspace/Test_openjdk11_j9_sanity.functional_s390x_linux_Personal_testList_1/openjdk-tests/functional/cmdline_options_tester/src/Test.java:432: error: cannot find symbol
13:53:21      [javac] 			Method pidMethod = Process.class.getMethod("pid");
13:53:21      [javac] 			^
13:53:21      [javac]   symbol:   class Method
13:53:21      [javac]   location: class Test
13:53:21      [javac] Note: Some input files use unchecked or unsafe operations.
13:53:21      [javac] Note: Recompile with -Xlint:unchecked for details.
13:53:21      [javac] 1 error

@pshipton
Fix getUnixPID() for jdk11+ and stop waiting for tests after 10 min
0662412 
Issue eclipse-openj9#11196
Issue eclipse-openj9#11197

[ci skip]

Signed-off-by: Peter Shipton <Peter_Shipton@ca.ibm.com>
@pshipton
Copy link
Member Author

Missing import.

jenkins test sanity zlinux jdk8,jdk11

@pshipton
Copy link
Member Author

@llxia fixed now

@llxia llxia merged commit 4e25e83 into eclipse-openj9:master Nov 17, 2020
@pshipton pshipton mentioned this pull request Nov 18, 2020
Closed
@pshipton pshipton mentioned this pull request Dec 8, 2020
Closed
@knn-k knn-k mentioned this pull request Nov 3, 2022
Closed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@llxia llxia llxia approved these changes

Assignees
No one assigned
Labels
comp:test
Projects
None yet
Milestone
No milestone
2 participants
@pshipton @llxia