Casey Ching created KUDU-1456:
---------------------------------

             Summary: Kudu python module should not require unittest2
                 Key: KUDU-1456
                 URL: https://issues.apache.org/jira/browse/KUDU-1456
             Project: Kudu
          Issue Type: Improvement
          Components: python
    Affects Versions: 0.8.0
            Reporter: Casey Ching


Trying to use the python client on centos 6 fails if unittest2 isn't installed 

{noformat}
...          from kudu import connect as kudu_connect
23:22:04   File 
"/data/jenkins/workspace/impala-external-gerrit-verify-merge/repos/Impala/infra/python/env/lib/python2.6/site-packages/kudu/__init__.py",
 line 18, in <module>
23:22:04     from kudu.client import (Client, Table, Scanner, Session,  # noqa
23:22:04   File "kudu/schema.pxd", line 24, in init kudu.client 
(kudu/client.cpp:18830)
23:22:04     cdef class KuduType(object):
23:22:04   File "kudu/schema.pyx", line 23, in init kudu.schema 
(kudu/schema.cpp:8622)
23:22:04     from kudu.compat import tobytes, frombytes
23:22:04   File 
"/data/jenkins/workspace/impala-external-gerrit-verify-merge/repos/Impala/infra/python/env/lib/python2.6/site-packages/kudu/compat.py",
 line 34, in <module>
23:22:04     import unittest2 as unittest
23:22:04 ImportError: No module named unittest2
{noformat}

compat.py:

{code}
if PY26:
    import unittest2 as unittest
else:
    import unittest
{code}

Any testing-only modules shouldn't be required. Moving them to something like 
"test_compat.py" would be nice.



--
This message was sent by Atlassian JIRA
(v6.3.4#6332)

Reply via email to