You are viewing a plain text version of this content. The canonical link for it is here.

Posted to commits@hudi.apache.org by GitBox <gi...@apache.org> on 2021/06/07 22:18:53 UTC

[GitHub] [hudi] vinothchandar edited a comment on pull request #2833: [HUDI-89] Add configOption & refactor Hudi configuration framework

vinothchandar edited a comment on pull request #2833:
URL: https://github.com/apache/hudi/pull/2833#issuecomment-856299824


   @zhedoubushishi could you please review the change here that re-implements config registry, by simply fetching all static ConfigProperty members of a class and setting their default values. (HoodieConfig has the changes) 
   
   https://github.com/apache/hudi/pull/2833/commits/0bb936681bdea93bab18298aa56b118c58d748cf#diff-48dd99966f88f47092578e974c688ce70538697a155fc8839de6d48731c5c258 


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

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