flink-dev mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From Pavel Fadeev <pavel.fad...@gmail.com>
Subject Re: [FLINK-305] Code test coverage - how FLINK using it?
Date Fri, 19 Aug 2016 19:48:56 GMT
Fabian, Till,
thanks for your comments!

I`ve raised https://issues.apache.org/jira/browse/INFRA-12458 for the same.
Will play around to see if this solution acceptable and does not affect
build duration too much.

2016-08-19 12:51 GMT+03:00 Till Rohrmann <trohrmann@apache.org>:

> Hi Pavel,
>
> I think it's a good point you're raising here. The Flink community isn't
> using metrics like test coverage to ensure high quality code yet. I think
> that is one thing which we can/should improve. Unfortunately, the ASF does
> not allow to use codecov.io (or at least the Apache Infra team) [1].
> However, they encourage to use coveralls.io.
>
> Do you know this tool and want to take a look how it could be integrated
> with the Flink repository? It seems as if it is free for open source
> projects. Maybe you can create a JIRA issue for this integration and then
> take the lead there.
>
> [1] https://issues.apache.org/jira/browse/INFRA-11273
>
> Cheers,
> Till
>
> On Fri, Aug 19, 2016 at 11:33 AM, Fabian Hueske <fhueske@gmail.com> wrote:
>
> > Hi Pavel,
> >
> > the Cobertura plugin was removed in this PR:
> > https://github.com/apache/flink/pull/454
> > Not sure if it was accidentally removed or on purpose.
> > It was not included in the regular builds to reduce build time and AFAIK,
> > it wasn't manually used either (otherwise somebody would have noticed
> that
> > it is gone).
> >
> > I am not aware of any code coverage stats for the Flink code base but it
> > would be nice to some, IMO.
> >
> > Best, Fabian
> >
> > 2016-08-19 0:36 GMT+02:00 Pavel Fadeev <pavel.fadeev@gmail.com>:
> >
> > > Dear team,
> > >
> > > I`m just looking around into the project - complete novice at Flink :)
> > > Sorry if Qs below already have answers!
> > >
> > > At first glance I`ve discovered that code coverage feature has been
> > > introduced with FLINK-305 <https://issues.apache.org/
> > jira/browse/FLINK-305
> > > >
> > > and then removed for some reason on March 2015.
> > >
> > > Are you aware if it is not required anymore? Worried a bit about this
> > after
> > > local coverage run for flink-code as well. Do we have some integration
> > like
> > > codecov <https://codecov.io> here and do you feel if it is required?
> > >
> > > Also, do you know if there are some statistics (or team knowledge) for
> > > regression bugs from uncovered code?
> > >
> >
>

Mime
  • Unnamed multipart/alternative (inline, None, 0 bytes)
View raw message