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: us...@infra.apache.org --------------------------------------------------------------------- To unsubscribe, e-mail: common-issues-unsubscr...@hadoop.apache.org For additional commands, e-mail: common-issues-h...@hadoop.apache.org