gianm commented on PR #12753: URL: https://github.com/apache/druid/pull/12753#issuecomment-1181905372
Nice work! Very cool. I just noticed this when you posted it on Slack. The first thing that comes to mind is questions about the intended maturity of this feature after initial contribution: - Is it intentionally undocumented in this PR? Do you plan to add documentation? - Are there any impediments to maintaining forwards compatibility, such that new versions of Druid will always be able to read JSON columns written by older versions? Do you foresee any reason we might want to break compatibility? - Would you recommend we present this feature in its current state as experimental or production-ready, & why? -- 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. To unsubscribe, e-mail: [email protected] For queries about this service, please contact Infrastructure at: [email protected] --------------------------------------------------------------------- To unsubscribe, e-mail: [email protected] For additional commands, e-mail: [email protected]
