Hi Guys, Today a colleague of mine ran into a nasty NPE in the generated code. I found and fixed it. Can you please review it. I would really like to see this one go into the 1.8.2 is it is possible.
Thanks Niels Basjes On Mon, Oct 31, 2016 at 6:22 PM, S G <sg.online.em...@gmail.com> wrote: > Thanks for merging the patch Ryan. Appreciate it ! > > On Sun, Oct 30, 2016 at 1:59 PM, Ryan Blue <rb...@netflix.com.invalid> > wrote: > > > SG, > > > > I'll take a look at it. Thanks! > > > > rb > > > > On Sun, Oct 30, 2016 at 1:58 PM, S G <sg.online.em...@gmail.com> wrote: > > > > > Hey Ryan, > > > > > > Any chance https://github.com/apache/avro/pull/113 can make it to this > > > release? > > > Its been pending review for a couple months now. > > > > > > Thanks > > > SG > > > > > > On Sun, Oct 30, 2016 at 1:46 PM, Ryan Blue <b...@apache.org> wrote: > > > > > > > Hi everyone, > > > > > > > > I'm going to try to get a 1.8.2 release candidate out this week. > > There's > > > > only one issue that I think is a blocker on AVRO-1885, and I think we > > can > > > > get that bug fixed. If anyone would like to get other commits in or > > think > > > > there are more blockers, please link them to AVRO-1885 and we can > > discuss > > > > on that issue. > > > > > > > > Thanks! > > > > > > > > rb > > > > > > > > -- > > > > Ryan Blue > > > > > > > > > > > > > > > -- > > Ryan Blue > > Software Engineer > > Netflix > > > -- Best regards / Met vriendelijke groeten, Niels Basjes