Aggarwal-Raghav commented on PR #5976: URL: https://github.com/apache/hive/pull/5976#issuecomment-3124295980
**NOTE**: Just want to re-iterate and highlight the breaking change here. Anyway as +1 is there on the PR which means it's ok from dev perspective. Have give some nit comments for improving code readability which are purely optional. **Before this PR:** <img width="774" height="480" alt="scenarion2_3_without_patch" src="https://github.com/user-attachments/assets/c8ba93dd-00a3-4006-ac71-22390e1bfa1e"; /> **After this PR:** <img width="1512" height="749" alt="Screenshot 2025-07-27 at 3 43 39 PM" src="https://github.com/user-attachments/assets/2e477ead-b0fe-4f4d-84e2-ec538a9e8789"; /> -- 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: gitbox-unsubscr...@hive.apache.org For queries about this service, please contact Infrastructure at: us...@infra.apache.org --------------------------------------------------------------------- To unsubscribe, e-mail: gitbox-unsubscr...@hive.apache.org For additional commands, e-mail: gitbox-h...@hive.apache.org
