[LIVY-848] Adjusting Livy test cases to be compatible with Python3#316
[LIVY-848] Adjusting Livy test cases to be compatible with Python3#316jameschen1519 wants to merge 2 commits intoapache:masterfrom
Conversation
|
Not too familiar with how the packages around here work, but it looks like the TravisCI log has some sort of versioning issue with setuptools_scm-6.0.1: `[INFO] ------------------< org.apache.livy:livy-python-api >------------------- File "/usr/lib/python2.7/distutils/core.py", line 111, in setup File "/tmp/easy_install-mAOVjT/pytest-runner-5.3.0/temp/easy_install-o_FWda/setuptools_scm-6.0.1/src/setuptools_scm/init.py", line 8, in This is irrelevant to this change, and appears to be caused by "print(*k)" not being valid syntax in Python 2.7 (since unpacking isn't supported in this way). Ideally, we would be downgrading the version of setuptools_scm to something before the version at pypa/setuptools-scm@22a44c1, or upgrading everything to Python 3. |
|
This pull request has been automatically marked as stale because it has had no activity for at least 3 months. If you are still working on this change or plan to move it forward, please leave a comment or push a new commit so we know to keep it open. Otherwise, this PR will be closed automatically in about one month. Thank you for your contribution to Apache Livy! |
What changes were proposed in this pull request?
At the moment, the Livy test cases error out when run with Python3 instead of Python2. Many of these errors are simple errors revolving around parentheses, or errors involved imports with changed names, and the changes are mostly backwards compatible (with mild tinkering required in the case of imports with adjusted locations).
https://issues.apache.org/jira/browse/LIVY-848
How was this patch tested?
This patch was tested with a slightly different build of Livy. While the current version of Livy was not explicitly tested, the fundamental reasoning behind each change in this PR should still be applicable and benefit the health of the repository.