Shohei Okumiya
Shohei Okumiya
Thanks. Would you mind if we prioritized other PRs? I agree with the goal of HIVE-28595. However, SemanticAnalyzer is complex and critical for data integrity. We might need to review...
@KiranVelumuri Thanks for the reminder. I'm delighted that you turned to me. But you may ask another person if you would like to get this merged. For transparency, [I am...
@JeongDaeKim Taking a glance, your finding and fix seems to be very reasonable. Could you please regenerate the result file of `udaf_collect_set_2.q`?
Sure. I was not confident that I should review coworker's patch to avoid bias. I will check this anyway.
Looks good to me, but we're running additional in-house integration tests just in case.
Update: I want to run some additional tests, but I haven't taken the time. I will update the status in 2 weeks
I rebased this branch and also added two cosmetic changes.
I'm checking why TestDAGRecovery failed
@abstractdog Sorry, my hand-made refactoring included a mistake. I copy-pasted the method names used in the original condition. I appreciate it if you could double-check it. https://github.com/apache/tez/pull/361/commits/bdece70288d20463b131d852511128103f093cdf