You are viewing a plain text version of this content. The canonical link for it is here.
Posted to dev@superset.apache.org by gi...@git.apache.org on 2017/07/31 10:54:02 UTC

[GitHub] jan-schulz-k24 opened a new issue #3208: Quoting column names in time series line charts

jan-schulz-k24 opened a new issue #3208: Quoting column names in time series line charts
URL: https://github.com/apache/incubator-superset/issues/3208
 
 
   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 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
   
   
   ### Superset version
   {"GIT_SHA": "b'219f33f0d1fcd05db1807f5b95caea8b0edd92f7\\n'", "version": "0.18.5"} (master from 2 hours ago)
   
   ### Expected results
   Referencing Capitalized columns in time series line charts works
   
   
   ### Actual results
   It doesn't work the generated SQL is not quoting the selected column
   
   ### Steps to reproduce
   * Have a postgres data source, where the time column name needs quotes (Capitalized or with spaces in them)
   * Try to build a time series chart where that column is used as time dimension
   
   The problem is that the generated SQL is not quoting the selected columns:
   
   ```
   (psycopg2.ProgrammingError) column "bestelldatum" does not exist LINE 1: SELECT Bestelldatum AS __timestamp, ^ HINT: Perhaps you meant to reference the column "bestellartikel_data_set.Bestelldatum". [SQL: 'SELECT Bestelldatum AS __timestamp,\n SUM(ds.bestellartikel_data_set."Bruttobestellwert") AS "sum__Bruttobestellwert"\nFROM ds.bestellartikel_data_set\nWHERE "Bestelldatum" >= \'2017-07-24 12:45:22\'\n AND "Bestelldatum" <= \'2017-07-31 12:45:22\'\nGROUP BY Bestelldatum\nLIMIT 5000']
   ```
   In the WHERE, it's quoted, but not in the SELECT part and not in the GROUP BY part .
   
 
----------------------------------------------------------------
This is an automated message from the Apache Git Service.
To respond to the message, please log on 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