Mailing-List: contact dev-help@avro.apache.org; run by ezmlm
Precedence: bulk
Reply-To: dev@avro.apache.org
MIME-Version: 1.0
In-Reply-To: <CAO4re1=Lb+GUGT32=4bf8EO8bLcYYg2FSgg6=USgoOfNeLU8Jg@mail.gmail.com>
References: <CAO4re1=cme2yJYYUMxrO_7RLBK-hOjPS1-Zh0cBRBauqWn6NHA@mail.gmail.com>
 <CAK7GPabfgOf0WeM8ao4MRSEtG2Zzed9pe=isUJZgh+TAzMwZ8Q@mail.gmail.com>
 <CADoiZqobAtZzgCztz8nohwG=nu+uNn_URp=vP_unqt5biTU7gQ@mail.gmail.com>
 <CADoiZqoZc1E7wBMSdK_uYSALqgmJdu+=Nwr90NKo08nSERkjug@mail.gmail.com>
 <CAO4re1neztV18qyWD1B1jWSW5+R9KiOukT9o5=q+msAbxhMRSQ@mail.gmail.com>
 <CADoiZqrGQPE6tKaKQYRwVmR1K+GL_Gd21+SJN1u=1iLA9HHYzw@mail.gmail.com> <CAO4re1=Lb+GUGT32=4bf8EO8bLcYYg2FSgg6=USgoOfNeLU8Jg@mail.gmail.com>
From: Suraj Acharya <suraj@apache.org>
Date: Tue, 28 Mar 2017 15:44:42 -0500
Message-ID: <CAFK5+R+MoOc_aQ3WE6UhJm7cqaTY+TjkZ3KtUTLO3t014TP5qQ@mail.gmail.com>
Subject: Re: [VOTE] Release Apache Avro 1.8.2 RC1
To: dev@avro.apache.org
Content-Type: multipart/alternative; boundary=001a114f6854c376bb054bd088cd
archived-at: Tue, 28 Mar 2017 20:45:05 -0000

--001a114f6854c376bb054bd088cd
Content-Type: text/plain; charset=UTF-8

Hi,
If the decision was to continue with the problem, and no one has any
objection I will try to build a RC sometime this week.

Hope that is cool with everyone.

Suraj

On Thu, Nov 10, 2016 at 10:48 AM, Ryan Blue <rblue@netflix.com.invalid>
wrote:

