Aggarwal-Raghav commented on PR #6253:
URL: https://github.com/apache/hive/pull/6253#issuecomment-3700056335

   LGTM +1 (non-binding)
   I was able to repro the issue
   <img width="1340" height="186" alt="Screenshot 2025-12-30 at 10 38 44 PM" 
src="https://github.com/user-attachments/assets/23a114f3-b1b0-42c4-b5bf-17cd389150f4";
 />
   
   @tanishq-chugh , for select query, there are 4 entry (1 table, 3 partitions 
)instead of 3 as mentioned in the PR description. Please update the PR 
description.
   
   Just thinking out loud 🤔, if there is any benefit of keeping 
`entity.getType() == PARTITION` as table name will always be present in a SQL 
query and partition alone can't exist.


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