forked from brianfrankcooper/YCSB
-
Notifications
You must be signed in to change notification settings - Fork 12
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
Python3 support for bin/ycsb #12
Comments
A symptomatic fix would be to change the shebang in bin/ycsb to request python2 from env. The simplest fix would be the following patch: - #!/usr/bin/env python
+ #!/usr/bin/env python2 Anything beyond that will necessarily be more involved. It might be worth checking upstream how YCSB handles the python2 sunsetting. |
+1 for the env solution. Please file a PR. #hacktoberfest
Clemens Lieb <[email protected]> schrieb am So., 14. Okt. 2018,
14:49:
… A symptomatic fix would be to change the shebang in bin/ycsb to request
python2 from env.
It should be noted that python 2.x is sunsetted upstream. That implies a
migration to python 3 would be in order either way.
The simplest fix would be the following patch:
- #!/usr/bin/env python+ #!/usr/bin/env python2
Anything beyond that will necessarily be more involved. It might be worth
checking upstream how YCSB handles the python2 sunsetting.
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
<#12 (comment)>,
or mute the thread
<https://github.com/notifications/unsubscribe-auth/ABTaflL6dTjARqh15qbOty5U6X5-Mwesks5ukzK_gaJpZM4XbFnB>
.
|
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
I have set python3 as my default version of python, system wide.
As a result, if I run bin/ycsb the program fails and simply states a syntax-error...
Please add the functionality to check if python is compatible and if not, try using python2 command to run ycsb.
-Cheers.
The text was updated successfully, but these errors were encountered: