You are viewing a plain text version of this content. The canonical link for it is here.
Posted to dev@arrow.apache.org by Antoine Pitrou <an...@python.org> on 2021/02/23 10:53:17 UTC
[C++][CI] Make "C++ on s390x" build mandatory?
Hello,
For a while we've had a big endian (s390x-based) build on Travis-CI.
The build is optional, meaning errors don't actually fail the CI.
The build has been reasonably stable for some time apart for some
occasional regressions, which often don't get spotted because the build
is reported as "green" anyway (because it's optional).
I propose we make the build mandatory, to avoid missing further
regressions. What do you think?
Regards
Antoine.
Re: [C++][CI] Make "C++ on s390x" build mandatory?
Posted by Bryan Cutler <cu...@gmail.com>.
+1 sgtm
On Tue, Feb 23, 2021, 9:47 AM Micah Kornfield <em...@gmail.com> wrote:
> +1, but let's keep an eye on it to make sure it remains stable.
>
> On Tue, Feb 23, 2021 at 5:34 AM Kazuaki Ishizaki <IS...@jp.ibm.com>
> wrote:
>
> > Thank you. +1 for this proposal,
> >
> > Kazuaki Ishizaki
> >
> >
> >
> > From: Benjamin Kietzman <be...@gmail.com>
> > To: dev <de...@arrow.apache.org>
> > Date: 2021/02/23 21:19
> > Subject: [EXTERNAL] Re: [C++][CI] Make "C++ on s390x" build
> > mandatory?
> >
> >
> >
> > +1 for making it mandatory
> >
> > On Tue, Feb 23, 2021, 07:07 Krisztián Szűcs <sz...@gmail.com>
> > wrote:
> >
> > > Hi!
> > >
> > > On Tue, Feb 23, 2021 at 11:53 AM Antoine Pitrou <an...@python.org>
> > > wrote:
> > > >
> > > >
> > > > Hello,
> > > >
> > > > For a while we've had a big endian (s390x-based) build on Travis-CI.
> > > > The build is optional, meaning errors don't actually fail the CI.
> > > >
> > > > The build has been reasonably stable for some time apart for some
> > > > occasional regressions, which often don't get spotted because the
> > build
> > > > is reported as "green" anyway (because it's optional).
> > > >
> > > > I propose we make the build mandatory, to avoid missing further
> > > > regressions. What do you think?
> > > Sounds good to me.
> > > >
> > > > Regards
> > > >
> > > > Antoine.
> > >
> >
> >
> >
> >
>
Re: [C++][CI] Make "C++ on s390x" build mandatory?
Posted by Micah Kornfield <em...@gmail.com>.
+1, but let's keep an eye on it to make sure it remains stable.
On Tue, Feb 23, 2021 at 5:34 AM Kazuaki Ishizaki <IS...@jp.ibm.com>
wrote:
> Thank you. +1 for this proposal,
>
> Kazuaki Ishizaki
>
>
>
> From: Benjamin Kietzman <be...@gmail.com>
> To: dev <de...@arrow.apache.org>
> Date: 2021/02/23 21:19
> Subject: [EXTERNAL] Re: [C++][CI] Make "C++ on s390x" build
> mandatory?
>
>
>
> +1 for making it mandatory
>
> On Tue, Feb 23, 2021, 07:07 Krisztián Szűcs <sz...@gmail.com>
> wrote:
>
> > Hi!
> >
> > On Tue, Feb 23, 2021 at 11:53 AM Antoine Pitrou <an...@python.org>
> > wrote:
> > >
> > >
> > > Hello,
> > >
> > > For a while we've had a big endian (s390x-based) build on Travis-CI.
> > > The build is optional, meaning errors don't actually fail the CI.
> > >
> > > The build has been reasonably stable for some time apart for some
> > > occasional regressions, which often don't get spotted because the
> build
> > > is reported as "green" anyway (because it's optional).
> > >
> > > I propose we make the build mandatory, to avoid missing further
> > > regressions. What do you think?
> > Sounds good to me.
> > >
> > > Regards
> > >
> > > Antoine.
> >
>
>
>
>
Re: [C++][CI] Make "C++ on s390x" build mandatory?
Posted by Kazuaki Ishizaki <IS...@jp.ibm.com>.
Thank you. +1 for this proposal,
Kazuaki Ishizaki
From: Benjamin Kietzman <be...@gmail.com>
To: dev <de...@arrow.apache.org>
Date: 2021/02/23 21:19
Subject: [EXTERNAL] Re: [C++][CI] Make "C++ on s390x" build
mandatory?
+1 for making it mandatory
On Tue, Feb 23, 2021, 07:07 Krisztián Szűcs <sz...@gmail.com>
wrote:
> Hi!
>
> On Tue, Feb 23, 2021 at 11:53 AM Antoine Pitrou <an...@python.org>
> wrote:
> >
> >
> > Hello,
> >
> > For a while we've had a big endian (s390x-based) build on Travis-CI.
> > The build is optional, meaning errors don't actually fail the CI.
> >
> > The build has been reasonably stable for some time apart for some
> > occasional regressions, which often don't get spotted because the
build
> > is reported as "green" anyway (because it's optional).
> >
> > I propose we make the build mandatory, to avoid missing further
> > regressions. What do you think?
> Sounds good to me.
> >
> > Regards
> >
> > Antoine.
>
Re: [C++][CI] Make "C++ on s390x" build mandatory?
Posted by Benjamin Kietzman <be...@gmail.com>.
+1 for making it mandatory
On Tue, Feb 23, 2021, 07:07 Krisztián Szűcs <sz...@gmail.com>
wrote:
> Hi!
>
> On Tue, Feb 23, 2021 at 11:53 AM Antoine Pitrou <an...@python.org>
> wrote:
> >
> >
> > Hello,
> >
> > For a while we've had a big endian (s390x-based) build on Travis-CI.
> > The build is optional, meaning errors don't actually fail the CI.
> >
> > The build has been reasonably stable for some time apart for some
> > occasional regressions, which often don't get spotted because the build
> > is reported as "green" anyway (because it's optional).
> >
> > I propose we make the build mandatory, to avoid missing further
> > regressions. What do you think?
> Sounds good to me.
> >
> > Regards
> >
> > Antoine.
>
Re: [C++][CI] Make "C++ on s390x" build mandatory?
Posted by Krisztián Szűcs <sz...@gmail.com>.
Hi!
On Tue, Feb 23, 2021 at 11:53 AM Antoine Pitrou <an...@python.org> wrote:
>
>
> Hello,
>
> For a while we've had a big endian (s390x-based) build on Travis-CI.
> The build is optional, meaning errors don't actually fail the CI.
>
> The build has been reasonably stable for some time apart for some
> occasional regressions, which often don't get spotted because the build
> is reported as "green" anyway (because it's optional).
>
> I propose we make the build mandatory, to avoid missing further
> regressions. What do you think?
Sounds good to me.
>
> Regards
>
> Antoine.