flink-dev mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From Till Rohrmann <trohrm...@apache.org>
Subject Re: [FLINK-305] Code test coverage - how FLINK using it?
Date Fri, 19 Aug 2016 09:51:32 GMT
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


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?
> >

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