Open pshipton opened 1 year ago
From the output it's not clear whether the client attempted to compile anything. The required message "Connected to a server" appears only during a compilation attempt.
I think we need to change the test to include verbose={compilePerformance}
(or at the very least verbose={compileEnd}
) to see whether compilations were done without connecting to the server.
Another suggestion: if the goal of the test is simply to make sure that -XX+UseJITServer
option is processed correctly, then the output
#JITServer: JITServer Client Mode. Server address: localhost port: 44089. Connection Timeout 30000ms
is sufficient to make that determination.
Attn @dsouzai
I can update the test to have compilePerformance
. I suppose we can just check for the Client Mode
string, but other tests in this suite check that we don't connect to a server if -XX:-UseJITServer
is specified, so for consistency, we probably should ensure that -XX:+UseJITServer
does result in the client connecting if there is a jitserver instance available.
Also, would the client print out
[ERR] #JITServer: Sent clientSessionTerminate message
if it's not connected to a server?
https://openj9-jenkins.osuosl.org/job/Test_openjdk8_j9_sanity.functional_s390x_linux_Nightly_testList_1/556 - rh8-390-1 testJitserverArguments_0
@mpirvu fyi