logan-keede commented on PR #16186:
URL: https://github.com/apache/datafusion/pull/16186#issuecomment-2917556654

   Currently error look like:-
   ```sql 
   > explain SELECT e1.employee_name, e1.salary
   FROM employees e1
   WHERE e1.salary > (
       SELECT AVG(e2.salary)
       FROM employees e2 
       WHERE e2.dept_id = e1.dept_id
       AND e2.salary > (
           SELECT AVG(e3.salary)
           FROM employees e3
           WHERE e3.dept_id = e1.dept_id
       )
   );
   Schema error: No field named e1.dept_id. Did you mean 'e3.dept_id'?.
   > explain SELECT e1.employee_name, e1.salary
   FROM employees e1
   WHERE e1.salary > (
       SELECT AVG(e2.salary)
       FROM employees e2 
       WHERE e2.dept_id = e1.dept_id
       AND e2.salary > (
           SELECT AVG(e3.salary)
           FROM employees e3
           WHERE e3.dept_id = e1.depd
       )
   );
   Schema error: No field named e1.depd. Valid fields are e3.employee_id, 
e3.employee_name, e3.dept_id, e3.salary.
   >
   ```
   After This PR:-
   ```sql
   > explain SELECT e1.employee_name, e1.salary
   FROM employees e1
   WHERE e1.salary > (
       SELECT AVG(e2.salary)
       FROM employees e2 
       WHERE e2.dept_id = e1.dept_id
       AND e2.salary > (
           SELECT AVG(e3.salary)
           FROM employees e3
           WHERE e3.dept_id = e1.depd
       )
   );
   Schema error: No field named e1.depd. Valid fields are e3.employee_id, 
e3.employee_name, e3.dept_id, e3.salary.
   > explain SELECT e1.employee_name, e1.salary
   FROM employees e1
   WHERE e1.salary > (
       SELECT AVG(e2.salary)
       FROM employees e2 
       WHERE e2.dept_id = e1.dept_id
       AND e2.salary > (
           SELECT AVG(e3.salary)
           FROM employees e3
           WHERE e3.dept_id = e1.dept_id
       )
   );
   Schema error: No field named e1.dept_id. Valid fields are e2.salary, 
__scalar_sq_2."avg(e3.salary)", __scalar_sq_2.dept_id.
   ```
   The results are a little inconsistent.  `__scalar_sq_2."avg(e3.salary)", 
__scalar_sq_2.dept_id` are not valid fields in the above context.
   Ideally, all the field in e1, e2 and e3 should come up here as they are 
valid. 
   


-- 
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: github-unsubscr...@datafusion.apache.org

For queries about this service, please contact Infrastructure at:
us...@infra.apache.org


---------------------------------------------------------------------
To unsubscribe, e-mail: github-unsubscr...@datafusion.apache.org
For additional commands, e-mail: github-h...@datafusion.apache.org

Reply via email to