I have reviewed the changes and it looks good to me. I have also created a JIRA so that it is tracked better : https://issues.apache.org/jira/browse/AVRO-1988
Ill now try to get it in the 1.7 branch soon. -Suraj Acharya On Tue, Jan 24, 2017 at 2:40 PM, Ben McCann <[email protected]> wrote: > Thank you! > > On Tue, Jan 24, 2017 at 11:52 AM, suraj acharya <[email protected]> > wrote: > > > Hey Ben, > > Thanks a lot for this request. > > Ill look at it this week. And find someone to commit it by Monday of next > > week. > > > > > > > > -Suraj Acharya > > > > On Tue, Jan 24, 2017 at 12:50 PM, Ben McCann > <[email protected] > > > > > wrote: > > > > > Would someone be able to look at this PR? The tests are currently > broken > > > without it > > > > > > https://github.com/apache/avro/pull/139 > > > > > > Thanks, > > > Ben > > > > > > > > > On Sat, Jan 21, 2017 at 9:50 PM, Ben McCann <[email protected]> > > wrote: > > > > > > > Hi, > > > > > > > > I wanted to check and see if it'd be possible to get this PR merged > to > > > fix > > > > the currently broken tests. > > > > > > > > Thanks, > > > > Ben > > > > > > > > On Jan 17, 2017 1:09 PM, "Ben McCann" <[email protected]> wrote: > > > > > > > >> The tests on the 1.7 branch are currently broken due to a partially > > > >> applied commit. Would you be able to merge the following PR? I've > > > confirmed > > > >> it fixes the issue by applying the remainder of the commit. > > > >> > > > >> https://github.com/apache/avro/pull/139 > > > >> > > > >> Thanks, > > > >> Ben > > > >> > > > >> > > > > > >
