eakmanrq / sqlframe

Turning PySpark Into a Universal DataFrame API
https://sqlframe.readthedocs.io/en/stable/
MIT License
191 stars 3 forks source link

fix: properly identify ambiguous column with an alias #52

Closed eakmanrq closed 1 month ago

eakmanrq commented 1 month ago

I checked the column_expressions for ambiguity but then later I just referred to the column itself for checking the name and updating the table reference. Now it consistently checks for the column_expression and the test was updated to properly verify this.

Fixes: https://github.com/eakmanrq/sqlframe/issues/50