Github user kiszk commented on the issue:
https://github.com/apache/spark/pull/20772
I may be wrong, but I do not find algorithm changes in
`calculateParamLength`.
I believe that the original implementation is correct. This is because
`calculateParamLength` returns the number of Java stack operands in JVM. `Long`
or `Double` requires 2 stack operands to pass a value. In other words, we can
pass up to 127 long values in a method argument list.
---
---------------------------------------------------------------------
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional commands, e-mail: reviews-h...@spark.apache.org
