Instead of defining a new Exception that's used once, maybe just throw a RuntimeException here? It would be easier to read imho.