-
Notifications
You must be signed in to change notification settings - Fork 299
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
Updated version of some tools in main script, since some mirrors have removed old versions #21
base: master
Are you sure you want to change the base?
Conversation
Thank you for submitting this pull request, however I do not see a valid CLA on file for you. Before we can merge this request please visit https://yahoocla.herokuapp.com/ and agree to the terms. Thanks! 😄 |
CLA is valid! |
Changing
|
I had same issue, this change did work, please merge it. |
When I run the tests with this patch I still see an empty seen.txt on spark. @ivanliu what platform are you running on that has this all work? |
@revans2 I'm running the script on Mac OS (10.11.2). Without this patch, some components cannot get installed with stream-bench.sh SETUP. I tried STORM_TEST, it seemed to work, but SPARK_TEST gave me empty seen.txt. |
@ivanliu that is the real problem we can run the benchmark but it does not really do anything and we are still trying to debug why that is. |
The setup fails for some tools, needed this minimal change to get it going.