tanishq-chugh commented on PR #6656: URL: https://github.com/apache/hive/pull/6656#issuecomment-6363807383
@Manya0407 As you mentioned that this is being introduced by your patch and its been identified before merge, then it should be handled in this PR itself. Regarding the solutions you mentioned, check around how each of the solution is behaving? would either make the state to be reported as Launching when end-user queries for a non-existent cluster? How was the same case handled and reported in yarn based deployments initially? As an end-user, i expect it to be App not found. we would want to avoid any mis-info being reported or regressions to slip in -- 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]
