IMHO we can simpler skip the Gandiva tests on Python 2.7.

Regards

Antoine.


Le 07/01/2019 à 21:12, Ravindra Pindikura a écrit :
> Thanks Antoine. 
> 
> This works on my test machine. But fails in CI. I’m guessing it’s a python 
> 2.7 issue.
> 
> https://travis-ci.org/apache/arrow/jobs/476469678 
> <https://travis-ci.org/apache/arrow/jobs/476469678>
> 
> 
> ImportError while loading conftest 
> '/home/travis/build/apache/arrow/python/pyarrow/tests/conftest.py'.
>  <>../pyarrow-test-2.7/lib/python2.7/site-packages/six.py:709: in exec_
>  <>    exec("""exec _code_ in _globs_, _locs_""")
>  <>pyarrow/tests/conftest.py:66: in <module>
>  <>    import pyarrow.gandiva # noqa
>  <>pyarrow/gandiva.pyx:85: in init pyarrow.gandiva
>  <>    _dll = ctypes.CDLL(__file__, ctypes.RTLD_GLOBAL)
>  <>E   NameError: name '__file__' is not defined
> 
> 
> 
> 
> 
>> On Jan 7, 2019, at 8:05 PM, Antoine Pitrou <anto...@python.org> wrote:
>>
>>
>> Hi Ravindra,
>>
>> I've found that it's the same issue as previously in Java.
>> Here is a workaround that works here:
>> https://gist.github.com/pitrou/7eed7b980eec3187d619bc146b75cc4b
>>
>> Regards
>>
>> Antoine.
>>
>>
>> Le 07/01/2019 à 07:16, Ravindra Pindikura a écrit :
>>> I missed mentioning this. The repro requires my changes for decimal (in the 
>>> PR).
>>>
>>> https://github.com/apache/arrow/pull/2942 
>>> <https://github.com/apache/arrow/pull/2942>
> 
> 

Reply via email to