Raghav Aggarwal

> > I'm speculating and might be wrong but is it possible that the `-Dmaven.repo.local=/home/jenkins/jenkins-home/workspace/tez-multibranch_PR-441/yetus-m2/tez-master-patch-0` is gone by the time this ran? > > ``` > > /patch-mvninstall-tez-tests.txt > >...

The goal was to keep the changes at minimum and convert to python3. Otherwise there is scope of improvement in scripts by using `argparser` instead of `sys.argv`, `pathlib` instead of...

> LGTM @Aggarwal-Raghav can you please check the above failures in yetus report. thanks ! thanks for the review @maheshrajus , sure, I'll fix the failures.

@maheshrajus @abstractdog , it seems that the pylint is not working in yetus based on [/branch-pylint-stderr.txt](https://ci-hadoop.apache.org/job/tez-multibranch/job/PR-444/1/artifact/out/branch-pylint-stderr.txt) , As i ran the pylint in local, there are lot of issues unrelated...

i have removed the tabs with space (just indentation change). that should fix the [/blanks-eol.txt](https://ci-hadoop.apache.org/job/tez-multibranch/job/PR-444/1/artifact/out/blanks-eol.txt) [/blanks-tabs.txt](https://ci-hadoop.apache.org/job/tez-multibranch/job/PR-444/1/artifact/out/blanks-tabs.txt) and requirements.txt for now

Earlier pylint rating: ``` logsplit.py: Your code has been rated at 6.85/10 swimlane.py: Your code has been rated at 0.00/10 counter-diff.py: Your code has been rated at 0.00/10 amlogparser.py: Your...

> patch looks good to me now, thanks @Aggarwal-Raghav, I'm merging this soon can you please create a follow-up for the pylint issue? Thanks for merging this @abstractdog . I'll...