dongjoon-hyun commented on code in PR #49813:
URL: https://github.com/apache/spark/pull/49813#discussion_r1951633984
##########
sql/connect/client/jvm/src/test/scala/org/apache/spark/sql/connect/ClientE2ETestSuite.scala:
##########
@@ -228,11 +228,12 @@ class ClientE2ETestSuite
}
test("spark deep recursion") {
+ var recursionDepth = if (System.getProperty("os.arch") == "s390x") 400
else 500
Review Comment:
Which Java distribution did you used on this `os.arch==s390x`?
> ClientE2ETestSuite: Change the recursion depth from 500 to 400 for s390x
only since the jvm uses more memory on that platform.
Specifically, I'm wondering
- if you tested both Java 17 and 21 on IBM Java distribution.
- if you tested other Java distributions (Zulu, Temurin, Oracle)
--
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: [email protected]
For queries about this service, please contact Infrastructure at:
[email protected]
---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]
