stillmatic opened a new pull request #4869: remove DISTINCT ON statement
URL: https://github.com/apache/incubator-superset/pull/4869
 
 
   this function is used in the 'Enable Filter Select' function. Because it 
passes `column_name` to `distinct` the function is only usable by backends 
which support the `DISTINCT ON` statement, i.e. only postgres.
   
   Here is (gist of) the error string I get:
   
   ```
   SELECT DISTINCT ON is not supported [SQL: 'SELECT DISTINCT ON (column_name) 
column_name AS column_name \nFROM public.table_name \n LIMIT 10000 (Background 
on this error at: http://sqlalche.me/e/tw8g)
   ```
   
   DISTINCT ON is only meaningful when selecting values from multiple columns. 
Because we are selecting rows from only one column, the above processed 
statement is logically identical to `SELECT DISTINCT column_name FROM 
table_name LIMIT 10`. 
   
   this passes CI / tox / etc since it's a one line change.

----------------------------------------------------------------
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:
us...@infra.apache.org


With regards,
Apache Git Services

Reply via email to