You are viewing a plain text version of this content. The canonical link for it is here.
Posted to notifications@superset.apache.org by GitBox <gi...@apache.org> on 2019/09/05 09:04:25 UTC

[GitHub] [incubator-superset] squalou opened a new issue #8177: Table Sort broken : can't resolve label reference for ORDER BY / GROUP BY Textual SQL Expression 'blah' should be explicitely declared as text('blah')

squalou opened a new issue #8177: Table Sort broken : can't resolve label reference for ORDER BY / GROUP BY Textual SQL Expression 'blah' should be explicitely declared as text('blah')
URL: https://github.com/apache/incubator-superset/issues/8177
 
 
   Charts with an "order by" that used to work in 0.28 are now broken whith ORDER BY or GROUP BY queries
   
   ### Expected results
   
   A Table Chart is displayed
   (example chart option in 'how to reproduce' screenshot)
   
   ### Actual results
   
   Exception displayed (see screenshot)
   
   
   #### Screenshots
   
   ![unnamed](https://user-images.githubusercontent.com/4623644/64320928-d02faf80-cfbf-11e9-9c53-b5fa6fdbb767.png)
   
   
   #### How to reproduce the bug
   
   1. Go to Chart, ceate a Table chart
   2. Select some fields in "NOT GROUP BY", choose a column for "ordering"
   3. Run
   4. See error
   
   ![2019-09-05 11-03-16 screenshot](https://user-images.githubusercontent.com/4623644/64327911-ce201d80-cfcc-11e9-80a9-3c4fa9632e03.png)
   
   
   ### Environment
   
   (please complete the following information):
   
   - superset version: 0.34 - also tried on master HEAD @ de6d9634
   - python version: 3.6
   - node.js version: 12.9.0
   
   
   ### Checklist
   
   Make sure these boxes are checked before submitting your issue - thank you!
   
   - [x ] I have checked the superset logs for python stacktraces and included it here as text if there are any.
   - [x ] 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.
   
   ### Additional context
   
   Could be related to SQLAlchemy syntax, 
   looks like this : https://stackoverflow.com/questions/35132463/sqlalchemy-core-order-by-desc
   
   
   
   

----------------------------------------------------------------
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
 
For queries about this service, please contact Infrastructure at:
users@infra.apache.org


With regards,
Apache Git Services

---------------------------------------------------------------------
To unsubscribe, e-mail: notifications-unsubscribe@superset.apache.org
For additional commands, e-mail: notifications-help@superset.apache.org