+1 sgtm On Tue, Feb 23, 2021, 9:47 AM Micah Kornfield <emkornfi...@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 <ishiz...@jp.ibm.com> > wrote: > > > Thank you. +1 for this proposal, > > > > Kazuaki Ishizaki > > > > > > > > From: Benjamin Kietzman <bengil...@gmail.com> > > To: dev <dev@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 <szucs.kriszt...@gmail.com> > > wrote: > > > > > Hi! > > > > > > On Tue, Feb 23, 2021 at 11:53 AM Antoine Pitrou <anto...@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. > > > > > > > > > > > >
