vinothchandar commented on pull request #3505: URL: https://github.com/apache/hudi/pull/3505#issuecomment-902393044
Removing `setDefaults()` mean, that the default values are never loaded into the property maps in these classes. So this patch as-is cannot be landed. My understanding on Spark is that all the hudi classes will get loaded onto a class loader that is accessible to the tasks. So I have not seen this issue here. the NPE due to lack of synchronization is legit though, @nsivabalan did you run tests with spark.executor.cores > 1 in any of the runs. Do you know if we are safe there -- 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]
