stoty commented on pull request #1133:
URL: https://github.com/apache/phoenix/pull/1133#issuecomment-774264986


   This causes a few tests that try to access columns by un-aliased names to 
fail.
   My first instinct was to look up the standard, but It's not free. However, 
what resources ARE available, all state that you must not refer to an aliased 
table by its original name
   
   https://crate.io/docs/sql-99/en/latest/chapters/18.html#correlation-name
   
   `Once you’ve defined a <Correlation name> for a Table, you must use it to 
refer to that Table throughout the entire transaction – for example, whenever 
you would normally use the <Table name> to qualify a <Column name>, use the 
<Correlation name> instead.
   `
   I've tested with mariadb, and it works as described above.
   
   I don't know if this behaviour has historical reasons, or a simple bug that 
got codified in the tests, but think that we should strive to sql compatible, 
and follow the standard.
   
   WDYT @gjacoby126 ?


----------------------------------------------------------------
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.

For queries about this service, please contact Infrastructure at:
[email protected]


Reply via email to