[ 
https://issues.apache.org/jira/browse/HUDI-2321?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=17401984#comment-17401984
 ] 

ASF GitHub Bot commented on HUDI-2321:
--------------------------------------

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: commits-unsubscr...@hudi.apache.org

For queries about this service, please contact Infrastructure at:
us...@infra.apache.org


> Fix ClassLoader error due to HoodieConfig$setDefaults() method call reflection
> ------------------------------------------------------------------------------
>
>                 Key: HUDI-2321
>                 URL: https://issues.apache.org/jira/browse/HUDI-2321
>             Project: Apache Hudi
>          Issue Type: Bug
>          Components: configs
>            Reporter: Zheng yunhong
>            Priority: Major
>              Labels: pull-request-available
>             Fix For: 0.10.0
>
>
> Fix ClassLoader error due to HoodieConfig$setDefaults() method call 
> reflection.



--
This message was sent by Atlassian Jira
(v8.3.4#803005)

Reply via email to