You are viewing a plain text version of this content. The canonical link for it is here.
Posted to dev@parquet.apache.org by Julien Le Dem <ju...@twitter.com.INVALID> on 2015/05/01 07:15:29 UTC
Re: [parquet-mr] PARQUET-227 Enforce that unions have only 1 set
value, tolerate bad records in read path (#153)
Hi Lukas,
Has this change been reviewed?
https://github.com/apache/parquet-mr/pull/183
https://issues.apache.org/jira/browse/PARQUET-175
https://github.com/apache/parquet-mr/commit/98f54c158acb12a26fa6f335b1665accd2aed347
To clarify: Changes need to be reviewed by a committer other than the
author of the change.
Please get a +1 from another committer before merging your pull requests.
Don't hesitate to ping people on the mailing list or by mentioning them in
the PR if necessary.
Julien
On Wed, Apr 29, 2015 at 11:19 PM, asfgit <no...@github.com> wrote:
> Closed #153 <https://github.com/apache/parquet-mr/pull/153> via 9993450
> <https://github.com/apache/parquet-mr/commit/9993450ad1f023e0e2b59291361d0b3b9f0e1c8d>
> .
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/apache/parquet-mr/pull/153#event-293713875>.
>
Re: [parquet-mr] PARQUET-227 Enforce that unions have only 1 set
value, tolerate bad records in read path (#153)
Posted by Julien Le Dem <ju...@twitter.com.INVALID>.
Sorry, I replied to the wrong commit message.
That was meant to the commit to #183
On Mon, May 4, 2015 at 4:07 PM, Alex Levenson <
alexlevenson@twitter.com.invalid> wrote:
> The subject of this email is misleading, but +1 to what Julien said
> regarding https://github.com/apache/parquet-mr/pull/183
>
> On Thu, Apr 30, 2015 at 10:15 PM, Julien Le Dem <julien@twitter.com.invalid
> >
> wrote:
>
> > Hi Lukas,
> > Has this change been reviewed?
> > https://github.com/apache/parquet-mr/pull/183
> > https://issues.apache.org/jira/browse/PARQUET-175
> >
> >
> https://github.com/apache/parquet-mr/commit/98f54c158acb12a26fa6f335b1665accd2aed347
> > To clarify: Changes need to be reviewed by a committer other than the
> > author of the change.
> > Please get a +1 from another committer before merging your pull requests.
> > Don't hesitate to ping people on the mailing list or by mentioning them
> in
> > the PR if necessary.
> > Julien
> >
> >
> >
> >
> >
> > On Wed, Apr 29, 2015 at 11:19 PM, asfgit <no...@github.com>
> wrote:
> >
> > > Closed #153 <https://github.com/apache/parquet-mr/pull/153> via
> 9993450
> > > <
> >
> https://github.com/apache/parquet-mr/commit/9993450ad1f023e0e2b59291361d0b3b9f0e1c8d
> > >
> > > .
> > >
> > > —
> > > Reply to this email directly or view it on GitHub
> > > <https://github.com/apache/parquet-mr/pull/153#event-293713875>.
> > >
> >
>
>
>
> --
> Alex Levenson
> @THISWILLWORK
>
Re: [parquet-mr] PARQUET-227 Enforce that unions have only 1 set
value, tolerate bad records in read path (#153)
Posted by Alex Levenson <al...@twitter.com.INVALID>.
The subject of this email is misleading, but +1 to what Julien said
regarding https://github.com/apache/parquet-mr/pull/183
On Thu, Apr 30, 2015 at 10:15 PM, Julien Le Dem <ju...@twitter.com.invalid>
wrote:
> Hi Lukas,
> Has this change been reviewed?
> https://github.com/apache/parquet-mr/pull/183
> https://issues.apache.org/jira/browse/PARQUET-175
>
> https://github.com/apache/parquet-mr/commit/98f54c158acb12a26fa6f335b1665accd2aed347
> To clarify: Changes need to be reviewed by a committer other than the
> author of the change.
> Please get a +1 from another committer before merging your pull requests.
> Don't hesitate to ping people on the mailing list or by mentioning them in
> the PR if necessary.
> Julien
>
>
>
>
>
> On Wed, Apr 29, 2015 at 11:19 PM, asfgit <no...@github.com> wrote:
>
> > Closed #153 <https://github.com/apache/parquet-mr/pull/153> via 9993450
> > <
> https://github.com/apache/parquet-mr/commit/9993450ad1f023e0e2b59291361d0b3b9f0e1c8d
> >
> > .
> >
> > —
> > Reply to this email directly or view it on GitHub
> > <https://github.com/apache/parquet-mr/pull/153#event-293713875>.
> >
>
--
Alex Levenson
@THISWILLWORK