-
Notifications
You must be signed in to change notification settings - Fork 128
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
Improvement of the Presto Example : #271
base: unreleased
Are you sure you want to change the base?
Conversation
If you have any questions on my PR, feel free |
fix: dev: SDK-364: Oracle Multi AD changes required in qds-sdk
- Simplified a lot the whole script - Made everything compatible with Python 3 - Removed the print() for proper logger - Added assert() for parameters sanity checks - Allows passing keywords arguments to pd.read_csv() ...
Help for the bug of the missing column names
Hey guys,
Now, even if Qubole query got stuck, the function still return with error. |
Also, it seems that Travis CI is failing to install Python 3.3 and 2.6 (??) but I don't think it's related to this PR |
yes @JeanLescut looks like the Ubuntu version was not pinned in our configuration and hence it got updated to 16.XX in which these python versions are not present, will fix it. Thanks for the PR btw |
@JeanLescut I have fixed the issue, please rebase your branch once more and squash all your commits. |
8a95947
to
caec9b1
Compare
...