Raghav Aggarwal

> Just for clarification. I recently reviewed #5036, which would rely on these files. I expect this PR will not require #5036 to be modified as the class in Apache...

@abstractdog, can you please help review this?

@ayushtkn, If I remove all the exclusions from the tez-protobuf-history-plugin, then no log4j or any hadoop dependency is coming. Attaching the dependency tree for reference. So, should we keep exclusions...

Have updated the PR, considered the suggestions from Laszlo, and have excluded hadoop dependencies which are used in https://github.com/apache/tez/blob/rel/release-0.10.3/tez-plugins/tez-protobuf-history-plugin/pom.xml

> I think you haven't all the excluded dependencies, earlier you had hadoop-mapreduce-client-core and others excluded in ql/pom.xml but now in the parent pom, they aren't excluded, why? Will add...

@abstractdog, can you please review this?

@zhangbutao, can we reborn this PR for master branch. This is a much needed patch. Also, can we expose **fs.s3a.security.credential.provider.path** in conf as well, it is another way to pass...

> > @zhangbutao, can we reborn this PR for master branch. This is a much needed patch. Also, can we expose **fs.s3a.security.credential.provider.path** in conf as well, it is another way...

+1 (non-binding)

@abstractdog @harishjp. Can you please help get this in tez 0.10.3