Add error message to DatabaseError output

JohnOmernik / sqlalchemy-drill

Apache Drill Dialect for SQL Alchemy

Other

53 stars 35 forks source link

Add error message to DatabaseError output #78

Closed ibarakaiev closed 2 years ago

ibarakaiev commented 2 years ago

Drill 1.19 omits the error message in REST API by default, which is overwritable by setting http.rest.errors.verbose to true. Some non-technical users may benefit from having an error message in Superset UI in case there is a database error or their SQL query is corrupted, which this PR implements. If no error message is provided, it is also useful to know how to overwrite that behavior as it is not easily searchable online.

jnturton commented 2 years ago

Thanks, this looks like a nice little improvement to me. CC @cgivre

jnturton commented 2 years ago

Ping @cgivre.