Thanks a lot for your help with that Pavel :-) On Fri, Aug 19, 2016 at 9:48 PM, Pavel Fadeev <pavel.fad...@gmail.com> wrote:
> 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 <trohrm...@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 <fhue...@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.fad...@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? > > > > > > > > > >
