-
Notifications
You must be signed in to change notification settings - Fork 9.1k
HADOOP-17472. Upgrade hadoop default version in hadoop.tools.dynamometer.TestDynamometerInfra #2622
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Conversation
|
cc @sunchao |
|
💔 -1 overall
This message was automatically generated. |
|
Got another error now: |
|
There are more test failures even after this. It seems this test has been broken for some time. cc @xkrogen . |
|
Hmm, I manually commented the line comparing Is this test still valid to have? |
|
Thanks for the ping @sunchao . I'll try to take a look and see if I can figure out what's wrong. |
|
I haven't been able to make it this test run locally, I am getting errors like: AFAICT it looks like both Regarding This dir gets cleaned up by default when the test exits, so you may want to comment out line 296 to avoid deleting it and losing the output. Also re:
The right long term fix should be HDFS-14412, which will use the local Hadoop build instead of localizing it from the web... But not sure what to do in the short term. |
|
Thanks @xkrogen. Let me check the log files to see if there is any clue. |
|
With #2537 in, what is the status of this? |
|
Does #2537 help this issue? Let me sync up with trunk. |
|
Hi @viirya, are you still working on this? |
|
As we will release 3.3.1 soon, I'd like to rework this issue in #2471. |
See https://issues.apache.org/jira/browse/HADOOP-17472 for details.
The CI found a failed test
hadoop.tools.dynamometer.TestDynamometerInfrain #2611.Seems currently
hadoop.tools.dynamometer.TestDynamometerInfrain the trunk is failed.There is no more 3.1.3 in http://mirrors.ocf.berkeley.edu/apache/hadoop/common/. So I guess we need to upgrade it to 3.1.4.