glasser commented on PR #18515:
URL: https://github.com/apache/druid/pull/18515#issuecomment-4710198518

   > These changes also switch to using the createIndex helper tool for every 
index creation, to make future changes easier. Although this separates the 
table/index creation into 2 separate transactions, from what I can tell, this 
shouldn't have any correctness side-effects since the indices are always 
followed the createTable() statements which would just mean the index may not 
be created even if the table was created.
   
   I think this is a correctness improvement — we have been using Druid since 
v0.12 and found out today that the pending segments indexes (introduced in 
v0.13) never got created in our DB!


-- 
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]

Reply via email to