mcvsubbu commented on pull request #5324: URL: https://github.com/apache/incubator-pinot/pull/5324#issuecomment-622584455
> > Is the new util class to be used inside the Schema class (and everyone else outside sees only DateTimeFieldSpec)? > > That's right. Only Schema class should know about this converter. Maybe it can even go into the Schema class Yes, as methods in Schema class? ---------------------------------------------------------------- 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] --------------------------------------------------------------------- To unsubscribe, e-mail: [email protected] For additional commands, e-mail: [email protected]
