bastionlab icon indicating copy to clipboard operation
bastionlab copied to clipboard

"Could not run logical plan" error should not be Internal Server Error

Open cchudant opened this issue 2 years ago • 1 comments

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)

cchudant avatar Jan 12 '23 13:01 cchudant

Agreed we could use INVALID_ARGUMENT instead

dhalf avatar Jan 13 '23 10:01 dhalf