Bodor Laszlo

TEZ-4419 is resolved, I think we can close this PR please reopen if I'm wrong

for reference, what is the current failure for Java11 build? I remember we had some tickets for Java11 compile-time compatibility

I had some comments earlier, if you can confirm you addressed those @dengzhhu653 , I'll take a second look and approve, it's time to merge this I believe please create...

I believe we're quite close, also I admit that we don't have a clear policy regarding the tez dependencies and where to exclude hadoop stuff from there, so I believe,...

latest patch LGTM +1

@LA-Toth: is there a chance you can check the patch with JDK11? I tried your commits at https://github.com/apache/tez/pull/296 and I found that the new github action step fails for building...

this is strange, I wanted to test by building locally, but I can see that in jdk8 the problematic module is not included at all: ``` cd tez-tools mvn --batch-mode...

> We have upgraded guava: https://issues.apache.org/jira/browse/TEZ-4504 and in the latest guava version post 30+ I believe it is MIT licensed, So, we are sorted.. > > I think we can...

thanks @ayushtkn for the review! can you take a look at this @rbalamohan , @sidseth please?

> We have `0.14.1` now, maybe we can chase that? yes I think so as far as I remember, precommit picks up the Jenkinsfile from the PR, so we can...