> Thanks, Niels. I think it's fine to continue with this problem in the RC so
> I'm still happy voting +1. Lets plan on getting a fix for this into the
> next RC, since this vote failed for other reasons.
>
> rb
>
> On Thu, Nov 10, 2016 at 8:18 AM, Niels Basjes <Niels@basjes.nl> wrote:
>
> > Ryan,
> >
> > When you fix this build problem I vote +1 again.
> > Thanks.
> >
> > Niels
> >
> > On Thu, Nov 10, 2016 at 5:04 PM, Ryan Blue <rblue@netflix.com.invalid>
> > wrote:
> >
> > > The C# issue was fixed by AVRO-1626 [1], which had to be reverted for
> the
> > > 1.8.2 RC because it caused build failures in docker. I reopened the
> issue
> > > so we can fix it for next release. Either way, the tests pass in the
> > common
> > > environment and I would rather not fail an RC for something like this.
> > >
> > > [1]: https://issues.apache.org/jira/browse/AVRO-1626
> > >
> > > On Thu, Nov 10, 2016 at 7:17 AM, Niels Basjes <Niels@basjes.nl> wrote:
> > >
> > > > Guys,
> > > >
> > > > Please have a look at this idea: https://github.com/apache/
> > avro/pull/153
> > > > If you agree/disagree please go ahead and change/fix/commit it.
> > > > I'll be at the Apache Con BigData in sevilla next week so I wil
> > probably
> > > be
> > > > rather busy.
> > > >
> > > > Oh and yes, this also means I'm changing my stance on Avro 1.8.2 RC1
> > to :
> > > > -1
> > > > This RC1 is not good enough due to this issue.
> > > >
> > > > Niels Basjes
> > > >
> > > > On Thu, Nov 10, 2016 at 1:59 PM, Niels Basjes <Niels@basjes.nl>
> wrote:
> > > >
> > > > > @Simon I find it strange you say that the docker is 'leaking'
> > something
> > > > so
> > > > > I did some testing.
> > > > > I ran the tests on my system using docker and everything worked.
> > > > > Then I remove all docker containers and images related to the avro
> > > build
> > > > > environment and ran it again.
> > > > > The second time indeed it failed
> > > > >
> > > > > My best guess right now is that
> > > > > 1) during the creation of the docker image we do and update of the
> > base
> > > > > system and install all the tools.
> > > > > 2) Upstream something relevant to us changed.
> > > > >
> > > > > People (like me) who have and existing image on their system the
> > > 'update'
> > > > > of the OS is not done because the local image was built with the
> > > > > instructions that are in the dockerfile.
> > > > > So "New" docker images are different because the base OS has the
> > > updates
> > > > > installed from a later point in time.
> > > > >
> > > > > I created https://issues.apache.org/jira/browse/AVRO-1956
> > > > > Working on a fix.
> > > > >
> > > > > Niels
> > > > >
> > > > >
> > > > >
> > > > > On Tue, Nov 8, 2016 at 10:52 PM, Simon Woodford <
> > > simonwoodford@gmail.com
> > > > >
> > > > > wrote:
> > > > >
> > > > >> -1 from me. C# tests fail to run. The source of the problem is as
> > > > follows:
> > > > >>
> > > > >> Prior to this tag, the lang/csharp/build.sh file contained the
> test
> > > > line:
> > > > >> nunit-console --framework=4.0 Avro.nunit
> > > > >>
> > > > >> From discussion with @jcustenborder, it seems like the C# tests
> > won't
> > > > run
> > > > >> on a Mac if we keep the --framework=4.0 in this line; removing it
> > (as
> > > > has
> > > > >> been done for 1.8.2) makes the tests run. But removing it makes
> the
> > > > tests
> > > > >> fail on my system (CentOS 7). This suggests that something is
> > missing
> > > > from
> > > > >> the docker setup and is being picked up from the host computer,
> but
> > I
> > > > >> haven't had a chance to investigate.
> > > > >>
> > > > >> Given that it's a choice between it passing for me or passing on
> all
> > > > Macs
> > > > >> everywhere, go ahead and release. I'll chase this up for the 1.8.3
> > > > >> release.
> > > > >> All tests other than the C# pass when running build.sh test from
> > > docker.
> > > > >>
> > > > >> Simon
> > > > >>
> > > > >>
> > > > >> On Sun, Nov 6, 2016 at 9:59 PM, Ryan Blue <blue@apache.org>
> wrote:
> > > > >>
> > > > >> > Hi everyone,
> > > > >> >
> > > > >> > I propose the following RC to be released as official Apache
> Avro
> > > > 1.8.2
> > > > >> > release.
> > > > >> >
> > > > >> > The commit id is 7ec35ea24ff0270586a26afbc6f6f530d272d1f7
> > > > >> > * This corresponds to the tag: release-1.8.2-rc1
> > > > >> > * https://github.com/apache/avro/tree/7ec35ea2
> > > > >> > *
> > > > >> > https://git-wip-us.apache.org/repos/asf/projects/repo?p=
> > > > >> > avro.git&a=commit&h=7ec35ea2
> > > > >> >
> > > > >> > The release tarball, signature, and checksums are here:
> > > > >> > * https://dist.apache.org/repos/dist/dev/avro/avro-1.8.2-rc1/
> > > > >> >
> > > > >> > You can find the KEYS file here:
> > > > >> > * https://dist.apache.org/repos/dist/release/avro/KEYS
> > > > >> >
> > > > >> > Binary artifacts for Java are staged in Nexus here:
> > > > >> > * https://repository.apache.org/content/groups/staging/org/
> > > > apache/avro/
> > > > >> >
> > > > >> > This release includes:
> > > > >> > * A spec for single-message Avro encoding and a Java
> > implementation
> > > > >> > * Java: Bug fixes for Java logical types
> > > > >> > * Java: Support for Decimal with specific classes
> > > > >> > * Ruby: A fix for compatibility when using snappy
> > > > >> > * Python 3: Updated to use the standard module-level logging
> > pattern
> > > > >> > * C++: Support for Boost >= 1.59
> > > > >> > * And more bug fixes...
> > > > >> >
> > > > >> > Please download, verify, and test, then please vote by the end
> of
> > > > >> > Wednesday, 9 November.
> > > > >> >
> > > > >> > [ ] +1 Release this as Apache Avro 1.8.2
> > > > >> > [ ] +0
> > > > >> > [ ] -1 Do not release this because...
> > > > >> >
> > > > >> >
> > > > >> > --
> > > > >> > Ryan Blue
> > > > >> >
> > > > >>
> > > > >
> > > > >
> > > > >
> > > > > --
> > > > > Best regards / Met vriendelijke groeten,
> > > > >
> > > > > Niels Basjes
> > > > >
> > > >
> > > >
> > > >
> > > > --
> > > > Best regards / Met vriendelijke groeten,
> > > >
> > > > Niels Basjes
> > > >
> > >
> > >
> > >
> > > --
> > > Ryan Blue
> > > Software Engineer
> > > Netflix
> > >
> >
> >
> >
> > --
> > Best regards / Met vriendelijke groeten,
> >
> > Niels Basjes
> >
>
>
>
> --
> Ryan Blue
> Software Engineer
> Netflix
>

--001a114f6854c376bb054bd088cd--