comphead commented on PR #3882: URL: https://github.com/apache/arrow-datafusion/pull/3882#issuecomment-1284895937
> Ok, but the current branch now also fails with `select trim(a), trim(b) from table`? As the two are now giving the same alias. This is my main worry to merge this. As suffix that @comphead suggests seems a solution. This is correct, as we shortened the column name the collision will happen more often. But this gives us more motivation to make unique readable aliases. Because currently user may not be happy to refer to column from outer sql to nested subquery which contains not friendly alias like `btrim(test.a,Utf8(\"ab\"))` -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comment. To unsubscribe, e-mail: [email protected] For queries about this service, please contact Infrastructure at: [email protected]
