umehrot2 commented on pull request #2833: URL: https://github.com/apache/hudi/pull/2833#issuecomment-831666092
I recommend checking out: - https://github.com/apache/spark/blob/master/sql/catalyst/src/main/scala/org/apache/spark/sql/internal/SQLConf.scala - https://github.com/apache/spark/blob/master/core/src/main/scala/org/apache/spark/SparkConf.scala It has some good pieces we can adopt and specially as we will move towards a file based configuration. Some things that just standout based on quick look, but you can dig deeper: - Specifying version from which that property is supported - Whether the property is an internal config - Differentiate between Alternate and Deprecated configs. For ex: Hudi table name has alternatives -- 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: [email protected]
