You are viewing a plain text version of this content. The canonical link for it is here.
Posted to dev@metron.apache.org by Shane Ardell <sh...@gmail.com> on 2019/04/03 10:06:37 UTC

[DISCUSS] Reverting METRON-2023 (PR #1364)

Hello everyone,

I'm sending out this email to notify the community of my intention to
revert the commit for METRON-2023 (PR #1364
<https://github.com/apache/metron/pull/1364>). This commit introduced a bug
stemming from yauzl, the tool that Cypress uses to unzip itself. This bug
causes an error in Travis and is intermittent, which is how it passed
multiple build attempts before being merged into master. The issue is
currently being discussed with the Cypress team and can be viewed here
<https://github.com/cypress-io/cypress/issues/3515>.

If anyone has any strong objection to this, please let me know here.
Otherwise, I plan on reverting this in the next 24 hours.

Shane

Re: [DISCUSS] Reverting METRON-2023 (PR #1364)

Posted by Michael Miklavcic <mi...@gmail.com>.
+1 from me also

On Wed, Apr 3, 2019 at 6:18 AM Nick Allen <ni...@nickallen.org> wrote:

> +1
> Thanks for digging into this.
>
> On Wed, Apr 3, 2019, 6:07 AM Shane Ardell <sh...@gmail.com>
> wrote:
>
> > Hello everyone,
> >
> > I'm sending out this email to notify the community of my intention to
> > revert the commit for METRON-2023 (PR #1364
> > <https://github.com/apache/metron/pull/1364>). This commit introduced a
> > bug
> > stemming from yauzl, the tool that Cypress uses to unzip itself. This bug
> > causes an error in Travis and is intermittent, which is how it passed
> > multiple build attempts before being merged into master. The issue is
> > currently being discussed with the Cypress team and can be viewed here
> > <https://github.com/cypress-io/cypress/issues/3515>.
> >
> > If anyone has any strong objection to this, please let me know here.
> > Otherwise, I plan on reverting this in the next 24 hours.
> >
> > Shane
> >
>

Re: [DISCUSS] Reverting METRON-2023 (PR #1364)

Posted by Nick Allen <ni...@nickallen.org>.
+1
Thanks for digging into this.

On Wed, Apr 3, 2019, 6:07 AM Shane Ardell <sh...@gmail.com> wrote:

> Hello everyone,
>
> I'm sending out this email to notify the community of my intention to
> revert the commit for METRON-2023 (PR #1364
> <https://github.com/apache/metron/pull/1364>). This commit introduced a
> bug
> stemming from yauzl, the tool that Cypress uses to unzip itself. This bug
> causes an error in Travis and is intermittent, which is how it passed
> multiple build attempts before being merged into master. The issue is
> currently being discussed with the Cypress team and can be viewed here
> <https://github.com/cypress-io/cypress/issues/3515>.
>
> If anyone has any strong objection to this, please let me know here.
> Otherwise, I plan on reverting this in the next 24 hours.
>
> Shane
>