[GitHub] [spark] gengliangwang commented on pull request #39596: [SPARK-42084][SQL] Avoid leaking the qualified-access-only restriction

Tue, 17 Jan 2023 15:27:39 -0800 


gengliangwang commented on PR #39596:
URL: https://github.com/apache/spark/pull/39596#issuecomment-1386230467

   <img width="813" alt="image" 
src="https://user-images.githubusercontent.com/1097932/213033693-8baa37b2-349b-4018-9234-84d8579d60c0.png";>
   <img width="755" alt="image" 
src="https://user-images.githubusercontent.com/1097932/213033767-cfae7b73-f67b-4621-aef8-ea3b26d5ffb3.png";>
   
   @cloud-fan I feel the `qualifiedAccessOnly` attributes are still 
complicated. We have to check them here and there. Since it is for certain 
`Join`s only, shall we decouple it from the `metadataOutput` and have a new 
TreeNode method to infer these hidden lists? E.g. `hiddenOutputFromJoin`. In 
this way, we don't need to mark whether an attribute is `qualifiedAccessOnly`.


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


---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]

Reply via email to