Add text exceptions to the Scheduler

[mrocklin]

Fixes #5126

FYI @ian-r-rose @fjetter

[fjetter]

Do we need to worry about size? This stores the traceback in plain txt but we usually truncate exception messages. I guess we truncate exceptions for readability but there is no harm in actually submitting the entire traceback, is there?

[mrocklin]

Perhaps we should add a [-10000:] to the traceback text?

[fjetter]

Perhaps we should add a [-10000:] to the traceback text?

Fine by me but I'm also fine with doing nothing if it is not clear why we're doing it. Whether or not we need to truncate depends on the usage. For instance, logs will need to be truncated since too large message can break certain loggers.
I just wanted to ask in case I was missing anything

[mrocklin]

We don't currently truncate the actual traceback, so I don't think that this will be worse than current state

…

On Mon, Aug 2, 2021 at 7:18 AM Florian Jetter ***@***.***> wrote: Perhaps we should add a [-10000:] to the traceback text? Fine by me but I'm also fine with doing nothing if it is not clear why we're doing it. Whether or not we need to truncate depends on the usage. For instance, logs will need to be truncated since too large message can break certain loggers. I just wanted to ask in case I was missing anything — You are receiving this because you authored the thread. Reply to this email directly, view it on GitHub <#5148 (comment)>, or unsubscribe <https://github.com/notifications/unsubscribe-auth/AACKZTCIVWYI6A5ELQCZ533T22SLFANCNFSM5BJHVAGA> .