Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

[sql lab] Presto & Hive errors shows as big unreadable messages #4013

Closed
mistercrunch opened this issue Dec 5, 2017 · 1 comment · Fixed by #4019
Closed

[sql lab] Presto & Hive errors shows as big unreadable messages #4013

mistercrunch opened this issue Dec 5, 2017 · 1 comment · Fixed by #4019
Labels
lyft Related to Lyft

Comments

@mistercrunch
Copy link
Member

The code that handles error message formatting for Hive and Presto
https://github.com/apache/incubator-superset/blob/master/superset/db_engine_specs.py#L607 worked at Airbnb but don't work properly in at Lyft. We should handle this in a way that work in all environments.

related: #2349
@mistercrunch mistercrunch added the lyft Related to Lyft label Dec 5, 2017
@mistercrunch mistercrunch mentioned this issue Dec 5, 2017
Closed
@dmigo
Copy link
Contributor

dmigo commented Dec 6, 2017

@mistercrunch

worked at Airbnb

Does the past tense mean that it is not known if it works now?

@dmigo dmigo mentioned this issue Dec 6, 2017
Merged
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
lyft Related to Lyft
Projects
None yet
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

[sql_lab] Handle PyHive DatabaseError format dmigo/incubator-superset
2 participants
@mistercrunch @dmigo