kgyrtkirk commented on PR #18181: URL: https://github.com/apache/druid/pull/18181#issuecomment-3022518711
I wonder if `ROW_BASED_V2` should the default after say 2 releases ? I was thinking about this earlier....there are things which can't be done on-the-spot; but should be (much) later....is there an annotation or some other marking to ensure that will be considered? (this is now kinda out-of-scope) In the recent days I've bumped into a [similar issue](https://github.com/apache/druid/issues/13951) for which I feel like the fix might be closely related; if a `row_version` is added - it might be great to also cover that in the same version bump...what do you think? I think that could probably be fixed similarily inside `StringFieldReader` / `StringFieldWriter` -- 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]
