You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
#32785 redid table alias management, introducing a new "SQL alias manager" which is responsible for tracking which aliases are already in use in a query, and ensuring new ones are unique with respect to those. It also concentrates the actual logic of what table aliases look like (first char, lowercase) in a single place, so that this logic can be tweaked/overridden.
However, this didn't make any changes to projection (i.e. column) alias management - these are still hard-coded in various places. We should also refactor that logic into the new SQL manager.
roji
changed the title
Switch to managing column aliases via ISqlAliasManager as well
Switch to managing projection/column aliases via ISqlAliasManager as well
Jan 15, 2024
#32785 redid table alias management, introducing a new "SQL alias manager" which is responsible for tracking which aliases are already in use in a query, and ensuring new ones are unique with respect to those. It also concentrates the actual logic of what table aliases look like (first char, lowercase) in a single place, so that this logic can be tweaked/overridden.
However, this didn't make any changes to projection (i.e. column) alias management - these are still hard-coded in various places. We should also refactor that logic into the new SQL manager.
/cc @maumar
The text was updated successfully, but these errors were encountered: