Okay. I got it.
I'm going to pass CI.
Thank you Alexander.

2015-12-01 19:16 GMT+09:00 Alexander Bezzubov <[email protected]>:

> Yes, I know and have the same problem with #497
>
> The usual way such problems to be handled though is not just merge it, but
> - either try re-trigger the CI to see if the is flanky test indeed (is not
> reproduced then)
> - or try to fix the test a new PR, merge it first
>
> and then get back to this again and try one more time.
>
> I know this is not easy, but that's the pattern we used so far to fight
> with "technical dept" - CI must be green for a merge.
>
> It also worth asking @Leemoonsoo @jongyoul for help in tricky cases. I will
> take a look too.
>
> On Tue, Dec 1, 2015, 18:53 astroshim <[email protected]> wrote:
>
> > Github user astroshim commented on the pull request:
> >
> >
> >
> https://github.com/apache/incubator-zeppelin/pull/489#issuecomment-160916885
> >
> >     @bzz Thank you very much!!
> >     It seems like CI fails is not related to testcase. What do you think?
> >
> >
> > ---
> > If your project is set up for it, you can reply to this email and have
> your
> > reply appear on GitHub as well. If your project does not have this
> feature
> > enabled and wishes so, or if the feature is enabled but not working,
> please
> > contact infrastructure at [email protected] or file a JIRA
> ticket
> > with INFRA.
> > ---
> >
>

Reply via email to