bastionlab
bastionlab copied to clipboard
"Could not run logical plan" error should not be Internal Server Error
GRPCException: Internal server error: code=StatusCode.INTERNAL message=Could not run logical plan: Data types don't match: Series of dtype: Date != Utf8
should not be internal server error, it should be BadRequest/other
No reproduction code; the actual problematic code should be obvious with a simple grep on the codebase
Also, internal server errors should be logged to the console, as they mean something wrong happened (the kind of logical bugs that should never happen)
Agreed we could use INVALID_ARGUMENT instead