Can you tell me how to reproduce? I'll take a look on Monday.
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> wrote: >> >> >> >>> On Dec 28, 2018, at 1:38 AM, Wes McKinney <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/> 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> >>> 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> 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> >>>>> 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 >>>>>> >>>>>> 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. >>>>>> >>>>>> >>>> >> > >
