[GitHub] [incubator-pinot] Jackie-Jiang opened a new pull request #4858: Introduce the base JSON config class and clean up the config classes

[GitBox <gi...@apache.org>]

Jackie-Jiang opened a new pull request #4858: Introduce the base JSON config class and clean up the config classes
URL: https://github.com/apache/incubator-pinot/pull/4858
 
 
   Simplify the config classes by introducing the BaseJsonConfig class with the following methods:
   - toJsonNode()
   - toJsonString()
   - hashCode()
   - equals()
   - toString()
   
   NOTE: hashCode() and equlas() are for debugging and testing purpose only, as config is not meant to be compared
   
   Use JsonCreator to construct configs to prevent configs getting modified

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


With regards,
Apache Git Services

---------------------------------------------------------------------
To unsubscribe, e-mail: commits-unsubscribe@pinot.apache.org
For additional commands, e-mail: commits-help@pinot.apache.org