stoty commented on pull request #1133: URL: https://github.com/apache/phoenix/pull/1133#issuecomment-914045838
Some more analysis on possible alternative ways to solve this problem: (I think that the current patch is still correct, if not the most elegant way to do it) The pre-patch resolver will always (at least in every case in the tests) will (incorrectly) include the data table, and will return the ProjectedColumn object that was generated when projecting the table. With the current patch, the resolver will not consider the data table in some cases, and the Column object will be re-generated by ColumnRef, and will be a straight PColumnImpl instance. -- 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]
