amahussein commented on pull request #2443:
URL: https://github.com/apache/hadoop/pull/2443#issuecomment-724992363
> Thanx for the update, the approach and changes looks good. But I think
Jenkins won't have run all the tests? If not, you can touch a line in
hadoop-project/pom.xml and increase the timeout in Jenkinsfile from 20 to 30
hrs, if post that everything is good we can merge the current change.
Thanks @ayushtkn, I made the change to the PR.
The failing unit tests are the usual suspects. Even {{TestDynamometerInfra}}
started to show up in all commits since yesterday.
> Out of curiosity: You don't need a new PR for a new change, you could have
force-pushed to your previous branch itself, it would have updated the same PR.
Yeah, thanks. I know I could force commits but I did not want to lose track
of the previous implementation.
----------------------------------------------------------------
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
For queries about this service, please contact Infrastructure at:
[email protected]
---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]
