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>


> On Jan 6, 2019, at 4:23 PM, Ravindra Pindikura <ravin...@dremio.com> wrote:
> 
> 
> 
>> On Jan 6, 2019, at 3:38 PM, Antoine Pitrou <anto...@python.org 
>> <mailto:anto...@python.org>> wrote:
>> 
>> 
>> Can you tell me how to reproduce?  I'll take a look on Monday.
> 
> 
> Thank you. I used the following steps to repro (on ubuntu 14) :
> 
> 1. conda section to setup the python environment
> 
> https://github.com/apache/arrow/blob/master/docs/source/python/development.rst
>  
> <https://github.com/apache/arrow/blob/master/docs/source/python/development.rst>
> 
> 2. Build python gandiva tests
> 
> python setup.py build_ext --build-type=$ARROW_BUILD_TYPE --with-gandiva 
> —inplace
> 
> 3. Run python gandiva tests
> 
> pytest pyarrow --only-gandiva
> 
> The test silently dies without printing any error msg. So, I modified 
> test_gandiva.py to use pdb and stepped through till I got the error 
> (pdb.set_trace() in test_tree_expr_builder()).
> 
> Thanks & regards,
> Ravindra.
> 
>> 
>> Regards
>> 
>> Antoine.
>> 
>> 
>> Le 06/01/2019 à 10:24, Ravindra Pindikura a écrit :
>>> 
>>> 
>>>> On Dec 29, 2018, at 8:03 PM, Ravindra Pindikura <ravin...@dremio.com 
>>>> <mailto:ravin...@dremio.com>> wrote:
>>>> 
>>>> 
>>>> 
>>>>> On Dec 28, 2018, at 1:38 AM, Wes McKinney <wesmck...@gmail.com 
>>>>> <mailto:wesmck...@gmail.com>> wrote:
>>>>> 
>>>>> You might have to wait for Antoine's help after the holidays. In the
>>>>> meantime: do you get the error when building both with gcc 4.8/4.9 and
>>>>> clang-6.0? I did a bit of google searching and it seems that iostream
>>>>> is a common troublemaker for LLVM. I wonder if the code can be
>>>>> refactored to avoid using std::cout
>>>> 
>>>> Thanks Wes.
>>>> 
>>>> I’ll wait for Antoine. In the meanwhile, I’ll work on the refactoring, 
>>>> just in case.
>>> 
>>> Antoine, 
>>> 
>>> Can you please give your opinion on this ? I’ve hit this with java and 
>>> python. Fairly sure it’ll be a problem on windows too.
>>> 
>>> Should I just break down arrow/util/decimal.cc <http://decimal.cc/> 
>>> <http://decimal.cc/ <http://decimal.cc/>>  into two parts : one that works 
>>> with llvm ir (no iostream), and the remaining
>>> 
>>> Thanks & regards,
>>> ravindra.
>>> 
>>>> 
>>>> 
>>>>> 
>>>>> On Thu, Dec 27, 2018 at 3:56 AM Ravindra Pindikura <ravin...@dremio.com 
>>>>> <mailto:ravin...@dremio.com>> wrote:
>>>>>> 
>>>>>> Thanks Wes, that worked.
>>>>>> 
>>>>>> When I run the gandiva tests (I did a step-through with pdb), I get this 
>>>>>> error :
>>>>>> 
>>>>>>> /home/ravindra/git/arrow/python/pyarrow/tests/test_gandiva.py(50)test_tree_exp_builder()
>>>>>> -> schema, [expr], pa.default_memory_pool())
>>>>>> (Pdb)
>>>>>> LLVM ERROR: Program used external function '_ZNSt8ios_base4InitC1Ev' 
>>>>>> which could not be resolved!
>>>>>> 
>>>>>> I had hit the same issue with java/jni, and resolved it by doing a 
>>>>>> NativeLibrary.getInstance() (thanks to @pitrou). My knowledge of python 
>>>>>> is limited - can someone please help with the same for python ?
>>>>>> 
>>>>>> Thanks & regards,
>>>>>> ravindra.
>>>>>> 
>>>>>>> On Dec 26, 2018, at 7:55 PM, Wes McKinney <wesmck...@gmail.com 
>>>>>>> <mailto:wesmck...@gmail.com>> wrote:
>>>>>>> 
>>>>>>> hi Ravindra,
>>>>>>> 
>>>>>>> If you are using Anaconda, you need to use the conda instructions for
>>>>>>> environment setup instead of virtualenv. This should be called out
>>>>>>> more prominently in a warning in the docs
>>>>>>> 
>>>>>>> - Wes
>>>>>>> 
>>>>>>> On Tue, Dec 25, 2018 at 12:53 PM Ravindra Pindikura 
>>>>>>> <ravin...@dremio.com <mailto:ravin...@dremio.com>> wrote:
>>>>>>>> 
>>>>>>>> Hi,
>>>>>>>> 
>>>>>>>> My change (decimal support in gandiva) causes a CI failure in python 
>>>>>>>> test_gandiva.py
>>>>>>>> 
>>>>>>>> https://travis-ci.org/apache/arrow/jobs/471874254#L7167 
>>>>>>>> <https://travis-ci.org/apache/arrow/jobs/471874254#L7167>
>>>>>>>> 
>>>>>>>> However, the travis log has no further information on the failure.
>>>>>>>> 
>>>>>>>> To debug this, I tried setting up a python dev environment on ubuntu 
>>>>>>>> 14  using the steps in 
>>>>>>>> https://github.com/apache/arrow/blob/master/docs/source/python/development.rst
>>>>>>>> 
>>>>>>>> I got stuck at this (virtualenv pyarrow):
>>>>>>>> 
>>>>>>>> ValueError: failed to parse CPython sys.version: '3.6.7 |Anaconda, 
>>>>>>>> Inc.| (default, Oct 23 2018, 19:19:38) \n[GCC 7.3.0]'
>>>>>>>> 
>>>>>>>> 
>>>>>>>> Is this a known issue ? Any workaround ?
>>>>>>>> 
>>>>>>>> Thanks & regards,
>>>>>>>> Ravindra.
> 

Reply via email to