jackjlli commented on issue #5262: Refactor configs in pinot-spi
URL: https://github.com/apache/incubator-pinot/pull/5262#issuecomment-615028531
 
 
   > Thanks for cleaning this, should also move Schema under config?
   
   The common thing for those table/instance/tenant configs is that they all 
extend BaseJsonConfig class while Schema doesn't. Maybe we can have Schema 
extend that class as well. If so, I can make it in another PR.

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


With regards,
Apache Git Services

---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]

Reply via email to