MartijnVisser commented on PR #28735: URL: https://github.com/apache/flink/pull/28735#issuecomment-4983838796
> Is "410 Gone" a better response code here if we knew the TaskManager was there but is gone and this is permanent? Good question. I'd keep 404 here. The status comes from mapping `UnknownTaskExecutorException`, which fires whenever the id isn't in the currently-registered set: we can't actually distinguish "was here, now gone" from "never existed" (a stale/deep-link with a bogus id hits the same path). It also keeps this handler consistent with the sibling TaskManager handlers (log list, profiling list, file handlers), which all map `UnknownTaskExecutorException` to `NOT_FOUND`; that consistency is exactly what this PR is fixing. -- 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]
