chenhao-db commented on PR #50913: URL: https://github.com/apache/spark/pull/50913#issuecomment-2902013690
Increasing the limit won't bring any new OOM to existing workloads. The new OOM in tests is because we changed the test code to use a bigger input. I'm not sure whether we are able to find all the use cases of `toString`, which can happen in many places implicitly. Even if you provide this new version of `toString`, you cannot easily guarantee the no-parameter version is not called. -- 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: reviews-unsubscr...@spark.apache.org For queries about this service, please contact Infrastructure at: us...@infra.apache.org --------------------------------------------------------------------- To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org For additional commands, e-mail: reviews-h...@spark.apache.org
