anishshri-db commented on PR #49099: URL: https://github.com/apache/spark/pull/49099#issuecomment-2524563968
> The code change looks good to me. Have you tried with having test(s) leveraging local-cluster? IIUC, the issue was only happening with distributed setup, hence we'd want to test the case if feasible. In the test context, atleast I'm not able to compile if we don't pass the implicits in some form. But let me add a `local-cluster` suite anyway - to ensure that the serialization path is hit and working as expected -- 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
