schizophrenicmaniac commented on PR #40294: URL: https://github.com/apache/shardingsphere/pull/40294#issuecomment-6460803699
@terrymanu Thanks, all three are fixed in the latest commit. The label check compared the outer label to the physical column name ignoring case, so `"ID"` over `id` counted as unchanged and fell back to `id`. It now uses the column's bound info instead: any column that comes from a derived table or CTE keeps its exact outer label, and plain physical columns behave as before. I added a test for `SELECT d."ID", d.* FROM (SELECT id AS "ID" FROM foo_tbl) d` that checks both columns are described as `ID`. `getTable` has `@HighFrequencyInvocation` again, and `findSourceProjectionSegment` is declared on one line. -- 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]
