Bodor Laszlo

Results 179 comments of Bodor Laszlo

ready to merge, I checked the refactored patch on cluster, everything works as expected, only 1 ugi was created for a query: ``` query-executor 1 2024-09-16T08:28:43.780Z query-executor-0-0 query-executor 1 12b073f6-5537-44d0-8aca-578096c88c8d...

> As you mentioned: "NOTE: this patch won't compile until [TEZ-4548](https://issues.apache.org/jira/browse/TEZ-4548) is not released in Tez 0.10.4", I guess a pom.xml change would be necessary as well, but I don't...

> I had only minor questions, on overall, LGTM. thanks a lot, I addressed what was possible in https://github.com/apache/hive/pull/5174/commits/c2602866c753f0842a2d90bb7b48f237faeea188 and replied about the rest, please let me know if you...

@deniskuzZ : addressed all comments in https://github.com/apache/hive/pull/5174/commits/5b53236075f53fae497627b8e5f778df83b83ea6 unit tests added for the failure scenarios

> @abstractdog, are we still waiting for the Tez changes to land? yes, unfortunately, we cannot merge this until tez 0.10.4 is released, that's why I set it blocked by:...

rebasing the patch before merge

hm, we forgot about this, I'm reopening

PR is green, this is ready for review

> LGTM, but I would ask for @abstractdog opinion 1. can we fix the "SUFFLE" typos if we're already touching this part? 2. can you please share more details about...