LakshSingla commented on PR #16864: URL: https://github.com/apache/druid/pull/16864#issuecomment-2293200958
Makes sense. I'll merge this one, can you please raise a separate patch for changing the name to `dimensionSchemas`, since if this one makes it in one of the releases without the other patch then there'd be compatibility issues b/w versions? Having a standalone patch for the name change would make sure that we remember to treat this change with the other as one. Having the name change with other changes would make it difficult to track, and we might end up with a version that doesn't have the rename. -- 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]
