danny0405 commented on pull request #3505:
URL: https://github.com/apache/hudi/pull/3505#issuecomment-902407336


   > that the default values are never loaded into the property maps in these 
classes
   
   Why the default values must be loaded into the property maps, the 
`ConfigProperty` did have a default value and IMO it is the only truth for 
default value.
   
   The current configuration does not have a good design, such as the 
`HoodiePayloadProps` and `HoodiePayloadConfiguraion`, `LockConfiguration` and 
`HoodieLockConfig`. There are so many duplicate keys.


-- 
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]


Reply via email to