zhangbutao commented on PR #4909: URL: https://github.com/apache/hive/pull/4909#issuecomment-1863894612
> @zhangbutao I see the PR to remove the test has failures, do you think we can fix this in 4.0 & chase the removal in 4.1.0(master)? @ayushtkn The `compute_stats` is used in multiple qtests as a udf to verify some bugs though it was not used in current hive source code, so it seems that removal is not easy & I am afraid of removal will make code instable now. Maybe need more investigation about the removal. As for this fix, it maybe a trivial improvement from a user UDF usage, as the current master code will never have use this code except some qtests. So feel free to merge this fix if you think ok. Thanks. -- 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]
