SETL-Framework/setl

More explicit exception message when no deliverable could be found

qxzzxq opened this issue · 1 comments

If there is any missing deliverable, the pipeline will only throw an InvalidArgumentException without any details that can help user to find the missing deliverable (https://github.com/SETL-Developers/setl/blob/master/src/main/scala/com/jcdecaux/setl/workflow/Pipeline.scala#L386).

I think a more detailed exception message could be helpful for debugging.

for example, we could mention the factory class and the missing deliverable type

@JorisTruong what do you think?

Absolutely ! I think we should also mention the deliveryId and the producer of the missing deliverable.
I will make a pull request asap.