kfaraz commented on PR #15637: URL: https://github.com/apache/druid/pull/15637#issuecomment-1886188013
@AmatyaAvadhanula , let's keep it simple. Since the new task action is responsible only for cleanup of upgrade segments (something which doesn't happen in the current version anyway), let's just put the submission of the task action in a try catch. If it succeeds, cool. If the OL cannot identify the task action and throws an exception, we just log the exception saying that the OL is probably on an old version. This would ensure that the task itself doesn't fail if the OL is on an old version. Also, please make sure you test out this scenario. -- 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]
