sijie commented on issue #4151: (WIP) Pulsar SQL supports pulsar's primitive schemas URL: https://github.com/apache/pulsar/pull/4151#issuecomment-487243716 @srkukarni Current presto integration bypasses Pulsar's schema's implementation and interacts with *avro schema data* directly. It introduces several problems: 1) all the *properties* are lost when presto gets splits. 2) it doesn't handle schema versions properly. 3) it introduces a lot of duplicated code that has already been implemented in pulsar schema. it makes maintenance very hard. Hence I don't think we should continue using the current approach for supporting other schema types. This is a WIP for demonstrating the approach I mentioned in #4022 to supporting primitive types in Presto.
---------------------------------------------------------------- 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
