clintropolis commented on PR #11307: URL: https://github.com/apache/druid/pull/11307#issuecomment-1771657626
I kind of changed the world under this PR in the refactor of #12388, so quite a lot of things need to change to merge this, though some stuff might be a bit easier to implement now. After #12388, the right way to do this is to implement a `ColumnIndexSupplier` for the column indexers/incremental index stuff, hopefully it should be straightforward to do, but is quite a lot different than what things were when this PR was started. I'm happy to answer any questions or advise on how to move to that model. -- 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]
