You are viewing a plain text version of this content. The canonical link for it is here.

Posted to commits@pulsar.apache.org by GitBox <gi...@apache.org> on 2019/04/27 01:29:12 UTC

[GitHub] [pulsar] sijie commented on issue #4151: (WIP) Pulsar SQL supports pulsar's primitive schemas

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:
users@infra.apache.org


With regards,
Apache Git Services