tillrohrmann commented on pull request #15324: URL: https://github.com/apache/flink/pull/15324#issuecomment-809994736
I think the reason why `UnfulfillableSlotRequestException` inherits from `ResourceManagerException` is because it originates on the `ResourceManager`. I think it should be fine to change it and to make it a subclass of `NoResourceAvailableException`. Another idea could be to map `UnfulfillableSlotRequestException` to `NoResourceAvailableException` on the `JobMaster` if we want to keep these things separated. This would then also mean that `ResourceManager` should not throw a `NoResourceAvailableException` with the argument that it is a `JobException` which should only originate from the `JobMaster`/`Dispatcher`. -- 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
