okumin commented on PR #5077: URL: https://github.com/apache/hive/pull/5077#issuecomment-2185456287
@dengzhhu653 @zabetak Thanks. I understand we need some more consensus on the expectations of HIVE-24167. We know the current PR doesn't work with some sets of Hive parameters. I have a question here, "Should HIVE-24167 let any SQLs with any set of parameters successfully run?". If the answer is yes, I recommend [avoiding the current approach "let the entire query crush"](https://github.com/apache/hive/pull/5077#discussion_r1598642918) and fixing known problems incrementally in the following tickets because it is unrealistic to prove that no issue happens under arbitrary conditions. If the answer is no, we may need a consensus on what queries/parameters we should support in HIVE-24167. -- 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]
