You are viewing a plain text version of this content. The canonical link for it is here.
Posted to dev@arrow.apache.org by Julien Le Dem <ju...@dremio.com> on 2016/09/06 21:30:04 UTC
Arrow cpp travis-ci build broken
The Arrow cpp travis-ci build is broken right now because it depends on
parquet-cpp which has changed in an incompatible way. [1] [2] (or so it
looks to me)
Since parquet-cpp is not released yet it is totally fine to make
incompatible API changes.
However, we may want to pin the Arrow to Parquet dependency (on a git sha?)
to prevent cross project changes from breaking the master build.
Since I'm not one of the core cpp dev on those projects I mainly want to
start that conversation rather than prescribe a solution. Feel free to take
this as a straw man and suggest something else.
[1] https://travis-ci.org/apache/arrow/jobs/156080555
[2]
https://github.com/apache/arrow/blob/2d8ec789365f3c0f82b1f22d76160d5af150dd31/ci/travis_before_script_cpp.sh
--
Julien
Re: Arrow cpp travis-ci build broken
Posted by Wes McKinney <we...@gmail.com>.
Arrow builds should be fixed now hopefully. I'm rebasing the C++ patch queue.
On Tue, Sep 6, 2016 at 6:37 PM, Wes McKinney <we...@gmail.com> wrote:
> hi Julien,
>
> I'm very sorry about the inconvenience with this and the delay in
> getting it sorted out. I will triage this evening by disabling the
> Parquet tests in Arrow until we get the current problems under
> control. When we re-enable the Parquet tests in Travis CI I agree we
> should pin the version SHA.
>
> - Wes
>
> On Tue, Sep 6, 2016 at 5:30 PM, Julien Le Dem <ju...@dremio.com> wrote:
>> The Arrow cpp travis-ci build is broken right now because it depends on
>> parquet-cpp which has changed in an incompatible way. [1] [2] (or so it
>> looks to me)
>> Since parquet-cpp is not released yet it is totally fine to make
>> incompatible API changes.
>> However, we may want to pin the Arrow to Parquet dependency (on a git sha?)
>> to prevent cross project changes from breaking the master build.
>> Since I'm not one of the core cpp dev on those projects I mainly want to
>> start that conversation rather than prescribe a solution. Feel free to take
>> this as a straw man and suggest something else.
>>
>> [1] https://travis-ci.org/apache/arrow/jobs/156080555
>> [2]
>> https://github.com/apache/arrow/blob/2d8ec789365f3c0f82b1f22d76160d5af150dd31/ci/travis_before_script_cpp.sh
>>
>>
>> --
>> Julien
Re: Arrow cpp travis-ci build broken
Posted by Wes McKinney <we...@gmail.com>.
hi Julien,
I'm very sorry about the inconvenience with this and the delay in
getting it sorted out. I will triage this evening by disabling the
Parquet tests in Arrow until we get the current problems under
control. When we re-enable the Parquet tests in Travis CI I agree we
should pin the version SHA.
- Wes
On Tue, Sep 6, 2016 at 5:30 PM, Julien Le Dem <ju...@dremio.com> wrote:
> The Arrow cpp travis-ci build is broken right now because it depends on
> parquet-cpp which has changed in an incompatible way. [1] [2] (or so it
> looks to me)
> Since parquet-cpp is not released yet it is totally fine to make
> incompatible API changes.
> However, we may want to pin the Arrow to Parquet dependency (on a git sha?)
> to prevent cross project changes from breaking the master build.
> Since I'm not one of the core cpp dev on those projects I mainly want to
> start that conversation rather than prescribe a solution. Feel free to take
> this as a straw man and suggest something else.
>
> [1] https://travis-ci.org/apache/arrow/jobs/156080555
> [2]
> https://github.com/apache/arrow/blob/2d8ec789365f3c0f82b1f22d76160d5af150dd31/ci/travis_before_script_cpp.sh
>
>
> --
> Julien
Re: Arrow cpp travis-ci build broken
Posted by Wes McKinney <we...@gmail.com>.
hi Julien,
I'm very sorry about the inconvenience with this and the delay in
getting it sorted out. I will triage this evening by disabling the
Parquet tests in Arrow until we get the current problems under
control. When we re-enable the Parquet tests in Travis CI I agree we
should pin the version SHA.
- Wes
On Tue, Sep 6, 2016 at 5:30 PM, Julien Le Dem <ju...@dremio.com> wrote:
> The Arrow cpp travis-ci build is broken right now because it depends on
> parquet-cpp which has changed in an incompatible way. [1] [2] (or so it
> looks to me)
> Since parquet-cpp is not released yet it is totally fine to make
> incompatible API changes.
> However, we may want to pin the Arrow to Parquet dependency (on a git sha?)
> to prevent cross project changes from breaking the master build.
> Since I'm not one of the core cpp dev on those projects I mainly want to
> start that conversation rather than prescribe a solution. Feel free to take
> this as a straw man and suggest something else.
>
> [1] https://travis-ci.org/apache/arrow/jobs/156080555
> [2]
> https://github.com/apache/arrow/blob/2d8ec789365f3c0f82b1f22d76160d5af150dd31/ci/travis_before_script_cpp.sh
>
>
> --
> Julien