dropbox / dropbox/PyHive

Support for Juno/iPad - Consider removing dependency on past.builtins and future.utils

Open
#353 0 comments 0 reactions 0 assignees View on GitHub
Dominant language
Python
Stars
1.7k
Forks
545
PR merge metrics
No merged PRs in 30d

Description

Hi, I’ve been using Juno (Jupyter on iPad) and I had to make two small changes to PyHive to get it to work.

Juno only supports native modules and doesn’t support past.builtins. You only seem to use to in escape_item so I replaced the isinstance of basestring with isinstance of str, unicode. That may break Python 2 compatibility. I’ve never used python 2...

Juno also doesn’t support future.utils so I replace it with a link to six (which it does support). Six implements with_metaclass so I just replaced the import to future.utils to an import to six.

I’m a python novice (I’m only as few weeks into the language) so I’m not sure what I’m doing, but I thought that this might be helpful.

Contributor guide

No contributing guide indexed for this repository

Assessment

This issue has not been assessed yet.

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.