Commit
This commit does not belong to any branch on this repository, and may belong to a fork outside of the repository.
use column name in group by in get_column_values (#407)
Most database engines (e.g. all SQL Server versions) don't support grouping by the position of the column.
- Loading branch information
fc18046
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Hello,
This seems a reasonable change, but it has limited the way I can use this method. One of method that I have used is to concat(field_1, "_", field_2), so I can use field_1 as a condition and then assign field_2 value.
You may suggest that I can use get_query_as_dict method. that's correct. I then hit another problem - SQLfluff doesn't like get_query_as_dict that much... tough call right..