imply-cheddar commented on PR #14462: URL: https://github.com/apache/druid/pull/14462#issuecomment-1670295363
Generally speaking, I don't believe that we should merge the code with the `GenericIndexed` method of generating the columns. That's going to create the need to always support that. We have a better way to store things (the way that the String versions do it) and we shoudl follow those instead. If we really want to do these in two PRs, I'd suggest leaving this PR as is and create another PR on top of it to switch the serde around and then merge them in quick succession. -- 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]
