clintropolis commented on PR #13854:
URL: https://github.com/apache/druid/pull/13854#issuecomment-1454102880

   >Should something be added here: 
https://github.com/apache/druid/blob/master/web-console/src/druid-models/index-spec/index-spec.tsx#L28
 here 
https://github.com/apache/druid/blob/master/web-console/src/druid-models/index-spec/index-spec.tsx#L48
 and here 
https://github.com/apache/druid/blob/master/web-console/src/druid-models/index-spec/index-spec.tsx#L97
 ? Or do you want to do that in a follow on PR?
   
   The numbers for v1 seem nice enough that I'm considering just removing the 
option to write v0, which means no changes required here, so lets hold off 
until I decide for sure on that.


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