This is just getting ridiculous. If we can be so casual with m_targetTableAlias, I'm willing to believe that it is useless. Let's get rid of it and use m_tableScan.m_alias in the few places we need that value (dummy TVE initialization in the outputschema generation).