merlimat commented on issue #2004: Issue 1943: remove serializable from builders and add loadData to load configuration from a config map URL: https://github.com/apache/incubator-pulsar/pull/2004#issuecomment-399001572 @sijie one option to detect spelling mistakes (or variable name changes) would be to throw exception if there is any unknown key in the configuration map. This way if there's a typo, I would get an exception instead of my setting being ignored which will be much harder to detect and debug.
---------------------------------------------------------------- This is an automated message from the Apache Git Service. To respond to the message, please log on GitHub and use the URL above to go to the specific comment. For queries about this service, please contact Infrastructure at: [email protected] With regards, Apache Git Services
