vogievetsky commented on pull request #11228: URL: https://github.com/apache/druid/pull/11228#issuecomment-837582521
@suneet-s the fact that the endpoint field was incorrectly marked as `required` was there from the start of the data loader but until https://github.com/apache/druid/pull/10533 it just resulted in an incorrect border color that people would not even notice. https://github.com/apache/druid/pull/10533 started using the model definitions as validation also and that is when the original bug went from being (from a user perspective) almost unnoticeable to flow breaking. I think it is worth back-porting this. -- 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. For queries about this service, please contact Infrastructure at: us...@infra.apache.org --------------------------------------------------------------------- To unsubscribe, e-mail: commits-unsubscr...@druid.apache.org For additional commands, e-mail: commits-h...@druid.apache.org