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.

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

BUG: unable to select column with byteb datatype #6981

Closed
2 tasks
mmuru opened this issue Mar 5, 2019 · 4 comments · Fixed by #6987
Closed
2 tasks

BUG: unable to select column with byteb datatype #6981

mmuru opened this issue Mar 5, 2019 · 4 comments · Fixed by #6987

Comments

@mmuru
Copy link
Contributor

mmuru commented Mar 5, 2019

Make sure these boxes are checked before submitting your issue - thank you!

  • I have checked the superset logs for python stacktraces and included it here as text if there are any.
  • I have reproduced the issue with at least the latest released version of superset.
  • [ x] I have checked the issue tracker for the same issue and I haven't found one similar.

Superset version

0.28.1

Expected results

It should return query result

Actual results

Unserializable object <memory at 0x7f93249f9ac8> of type <class 'memoryview'>

Steps to reproduce

create a table with column datatype byteb

Other information:
Database: Postgresql
In Preview mode, the select query gets fired and shows the result.
When you run the query (select * from table) in sqlLab, it throws
Unserializable object <memory at 0x7f93249f9ac8> of type <class 'memoryview'>

@villebro
Copy link
Member

villebro commented Mar 6, 2019

Do you have a use case for visualising binary/byte data? It seems the most appropriate handling of bytes would be to either throw a more understandable exception or simply replace the byte values with a <BINARY DATA> label or similar.

@mmuru
Copy link
Contributor Author

mmuru commented Mar 6, 2019 via email

@mistercrunch
Copy link
Member

Do you have a stacktrace by any chance? It should definitely not raise... I think we handle some other binary types. It thought this would be a catchall: https://github.com/apache/incubator-superset/blob/master/superset/utils/core.py#L344

@mmuru
Copy link
Contributor Author

mmuru commented Mar 6, 2019

No exception raised otherwise I could have provided stacktrace. I think, the issue with decoding the binary data and surprisingly it works in preview mode.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging a pull request may close this issue.

3 participants