clintropolis commented on issue #7134: overhaul 'druid-orc-extensions' and move from 'contrib' to 'core' URL: https://github.com/apache/incubator-druid/issues/7134#issuecomment-481061957 >I agree here - in my experience writing the typeString is tough for people, and making it optional is a good thing. To clarify, my current PR eliminates `typeString`, so 'optional' isn't really correct since it would always be ignored. Do you think it necessary to have a compatibility layer of some sort to collect the column names from a `typeString` and map them to the actual column names defined in the `OrcStruct`?
---------------------------------------------------------------- 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]
