Il lun 5 feb 2018, 18:11 David Rusek <d...@streaml.io> ha scritto: > It sounds like we didn't do anything with the info for a long time. Enrico, > I'm glad you're looking at it! Are you planning on filing some issues > related to interpreting the coverage data and improving it? >
It was long time ago when I started to experiment with bookkeeper codebase. We had some problems and I had other priorities. I will try to resume this thread on next weeks I think that the culprit of our problems was the way we were performing BC tests. I have not much time so I will go on one step at a time, if you have time any help is appreciated. First step will be to test locally jacoco and then to restore the CI jobs Ideally I would like to have some automated way to keep an eye on BK and maybe (not sure it is a big deal) to perform code coverage analysis even on PRs. One big problem is that our corpus of tests is very heavy as most of the tests start a new cluster. Recently we started to use mockito in order to perform narrower unit testing. Stay tuned Enrico Enrico > > -Dave > > On Mon, Feb 5, 2018 at 10:08 AM, Enrico Olivelli <eolive...@gmail.com> > wrote: > > > Will do. > > Enrico > > > > Il lun 5 feb 2018, 17:45 Sijie Guo <guosi...@gmail.com> ha scritto: > > > > > Can’t remember exactly whether it is only on BC stuffs. But feel free > to > > > try it. > > > > > > On Mon, Feb 5, 2018 at 8:43 AM Enrico Olivelli <eolive...@gmail.com> > > > wrote: > > > > > >> I think was due to BC tests, we can restore now that we do not have > that > > >> shade stuff anymore > > >> > > >> Enrico > > >> > > >> Il lun 5 feb 2018, 17:40 Sijie Guo <guosi...@gmail.com> ha scritto: > > >> > > >>> Yes we disabled it long time ago. Because it failed CI with > > >>> ClassNotFoundException. > > >>> > > >>> Sijie > > >>> > > >>> On Mon, Feb 5, 2018 at 12:27 AM Enrico Olivelli <eolive...@gmail.com > > > > >>> wrote: > > >>> > > >>>> Sijie, > > >>>> it seems that Coveralls report is a bit outdated. > > >>>> Have we have disabled it on CI ? > > >>>> > > >>>> see > > >>>> https://coveralls.io/github/apache/bookkeeper?branch=master > > >>>> > > >>>> Enrico > > >>>> > > >>>> 2017-10-03 19:55 GMT+02:00 Enrico Olivelli <eolive...@gmail.com>: > > >>>> > > >>>>> Ok, I'll check it out. > > >>>>> > > >>>>> Il mar 3 ott 2017, 19:52 Sijie Guo <guosi...@gmail.com> ha > scritto: > > >>>>> > > >>>>>> It seems that the jacoco was not disabled at master build. so all > > the > > >>>>>> master builds are still failing. I just disabled it. > > >>>>>> > > >>>>>> - Sijie > > >>>>>> > > >>>>>> On Wed, Sep 20, 2017 at 11:51 PM, Enrico Olivelli < > > >>>>>> eolive...@gmail.com> wrote: > > >>>>>> > > >>>>>>> The problem is only on backward compatibiliy tests > > >>>>>>> I am going to disable Jacoco in master build > > >>>>>>> I wlil be back when I have news > > >>>>>>> > > >>>>>>> -- Enrico > > >>>>>>> > > >>>>>>> ------------------------------------------------------- > > >>>>>>> T E S T S > > >>>>>>> ------------------------------------------------------- > > >>>>>>> Running org.apache.bookkeeper.tests.backward.TestBookieRecovery > > >>>>>>> Tests run: 1, Failures: 0, Errors: 1, Skipped: 0, Time elapsed: > > 2.621 sec <<< FAILURE! - in org.apache.bookkeeper.tests. > > backward.TestBookieRecovery > > >>>>>>> > ensurePasswordUsedForOldLedgers(org.apache.bookkeeper.tests.backward.TestBookieRecovery) > > Time elapsed: 2.284 sec <<< ERROR! > > >>>>>>> java.lang.NoClassDefFoundError: org/apache/bk_v4_1_0/ > > bookkeeper/client/BookKeeper$DigestType > > >>>>>>> at org.apache.bookkeeper.tests. > > backward.TestBookieRecovery.ensurePasswordUsedForOldLedger > > s(TestBookieRecovery.java:266) > > >>>>>>> Caused by: java.lang.ClassNotFoundException: > org.apache.bk_v4_1_0. > > bookkeeper.client.BookKeeper$DigestType > > >>>>>>> at org.apache.bookkeeper.tests. > > backward.TestBookieRecovery.ensurePasswordUsedForOldLedger > > s(TestBookieRecovery.java:266) > > >>>>>>> > > >>>>>>> Running org.apache.bookkeeper.tests.backward.TestBackwardCompat > > >>>>>>> Tests run: 5, Failures: 0, Errors: 5, Skipped: 0, Time elapsed: > > 1.087 sec <<< FAILURE! - in org.apache.bookkeeper.tests. > > backward.TestBackwardCompat > > >>>>>>> testCompatHierarchicalLedgerManager(org.apache.bookkeeper. > > tests.backward.TestBackwardCompat) Time elapsed: 0.646 sec <<< ERROR! > > >>>>>>> java.lang.NoClassDefFoundError: org/apache/bk_v4_2_0/ > > bookkeeper/conf/ServerConfiguration > > >>>>>>> at org.apache.bookkeeper.tests. > > backward.TestBackwardCompat.testCompatHierarchicalLedgerMa > > nager(TestBackwardCompat.java:677) > > >>>>>>> Caused by: java.lang.ClassNotFoundException: > org.apache.bk_v4_2_0. > > bookkeeper.conf.ServerConfiguration > > >>>>>>> at org.apache.bookkeeper.tests. > > backward.TestBackwardCompat.testCompatHierarchicalLedgerMa > > nager(TestBackwardCompat.java:677) > > >>>>>>> > > >>>>>>> > testCompatWrites(org.apache.bookkeeper.tests.backward.TestBackwardCompat) > > Time elapsed: 0.033 sec <<< ERROR! > > >>>>>>> java.lang.NoClassDefFoundError: org/apache/bk_v4_1_0/ > > bookkeeper/conf/ServerConfiguration > > >>>>>>> at org.apache.bookkeeper.tests. > > backward.TestBackwardCompat.testCompatWrites(TestBackwardCompat.java:623) > > >>>>>>> Caused by: java.lang.ClassNotFoundException: > org.apache.bk_v4_1_0. > > bookkeeper.conf.ServerConfiguration > > >>>>>>> at org.apache.bookkeeper.tests. > > backward.TestBackwardCompat.testCompatWrites(TestBackwardCompat.java:623) > > >>>>>>> > > >>>>>>> > testCompatReads(org.apache.bookkeeper.tests.backward.TestBackwardCompat) > > Time elapsed: 0.041 sec <<< ERROR! > > >>>>>>> java.lang.NoClassDefFoundError: org/apache/bk_v4_1_0/ > > bookkeeper/conf/ServerConfiguration > > >>>>>>> at org.apache.bookkeeper.tests. > > backward.TestBackwardCompat.testCompatReads(TestBackwardCompat.java:573) > > >>>>>>> > > >>>>>>> > testCompat410(org.apache.bookkeeper.tests.backward.TestBackwardCompat) > > Time elapsed: 0.017 sec <<< ERROR! > > >>>>>>> java.lang.NoClassDefFoundError: org/apache/bk_v4_1_0/ > > bookkeeper/conf/ServerConfiguration > > >>>>>>> at org.apache.bookkeeper.tests. > > backward.TestBackwardCompat.testCompat410(TestBackwardCompat.java:455) > > >>>>>>> > > >>>>>>> testOldCookieAccessingNewCluster(org.apache.bookkeeper. > > tests.backward.TestBackwardCompat) Time elapsed: 0.018 sec <<< ERROR! > > >>>>>>> java.lang.NoClassDefFoundError: org/apache/bk_v4_1_0/ > > bookkeeper/conf/ServerConfiguration > > >>>>>>> at org.apache.bookkeeper.tests. > > backward.TestBackwardCompat.testOldCookieAccessingNewClust > > er(TestBackwardCompat.java:403) > > >>>>>>> > > >>>>>>> > > >>>>>>> 2017-09-20 20:37 GMT+02:00 Enrico Olivelli <eolive...@gmail.com > >: > > >>>>>>> > > >>>>>>>> Sorry, > > >>>>>>>> I did not try with the latest master but it should not be a > > problem. > > >>>>>>>> I will investigate and maybe revert > > >>>>>>>> It could be a problem related to the env, > > >>>>>>>> Eventually using docker will be nicer and more reproducible > > >>>>>>>> I will work on it asap tomorrow > > >>>>>>>> Enrico > > >>>>>>>> > > >>>>>>>> On mer 20 set 2017, 20:28 Sijie Guo <guosi...@gmail.com> wrote: > > >>>>>>>> > > >>>>>>>>> did you test this with current master before enabling in > > >>>>>>>>> bookkeeper-master job? It somehow broke the jenkins. > > >>>>>>>>> > > >>>>>>>>> because I saw ClassNotFoundException in latest jekins build. > > >>>>>>>>> > > >>>>>>>>> > > >>>>>>>>> https://builds.apache.org/blue/organizations/jenkins/ > > bookkeeper-master/detail/bookkeeper-master/1889/pipeline > > >>>>>>>>> > > >>>>>>>>> > > >>>>>>>>> > > >>>>>>>>> On Wed, Sep 20, 2017 at 2:34 AM, Enrico Olivelli < > > >>>>>>>>> eolive...@gmail.com> wrote: > > >>>>>>>>> > > >>>>>>>>>> I have updated master job > > >>>>>>>>>> https://builds.apache.org/job/bookkeeper-master/ > > >>>>>>>>>> > > >>>>>>>>>> let's see the results > > >>>>>>>>>> -- Enrico > > >>>>>>>>>> > > >>>>>>>>>> 2017-08-24 10:27 GMT+02:00 Enrico Olivelli < > eolive...@gmail.com > > >: > > >>>>>>>>>> > > >>>>>>>>>>> It seems that JaCoCo works like a charm > > >>>>>>>>>>> https://builds.apache.org/job/bookkeeper-codecoverage/3/ > > console > > >>>>>>>>>>> https://coveralls.io/github/apache/bookkeeper > > >>>>>>>>>>> > > >>>>>>>>>>> I will integrate it into nightly builds, the only concern is > > >>>>>>>>>>> that CodeCoverage instrumentation may affect tests > > >>>>>>>>>>> thoughts ? > > >>>>>>>>>>> > > >>>>>>>>>>> -- Enrico > > >>>>>>>>>>> > > >>>>>>>>>>> > > >>>>>>>>>>> > > >>>>>>>>>>> 2017-08-24 2:09 GMT+02:00 Jia Zhai <zhaiji...@gmail.com>: > > >>>>>>>>>>> > > >>>>>>>>>>>> 👍 > > >>>>>>>>>>>> > > >>>>>>>>>>>> On Wed, Aug 23, 2017 at 11:38 PM, Enrico Olivelli < > > >>>>>>>>>>>> eolive...@gmail.com> > > >>>>>>>>>>>> wrote: > > >>>>>>>>>>>> > > >>>>>>>>>>>> > 2017-08-23 17:34 GMT+02:00 Sijie Guo <guosi...@gmail.com > >: > > >>>>>>>>>>>> > > > >>>>>>>>>>>> > > On Aug 23, 2017 7:31 AM, "Enrico Olivelli" < > > >>>>>>>>>>>> eolive...@gmail.com> wrote: > > >>>>>>>>>>>> > > > > >>>>>>>>>>>> > > 2017-08-23 8:29 GMT+02:00 Enrico Olivelli < > > >>>>>>>>>>>> eolive...@gmail.com>: > > >>>>>>>>>>>> > > > > >>>>>>>>>>>> > > > Latest news > > >>>>>>>>>>>> > > > > > >>>>>>>>>>>> > > > The build works and coveralls reports a 63% code > > coverage. > > >>>>>>>>>>>> > > > Maybe the link to the badge on readme.md is wrong, I > > >>>>>>>>>>>> will check > > >>>>>>>>>>>> > > > > > >>>>>>>>>>>> > > > > >>>>>>>>>>>> > > Our core in README.md is good, but github caches images, > > >>>>>>>>>>>> this is the good > > >>>>>>>>>>>> > > link but GitHub cached an old version of the image > > >>>>>>>>>>>> > > https://coveralls.io/repos/github/apache/bookkeeper/ > > >>>>>>>>>>>> > > badge.svg?branch=master > > >>>>>>>>>>>> > > > > >>>>>>>>>>>> > > > > >>>>>>>>>>>> > > > > >>>>>>>>>>>> > > > Cobertura reports a lot of parsing errors, I will > > >>>>>>>>>>>> continue to try > > >>>>>>>>>>>> > JaCoCo > > >>>>>>>>>>>> > > > > > >>>>>>>>>>>> > > > > >>>>>>>>>>>> > > I have tried with JaCoCo the result is similar, but > there > > >>>>>>>>>>>> are no "scary" > > >>>>>>>>>>>> > > message on console output like this: > > >>>>>>>>>>>> > > > > >>>>>>>>>>>> > > > net.sourceforge.cobertura.javancss.parser.ParseException: > > >>>>>>>>>>>> Encountered > > >>>>>>>>>>>> > > " "default" "default "" at line 332, column 5. > > >>>>>>>>>>>> > > Was expecting one of: > > >>>>>>>>>>>> > > "abstract" ... > > >>>>>>>>>>>> > > "boolean" ... > > >>>>>>>>>>>> > > "byte" ... > > >>>>>>>>>>>> > > "char" ... > > >>>>>>>>>>>> > > "class" ... > > >>>>>>>>>>>> > > "double" ... > > >>>>>>>>>>>> > > "enum" ... > > >>>>>>>>>>>> > > "final" ... > > >>>>>>>>>>>> > > "float" ... > > >>>>>>>>>>>> > > "int" ... > > >>>>>>>>>>>> > > "interface" ... > > >>>>>>>>>>>> > > "long" ... > > >>>>>>>>>>>> > > "native" ... > > >>>>>>>>>>>> > > "private" ... > > >>>>>>>>>>>> > > "protected" ... > > >>>>>>>>>>>> > > "public" ... > > >>>>>>>>>>>> > > "short" ... > > >>>>>>>>>>>> > > "static" ... > > >>>>>>>>>>>> > > "strictfp" ... > > >>>>>>>>>>>> > > "synchronized" ... > > >>>>>>>>>>>> > > "transient" ... > > >>>>>>>>>>>> > > "void" ... > > >>>>>>>>>>>> > > "volatile" ... > > >>>>>>>>>>>> > > <IDENTIFIER> ... > > >>>>>>>>>>>> > > "}" ... > > >>>>>>>>>>>> > > ";" ... > > >>>>>>>>>>>> > > "@" ... > > >>>>>>>>>>>> > > "<" ... > > >>>>>>>>>>>> > > > > >>>>>>>>>>>> > > at > > >>>>>>>>>>>> net.sourceforge.cobertura.javancss.parser.JavaParser. > > >>>>>>>>>>>> > > generateParseException(JavaParser.java:10490) > > >>>>>>>>>>>> > > at > > >>>>>>>>>>>> net.sourceforge.cobertura.javancss.parser.JavaParser.jj_ > > >>>>>>>>>>>> > > consume_token(JavaParser.java:10366) > > >>>>>>>>>>>> > > at > > >>>>>>>>>>>> net.sourceforge.cobertura.javancss.parser.JavaParser. > > >>>>>>>>>>>> > > UnmodifiedInterfaceDeclaration(JavaParser.java:1475) > > >>>>>>>>>>>> > > at > > >>>>>>>>>>>> net.sourceforge.cobertura.javancss.parser.JavaParser. > > >>>>>>>>>>>> > > InterfaceDeclaration(JavaParser.java:1273) > > >>>>>>>>>>>> > > at > > >>>>>>>>>>>> net.sourceforge.cobertura.javancss.parser.JavaParser. > > >>>>>>>>>>>> > > TypeDeclaration(JavaParser.java:580) > > >>>>>>>>>>>> > > at > > >>>>>>>>>>>> net.sourceforge.cobertura.javancss.parser.JavaParser. > > >>>>>>>>>>>> > > CompilationUnit(JavaParser.java:318) > > >>>>>>>>>>>> > > at > > >>>>>>>>>>>> net.sourceforge.cobertura.javancss.parser.JavaParser. > > >>>>>>>>>>>> > > parse(JavaParser.java:100) > > >>>>>>>>>>>> > > at net.sourceforge.cobertura.javancss.Javancss._ > > >>>>>>>>>>>> > > measureSource(Javancss.java:192) > > >>>>>>>>>>>> > > at net.sourceforge.cobertura.javancss.Javancss._ > > >>>>>>>>>>>> > > measureRoot(Javancss.java:268) > > >>>>>>>>>>>> > > at > > >>>>>>>>>>>> net.sourceforge.cobertura.javancss.Javancss.<init>( > > >>>>>>>>>>>> > > Javancss.java:347) > > >>>>>>>>>>>> > > at > > >>>>>>>>>>>> net.sourceforge.cobertura.javancss.JavancssFactory. > > >>>>>>>>>>>> > > newInstance(JavancssFactory.java:40) > > >>>>>>>>>>>> > > at > > >>>>>>>>>>>> net.sourceforge.cobertura.reporting.ComplexityCalculator. > > >>>>>>>>>>>> > > > > >>>>>>>>>>>> getFunctionMetricsForSingleFile(ComplexityCalculator.java: > > 243) > > >>>>>>>>>>>> > > > > >>>>>>>>>>>> > > > > >>>>>>>>>>>> > > > > >>>>>>>>>>>> > > Maybe it is better to switch to JaCoCo, Cobertura wants > to > > >>>>>>>>>>>> read java code > > >>>>>>>>>>>> > > but cannot handle Java8 default methods interface syntax > > >>>>>>>>>>>> > > > > >>>>>>>>>>>> > > Issue: > > >>>>>>>>>>>> > > https://github.com/apache/bookkeeper/issues/464 > > >>>>>>>>>>>> > > > > >>>>>>>>>>>> > > Patch to introduce the configuration: > > >>>>>>>>>>>> > > https://github.com/apache/bookkeeper/pull/465 > > >>>>>>>>>>>> > > > > >>>>>>>>>>>> > > > > >>>>>>>>>>>> > > Good job! I made one comment in the pull request. > > >>>>>>>>>>>> Otherwise, overall > > >>>>>>>>>>>> > looks > > >>>>>>>>>>>> > > good. > > >>>>>>>>>>>> > > > > >>>>>>>>>>>> > > > > >>>>>>>>>>>> > > > > >>>>>>>>>>>> > > > > >>>>>>>>>>>> > > Once we have a stable code coverage tool I would like to > > >>>>>>>>>>>> setup an > > >>>>>>>>>>>> > automatic > > >>>>>>>>>>>> > > check upon Pull Requests > > >>>>>>>>>>>> > > > > >>>>>>>>>>>> > > > > >>>>>>>>>>>> > > Sure, are you planning to enforce pull request should > > >>>>>>>>>>>> increase coverage? > > >>>>>>>>>>>> > > > > >>>>>>>>>>>> > > > >>>>>>>>>>>> > > > >>>>>>>>>>>> > It would be great > > >>>>>>>>>>>> > but I think it is not really possible, sometimes we have > > >>>>>>>>>>>> patches on the > > >>>>>>>>>>>> > website or on poms > > >>>>>>>>>>>> > I have to understand better how code coverage tools work > > >>>>>>>>>>>> > for instance I see that interfaces code seems to be > > >>>>>>>>>>>> "uncovered" so > > >>>>>>>>>>>> > introducing a new interface will decrease the coverage > > >>>>>>>>>>>> > I have to play with it > > >>>>>>>>>>>> > > > >>>>>>>>>>>> > > > >>>>>>>>>>>> > > > > >>>>>>>>>>>> > > > > >>>>>>>>>>>> > > > > >>>>>>>>>>>> > > Thoughts ? > > >>>>>>>>>>>> > > > > >>>>>>>>>>>> > > Enrico > > >>>>>>>>>>>> > > > > >>>>>>>>>>>> > > > > >>>>>>>>>>>> > > > > >>>>>>>>>>>> > > I will try on DL to in case we have to switch > > >>>>>>>>>>>> > > > > > >>>>>>>>>>>> > > > Enrico > > >>>>>>>>>>>> > > > > > >>>>>>>>>>>> > > > On mar 22 ago 2017, 22:00 Enrico Olivelli < > > >>>>>>>>>>>> eolive...@gmail.com> wrote: > > >>>>>>>>>>>> > > > > > >>>>>>>>>>>> > > >> 2017-08-22 21:52 GMT+02:00 Sijie Guo < > > guosi...@gmail.com > > >>>>>>>>>>>> >: > > >>>>>>>>>>>> > > >> > > >>>>>>>>>>>> > > >>> Hi Enrico, > > >>>>>>>>>>>> > > >>> > > >>>>>>>>>>>> > > >>> You should use 'ubuntu' rather than 'Ubuntu' (it is > > >>>>>>>>>>>> case sensitive). > > >>>>>>>>>>>> > > It > > >>>>>>>>>>>> > > >>> is > > >>>>>>>>>>>> > > >>> running now - > > >>>>>>>>>>>> > > >>> > > >>>>>>>>>>>> https://builds.apache.org/job/bookkeeper-codecoverage/1/ > > console > > >>>>>>>>>>>> > > >>> > > >>>>>>>>>>>> > > >>> > > >>>>>>>>>>>> > > >> Thank you. I thought I had tried but without success. > > >>>>>>>>>>>> > > >> > > >>>>>>>>>>>> > > >> > > >>>>>>>>>>>> > > >>> Also It would be good to put code-coverage into > > >>>>>>>>>>>> > > >>> > > >>>>>>>>>>>> https://builds.apache.org/job/bookkeeper-release-nightly- > > snapshot/ > > >>>>>>>>>>>> > > >>> eventually. > > >>>>>>>>>>>> > > >>> > > >>>>>>>>>>>> > > >> > > >>>>>>>>>>>> > > >> I will play and the integrate the two jobs > > >>>>>>>>>>>> > > >> > > >>>>>>>>>>>> > > >> Thank you very much > > >>>>>>>>>>>> > > >> Enrico > > >>>>>>>>>>>> > > >> > > >>>>>>>>>>>> > > >> > > >>>>>>>>>>>> > > >>> > > >>>>>>>>>>>> > > >>> - Sijie > > >>>>>>>>>>>> > > >>> > > >>>>>>>>>>>> > > >>> > > >>>>>>>>>>>> > > >>> > > >>>>>>>>>>>> > > >>> On Mon, Aug 21, 2017 at 9:48 PM, Sijie Guo < > > >>>>>>>>>>>> guosi...@gmail.com> > > >>>>>>>>>>>> > wrote: > > >>>>>>>>>>>> > > >>> > > >>>>>>>>>>>> > > >>> > I will check it tomorrow. > > >>>>>>>>>>>> > > >>> > > > >>>>>>>>>>>> > > >>> > On Aug 20, 2017 1:23 AM, "Enrico Olivelli" < > > >>>>>>>>>>>> eolive...@gmail.com> > > >>>>>>>>>>>> > > >>> wrote: > > >>>>>>>>>>>> > > >>> > > > >>>>>>>>>>>> > > >>> >> Sijie, > > >>>>>>>>>>>> > > >>> >> I am not able to make the new job run > > >>>>>>>>>>>> > > >>> >> > > >>>>>>>>>>>> https://builds.apache.org/job/bookkeeper-codecoverage/ > > >>>>>>>>>>>> > > >>> >> > > >>>>>>>>>>>> > > >>> >> I cannot find any available node, I tried with > > >>>>>>>>>>>> ubumtu and haddop, > > >>>>>>>>>>>> > > with > > >>>>>>>>>>>> > > >>> >> different cases for the first char > > >>>>>>>>>>>> > > >>> >> > > >>>>>>>>>>>> > > >>> >> Ideas? > > >>>>>>>>>>>> > > >>> >> Enrico > > >>>>>>>>>>>> > > >>> >> > > >>>>>>>>>>>> > > >>> >> On sab 19 ago 2017, 15:35 Enrico Olivelli < > > >>>>>>>>>>>> eolive...@gmail.com> > > >>>>>>>>>>>> > > >>> wrote: > > >>>>>>>>>>>> > > >>> >> > > >>>>>>>>>>>> > > >>> >> > I have created a new jenkins job > > >>>>>>>>>>>> > > >>> >> > > > >>>>>>>>>>>> https://builds.apache.org/job/bookkeeper-codecoverage/ > > >>>>>>>>>>>> > > >>> >> > > > >>>>>>>>>>>> > > >>> >> > emails will be sent to my address > > >>>>>>>>>>>> eolive...@apache.org > > >>>>>>>>>>>> > > >>> >> > > > >>>>>>>>>>>> > > >>> >> > I am playing with cobertura, if I won't > succeed I > > >>>>>>>>>>>> will create a > > >>>>>>>>>>>> > PR > > >>>>>>>>>>>> > > >>> in > > >>>>>>>>>>>> > > >>> >> > order to introduce the configuration of JaCoCo > on > > >>>>>>>>>>>> master branch > > >>>>>>>>>>>> > > >>> >> > > > >>>>>>>>>>>> > > >>> >> > Enrico > > >>>>>>>>>>>> > > >>> >> > > > >>>>>>>>>>>> > > >>> >> > > > >>>>>>>>>>>> > > >>> >> > 2017-08-04 21:55 GMT+02:00 Sijie Guo < > > >>>>>>>>>>>> guosi...@gmail.com>: > > >>>>>>>>>>>> > > >>> >> > > > >>>>>>>>>>>> > > >>> >> >> Sure go for it. > > >>>>>>>>>>>> > > >>> >> >> > > >>>>>>>>>>>> > > >>> >> >> Sijie > > >>>>>>>>>>>> > > >>> >> >> > > >>>>>>>>>>>> > > >>> >> >> On Aug 4, 2017 12:46 PM, "Enrico Olivelli" < > > >>>>>>>>>>>> > eolive...@gmail.com> > > >>>>>>>>>>>> > > >>> >> wrote: > > >>>>>>>>>>>> > > >>> >> >> > > >>>>>>>>>>>> > > >>> >> >> > Il gio 20 lug 2017, 17:06 Enrico Olivelli < > > >>>>>>>>>>>> > eolive...@gmail.com > > >>>>>>>>>>>> > > > > > >>>>>>>>>>>> > > >>> ha > > >>>>>>>>>>>> > > >>> >> >> > scritto: > > >>>>>>>>>>>> > > >>> >> >> > > > >>>>>>>>>>>> > > >>> >> >> > > Jacoco is running as expected > > >>>>>>>>>>>> > > >>> >> >> > > see: > > >>>>>>>>>>>> > > >>> >> >> > > https://coveralls.io/jobs/27785961 > > >>>>>>>>>>>> > > >>> >> >> > > > > >>>>>>>>>>>> > > >>> >> >> > > I think it is better to switch to Jacoco > > >>>>>>>>>>>> > > >>> >> >> > > > > >>>>>>>>>>>> > > >>> >> >> > > I wonder why on DL there is no such > problem > > >>>>>>>>>>>> with Cobertura > > >>>>>>>>>>>> > > >>> >> >> > > @Sijie do you have any idea ? > > >>>>>>>>>>>> > > >>> >> >> > > > > >>>>>>>>>>>> > > >>> >> >> > > > >>>>>>>>>>>> > > >>> >> >> > Other opinions? I am going to test > codecov.io > > >>>>>>>>>>>> too in other > > >>>>>>>>>>>> > > >>> projects > > >>>>>>>>>>>> > > >>> >> and > > >>>>>>>>>>>> > > >>> >> >> > compare the results > > >>>>>>>>>>>> > > >>> >> >> > > > >>>>>>>>>>>> > > >>> >> >> > Enrico > > >>>>>>>>>>>> > > >>> >> >> > > > >>>>>>>>>>>> > > >>> >> >> > > > >>>>>>>>>>>> > > >>> >> >> > > > > >>>>>>>>>>>> > > >>> >> >> > > -- Enrico > > >>>>>>>>>>>> > > >>> >> >> > > > > >>>>>>>>>>>> > > >>> >> >> > > > > >>>>>>>>>>>> > > >>> >> >> > > > > >>>>>>>>>>>> > > >>> >> >> > > 2017-07-20 16:06 GMT+02:00 Enrico > Olivelli < > > >>>>>>>>>>>> > > >>> eolive...@gmail.com>: > > >>>>>>>>>>>> > > >>> >> >> > > > > >>>>>>>>>>>> > > >>> >> >> > >> I am sorry but it seems that cobertura is > > >>>>>>>>>>>> not working with > > >>>>>>>>>>>> > > >>> java8, > > >>>>>>>>>>>> > > >>> >> as > > >>>>>>>>>>>> > > >>> >> >> > >> reported in Coveralls.io homepage > > >>>>>>>>>>>> > > >>> >> >> > >> > > >>>>>>>>>>>> https://github.com/trautonen/coveralls-maven-plugin > > >>>>>>>>>>>> > > >>> >> >> > >> > > >>>>>>>>>>>> > > >>> >> >> > >> > > >>>>>>>>>>>> > > >>> >> >> > >> see our error: > > >>>>>>>>>>>> > > >>> >> >> > >> > > >>>>>>>>>>>> https://builds.apache.org/job/bookkeeper-tests/6/console > > >>>>>>>>>>>> > > >>> >> >> > >> > > >>>>>>>>>>>> > > >>> >> >> > >> I will be back to Jacoco which seems to > be > > >>>>>>>>>>>> the only one > > >>>>>>>>>>>> > > which > > >>>>>>>>>>>> > > >>> >> >> supports > > >>>>>>>>>>>> > > >>> >> >> > >> java8 for Coveralls > > >>>>>>>>>>>> > > >>> >> >> > >> > > >>>>>>>>>>>> > > >>> >> >> > >> -- Enrico > > >>>>>>>>>>>> > > >>> >> >> > >> > > >>>>>>>>>>>> > > >>> >> >> > >> > > >>>>>>>>>>>> > > >>> >> >> > >> 2017-07-19 18:02 GMT+02:00 Sijie Guo < > > >>>>>>>>>>>> guosi...@gmail.com > > >>>>>>>>>>>> > >: > > >>>>>>>>>>>> > > >>> >> >> > >> > > >>>>>>>>>>>> > > >>> >> >> > >>> Hmm he should mail you the token rather > > >>>>>>>>>>>> then posting on > > >>>>>>>>>>>> > the > > >>>>>>>>>>>> > > >>> jira > > >>>>>>>>>>>> > > >>> >> :) > > >>>>>>>>>>>> > > >>> >> >> > >>> > > >>>>>>>>>>>> > > >>> >> >> > >>> On Jul 19, 2017 8:56 AM, "Enrico > > Olivelli" < > > >>>>>>>>>>>> > > >>> eolive...@gmail.com> > > >>>>>>>>>>>> > > >>> >> >> > wrote: > > >>>>>>>>>>>> > > >>> >> >> > >>> > > >>>>>>>>>>>> > > >>> >> >> > >>> > We have the token, it is in the > ticket. > > >>>>>>>>>>>> > > >>> >> >> > >>> > I will do some tests soon > > >>>>>>>>>>>> > > >>> >> >> > >>> > Enrico > > >>>>>>>>>>>> > > >>> >> >> > >>> > > > >>>>>>>>>>>> > > >>> >> >> > >>> > Il mer 19 lug 2017, 17:52 Enrico > > Olivelli > > >>>>>>>>>>>> < > > >>>>>>>>>>>> > > >>> eolive...@gmail.com > > >>>>>>>>>>>> > > >>> >> > > > >>>>>>>>>>>> > > >>> >> >> ha > > >>>>>>>>>>>> > > >>> >> >> > >>> > scritto: > > >>>>>>>>>>>> > > >>> >> >> > >>> > > > >>>>>>>>>>>> > > >>> >> >> > >>> > > this is the INFRA ticket for having > a > > >>>>>>>>>>>> coveralls.io > > >>>>>>>>>>>> > > >>> token in > > >>>>>>>>>>>> > > >>> >> our > > >>>>>>>>>>>> > > >>> >> >> > >>> Jenkins > > >>>>>>>>>>>> > > >>> >> >> > >>> > > builds > > >>>>>>>>>>>> > > >>> >> >> > >>> > > > > >>>>>>>>>>>> https://issues.apache.org/jira/browse/INFRA-14649 > > >>>>>>>>>>>> > > >>> >> >> > >>> > > > > >>>>>>>>>>>> > > >>> >> >> > >>> > > -- Enrico > > >>>>>>>>>>>> > > >>> >> >> > >>> > > > > >>>>>>>>>>>> > > >>> >> >> > >>> > > 2017-07-17 23:49 GMT+02:00 Enrico > > >>>>>>>>>>>> Olivelli < > > >>>>>>>>>>>> > > >>> >> eolive...@gmail.com > > >>>>>>>>>>>> > > >>> >> >> >: > > >>>>>>>>>>>> > > >>> >> >> > >>> > > > > >>>>>>>>>>>> > > >>> >> >> > >>> > >> > > >>>>>>>>>>>> > > >>> >> >> > >>> > >> > > >>>>>>>>>>>> > > >>> >> >> > >>> > >> Il lun 17 lug 2017, 22:55 Sijie > Guo < > > >>>>>>>>>>>> > > guosi...@gmail.com > > >>>>>>>>>>>> > > > > > >>>>>>>>>>>> > > >>> ha > > >>>>>>>>>>>> > > >>> >> >> > >>> scritto: > > >>>>>>>>>>>> > > >>> >> >> > >>> > >> > > >>>>>>>>>>>> > > >>> >> >> > >>> > >>> On Tue, Jul 18, 2017 at 4:46 AM, > > >>>>>>>>>>>> Enrico Olivelli < > > >>>>>>>>>>>> > > >>> >> >> > >>> eolive...@gmail.com> > > >>>>>>>>>>>> > > >>> >> >> > >>> > >>> wrote: > > >>>>>>>>>>>> > > >>> >> >> > >>> > >>> > > >>>>>>>>>>>> > > >>> >> >> > >>> > >>> > Il lun 17 lug 2017, 21:34 Sijie > > Guo > > >>>>>>>>>>>> < > > >>>>>>>>>>>> > > >>> guosi...@gmail.com> > > >>>>>>>>>>>> > > >>> >> ha > > >>>>>>>>>>>> > > >>> >> >> > >>> scritto: > > >>>>>>>>>>>> > > >>> >> >> > >>> > >>> > > > >>>>>>>>>>>> > > >>> >> >> > >>> > >>> > > On Sun, Jul 16, 2017 at 8:06 > PM, > > >>>>>>>>>>>> Enrico > > >>>>>>>>>>>> > Olivelli > > >>>>>>>>>>>> > > < > > >>>>>>>>>>>> > > >>> >> >> > >>> > >>> eolive...@gmail.com> > > >>>>>>>>>>>> > > >>> >> >> > >>> > >>> > > wrote: > > >>>>>>>>>>>> > > >>> >> >> > >>> > >>> > > > > >>>>>>>>>>>> > > >>> >> >> > >>> > >>> > > > I have just committed > > >>>>>>>>>>>> coveralls.io and > > >>>>>>>>>>>> > > cobertura > > >>>>>>>>>>>> > > >>> >> >> plugins > > >>>>>>>>>>>> > > >>> >> >> > >>> into > > >>>>>>>>>>>> > > >>> >> >> > >>> > main > > >>>>>>>>>>>> > > >>> >> >> > >>> > >>> > > pom.xml > > >>>>>>>>>>>> > > >>> >> >> > >>> > >>> > > > (as DistributedLog does) > > >>>>>>>>>>>> > > >>> >> >> > >>> > >>> > > > The next step is to enable > > >>>>>>>>>>>> cobertura and > > >>>>>>>>>>>> > tests > > >>>>>>>>>>>> > > in > > >>>>>>>>>>>> > > >>> >> >> Travis. > > >>>>>>>>>>>> > > >>> >> >> > >>> > >>> > > > Currently we do not run > tests > > >>>>>>>>>>>> on Travis > > >>>>>>>>>>>> > because > > >>>>>>>>>>>> > > >>> it is > > >>>>>>>>>>>> > > >>> >> >> very > > >>>>>>>>>>>> > > >>> >> >> > >>> > >>> slow..... > > >>>>>>>>>>>> > > >>> >> >> > >>> > >>> > > > any ideas ? > > >>>>>>>>>>>> > > >>> >> >> > >>> > >>> > > > > > >>>>>>>>>>>> > > >>> >> >> > >>> > >>> > > > > >>>>>>>>>>>> > > >>> >> >> > >>> > >>> > > I believe you need a coveralls > > >>>>>>>>>>>> token for > > >>>>>>>>>>>> > > reporting > > >>>>>>>>>>>> > > >>> the > > >>>>>>>>>>>> > > >>> >> >> > coverage > > >>>>>>>>>>>> > > >>> >> >> > >>> > >>> results. > > >>>>>>>>>>>> > > >>> >> >> > >>> > >>> > > Jenkins is probably better for > > >>>>>>>>>>>> this purpose > > >>>>>>>>>>>> > > >>> because the > > >>>>>>>>>>>> > > >>> >> >> token > > >>>>>>>>>>>> > > >>> >> >> > >>> has > > >>>>>>>>>>>> > > >>> >> >> > >>> > to > > >>>>>>>>>>>> > > >>> >> >> > >>> > >>> be > > >>>>>>>>>>>> > > >>> >> >> > >>> > >>> > > store secretly. > > >>>>>>>>>>>> > > >>> >> >> > >>> > >>> > > > > >>>>>>>>>>>> > > >>> >> >> > >>> > >>> > > > >>>>>>>>>>>> > > >>> >> >> > >>> > >>> > In this doc it seems that using > > the > > >>>>>>>>>>>> coveralls > > >>>>>>>>>>>> > > plugin > > >>>>>>>>>>>> > > >>> >> inside > > >>>>>>>>>>>> > > >>> >> >> a > > >>>>>>>>>>>> > > >>> >> >> > >>> travis > > >>>>>>>>>>>> > > >>> >> >> > >>> > >>> build > > >>>>>>>>>>>> > > >>> >> >> > >>> > >>> > does not require tokens > > >>>>>>>>>>>> > > >>> >> >> > >>> > >>> > > > >>>>>>>>>>>> > > >>> >> >> > >>> > >>> > > > >>>>>>>>>>>> > > >>> >> >> > >>> > >>> https://github.com/trautonen/ > > >>>>>>>>>>>> > > >>> coveralls-maven-plugin/blob/ > > >>>>>>>>>>>> > > >>> >> >> > >>> > master/README.md > > >>>>>>>>>>>> > > >>> >> >> > >>> > >>> > > >>>>>>>>>>>> > > >>> >> >> > >>> > >>> > > >>>>>>>>>>>> > > >>> >> >> > >>> > >>> > > >>>>>>>>>>>> > > >>> >> >> > >>> > >>> I barely remembered that it > doesn't > > >>>>>>>>>>>> actually work, > > >>>>>>>>>>>> > > but > > >>>>>>>>>>>> > > >>> I > > >>>>>>>>>>>> > > >>> >> >> might be > > >>>>>>>>>>>> > > >>> >> >> > >>> > wrong. > > >>>>>>>>>>>> > > >>> >> >> > >>> > >>> > > >>>>>>>>>>>> > > >>> >> >> > >>> > >> > > >>>>>>>>>>>> > > >>> >> >> > >>> > >> I just pinged infra, my email was > not > > >>>>>>>>>>>> responded > > >>>>>>>>>>>> > > >>> >> >> > >>> > >> > > >>>>>>>>>>>> > > >>> >> >> > >>> > >> Enrico > > >>>>>>>>>>>> > > >>> >> >> > >>> > >> > > >>>>>>>>>>>> > > >>> >> >> > >>> > >>> > > >>>>>>>>>>>> > > >>> >> >> > >>> > >>> > > >>>>>>>>>>>> > > >>> >> >> > >>> > >>> > > > >>>>>>>>>>>> > > >>> >> >> > >>> > >>> > > > >>>>>>>>>>>> > > >>> >> >> > >>> > >>> > Enrico > > >>>>>>>>>>>> > > >>> >> >> > >>> > >>> > > > >>>>>>>>>>>> > > >>> >> >> > >>> > >>> > > > >>>>>>>>>>>> > > >>> >> >> > >>> > >>> > > > > >>>>>>>>>>>> > > >>> >> >> > >>> > >>> > > > > > >>>>>>>>>>>> > > >>> >> >> > >>> > >>> > > > Meanwhile I am going to > submit > > >>>>>>>>>>>> a PR with a > > >>>>>>>>>>>> > > >>> change in > > >>>>>>>>>>>> > > >>> >> >> Travis > > >>>>>>>>>>>> > > >>> >> >> > >>> > config > > >>>>>>>>>>>> > > >>> >> >> > >>> > >>> in > > >>>>>>>>>>>> > > >>> >> >> > >>> > >>> > > order > > >>>>>>>>>>>> > > >>> >> >> > >>> > >>> > > > to test Cobertura speed on > > >>>>>>>>>>>> Travis > > >>>>>>>>>>>> > > >>> >> >> > >>> > >>> > > > > > >>>>>>>>>>>> > > >>> >> >> > >>> > >>> > > > -- Enrico > > >>>>>>>>>>>> > > >>> >> >> > >>> > >>> > > > > > >>>>>>>>>>>> > > >>> >> >> > >>> > >>> > > > 2017-07-06 17:30 GMT+02:00 > > >>>>>>>>>>>> Enrico Olivelli < > > >>>>>>>>>>>> > > >>> >> >> > >>> eolive...@gmail.com > > >>>>>>>>>>>> > > >>> >> >> > >>> > >: > > >>>>>>>>>>>> > > >>> >> >> > >>> > >>> > > > > > >>>>>>>>>>>> > > >>> >> >> > >>> > >>> > > > > > > >>>>>>>>>>>> > > >>> >> >> > >>> > >>> > > > > > > >>>>>>>>>>>> > > >>> >> >> > >>> > >>> > > > > 2017-07-05 16:13 GMT+02:00 > > >>>>>>>>>>>> Enrico Olivelli > > >>>>>>>>>>>> > < > > >>>>>>>>>>>> > > >>> >> >> > >>> > eolive...@gmail.com > > >>>>>>>>>>>> > > >>> >> >> > >>> > >>> >: > > >>>>>>>>>>>> > > >>> >> >> > >>> > >>> > > > > > > >>>>>>>>>>>> > > >>> >> >> > >>> > >>> > > > >> > > >>>>>>>>>>>> > > >>> >> >> > >>> > >>> > > > >> > > >>>>>>>>>>>> > > >>> >> >> > >>> > >>> > > > >> 2017-07-05 16:05 > GMT+02:00 > > >>>>>>>>>>>> Sijie Guo < > > >>>>>>>>>>>> > > >>> >> >> > guosi...@gmail.com > > >>>>>>>>>>>> > > >>> >> >> > >>> >: > > >>>>>>>>>>>> > > >>> >> >> > >>> > >>> > > > >> > > >>>>>>>>>>>> > > >>> >> >> > >>> > >>> > > > >>> It would be worth > checking > > >>>>>>>>>>>> with the INFRA > > >>>>>>>>>>>> > > to > > >>>>>>>>>>>> > > >>> see > > >>>>>>>>>>>> > > >>> >> >> what > > >>>>>>>>>>>> > > >>> >> >> > >>> kind of > > >>>>>>>>>>>> > > >>> >> >> > >>> > >>> code > > >>>>>>>>>>>> > > >>> >> >> > >>> > >>> > > > >>> coverage > > >>>>>>>>>>>> > > >>> >> >> > >>> > >>> > > > >>> that it provides. I > > >>>>>>>>>>>> remembered that a few > > >>>>>>>>>>>> > > ASF > > >>>>>>>>>>>> > > >>> >> >> projects > > >>>>>>>>>>>> > > >>> >> >> > >>> using > > >>>>>>>>>>>> > > >>> >> >> > >>> > >>> > > > >>> coveralls.io, > > >>>>>>>>>>>> > > >>> >> >> > >>> > >>> > > > >>> INFRA can enable this > for > > >>>>>>>>>>>> you and give > > >>>>>>>>>>>> > you > > >>>>>>>>>>>> > > an > > >>>>>>>>>>>> > > >>> >> token > > >>>>>>>>>>>> > > >>> >> >> > >>> which you > > >>>>>>>>>>>> > > >>> >> >> > >>> > >>> can > > >>>>>>>>>>>> > > >>> >> >> > >>> > >>> > > > >>> configure > > >>>>>>>>>>>> > > >>> >> >> > >>> > >>> > > > >>> in the CI job. I think > > >>>>>>>>>>>> coveralls supports > > >>>>>>>>>>>> > > >>> JaCoco. > > >>>>>>>>>>>> > > >>> >> >> > >>> > >>> > > > >>> > > >>>>>>>>>>>> > > >>> >> >> > >>> > >>> > > > >> > > >>>>>>>>>>>> > > >>> >> >> > >>> > >>> > > > >> I will sent and email to > > >>>>>>>>>>>> INFRA and come > > >>>>>>>>>>>> > back > > >>>>>>>>>>>> > > >>> with > > >>>>>>>>>>>> > > >>> >> the > > >>>>>>>>>>>> > > >>> >> >> > >>> response > > >>>>>>>>>>>> > > >>> >> >> > >>> > >>> > > > >> > > >>>>>>>>>>>> > > >>> >> >> > >>> > >>> > > > >> > > >>>>>>>>>>>> > > >>> >> >> > >>> > >>> > > > > > > >>>>>>>>>>>> > > >>> >> >> > >>> > >>> > > > > no answer from INFRA > > >>>>>>>>>>>> > > >>> >> >> > >>> > >>> > > > > > > >>>>>>>>>>>> > > >>> >> >> > >>> > >>> > > > > meanwhile I was playing > with > > >>>>>>>>>>>> coveralls.io > > >>>>>>>>>>>> > > and > > >>>>>>>>>>>> > > >>> I > > >>>>>>>>>>>> > > >>> >> found > > >>>>>>>>>>>> > > >>> >> >> > the > > >>>>>>>>>>>> > > >>> >> >> > >>> our > > >>>>>>>>>>>> > > >>> >> >> > >>> > >>> repo > > >>>>>>>>>>>> > > >>> >> >> > >>> > >>> > is > > >>>>>>>>>>>> > > >>> >> >> > >>> > >>> > > > > somehow already > configured, > > >>>>>>>>>>>> see > > >>>>>>>>>>>> > > >>> >> >> > >>> > >>> > > > > > > https://coveralls.io/github/ > > >>>>>>>>>>>> > > apache/bookkeeper > > >>>>>>>>>>>> > > >>> >> >> > >>> > >>> > > > > > > >>>>>>>>>>>> > > >>> >> >> > >>> > >>> > > > > I don't know whoever did > it > > >>>>>>>>>>>> > > >>> >> >> > >>> > >>> > > > > > > >>>>>>>>>>>> > > >>> >> >> > >>> > >>> > > > > It seams that in order to > > >>>>>>>>>>>> have such service > > >>>>>>>>>>>> > > use > > >>>>>>>>>>>> > > >>> >> have > > >>>>>>>>>>>> > > >>> >> >> to > > >>>>>>>>>>>> > > >>> >> >> > >>> include > > >>>>>>>>>>>> > > >>> >> >> > >>> > >>> > JaCoCo > > >>>>>>>>>>>> > > >>> >> >> > >>> > >>> > > > > maven plugin, then enable > > >>>>>>>>>>>> some automatic > > >>>>>>>>>>>> > > build > > >>>>>>>>>>>> > > >>> >> using > > >>>>>>>>>>>> > > >>> >> >> > >>> Jenkins or > > >>>>>>>>>>>> > > >>> >> >> > >>> > >>> > Travis > > >>>>>>>>>>>> > > >>> >> >> > >>> > >>> > > > and > > >>>>>>>>>>>> > > >>> >> >> > >>> > >>> > > > > then add another coveralls > > >>>>>>>>>>>> plugin > > >>>>>>>>>>>> > > >>> >> >> > >>> > >>> > > > > > > >>>>>>>>>>>> > > >>> >> >> > >>> > >>> > > > > see this interesting > > document > > >>>>>>>>>>>> > > >>> >> >> > >>> > >>> > > > > https://github.com/ > > >>>>>>>>>>>> > > >>> asciidoctor/asciidoctor/wiki/ > > >>>>>>>>>>>> > > >>> >> >> > >>> > >>> > > > > > Coveralls.io-configuration- > > >>>>>>>>>>>> > > for-maven-projects > > >>>>>>>>>>>> > > >>> >> >> > >>> > >>> > > > > > > >>>>>>>>>>>> > > >>> >> >> > >>> > >>> > > > > I think that the starting > > >>>>>>>>>>>> point is to > > >>>>>>>>>>>> > > integrate > > >>>>>>>>>>>> > > >>> >> JaCoco > > >>>>>>>>>>>> > > >>> >> >> > >>> > >>> > > > > > > >>>>>>>>>>>> > > >>> >> >> > >>> > >>> > > > > > > >>>>>>>>>>>> > > >>> >> >> > >>> > >>> > > > > > > >>>>>>>>>>>> > > >>> >> >> > >>> > >>> > > > > -- Enrico > > >>>>>>>>>>>> > > >>> >> >> > >>> > >>> > > > > > > >>>>>>>>>>>> > > >>> >> >> > >>> > >>> > > > > > > >>>>>>>>>>>> > > >>> >> >> > >>> > >>> > > > > > > >>>>>>>>>>>> > > >>> >> >> > >>> > >>> > > > >> -- Enrico > > >>>>>>>>>>>> > > >>> >> >> > >>> > >>> > > > >> > > >>>>>>>>>>>> > > >>> >> >> > >>> > >>> > > > >> > > >>>>>>>>>>>> > > >>> >> >> > >>> > >>> > > > >>> > > >>>>>>>>>>>> > > >>> >> >> > >>> > >>> > > > >>> Sijie > > >>>>>>>>>>>> > > >>> >> >> > >>> > >>> > > > >>> > > >>>>>>>>>>>> > > >>> >> >> > >>> > >>> > > > >>> On Jul 5, 2017 1:57 AM, > > >>>>>>>>>>>> "Enrico > > >>>>>>>>>>>> > Olivelli" < > > >>>>>>>>>>>> > > >>> >> >> > >>> > eolive...@gmail.com > > >>>>>>>>>>>> > > >>> >> >> > >>> > >>> > > > >>>>>>>>>>>> > > >>> >> >> > >>> > >>> > > wrote: > > >>>>>>>>>>>> > > >>> >> >> > >>> > >>> > > > >>> > > >>>>>>>>>>>> > > >>> >> >> > >>> > >>> > > > >>> > I am attaching a > report > > >>>>>>>>>>>> (I am using > > >>>>>>>>>>>> > > Google > > >>>>>>>>>>>> > > >>> >> Drive > > >>>>>>>>>>>> > > >>> >> >> as > > >>>>>>>>>>>> > > >>> >> >> > >>> most > > >>>>>>>>>>>> > > >>> >> >> > >>> > >>> email > > >>>>>>>>>>>> > > >>> >> >> > >>> > >>> > > > clients > > >>>>>>>>>>>> > > >>> >> >> > >>> > >>> > > > >>> will > > >>>>>>>>>>>> > > >>> >> >> > >>> > >>> > > > >>> > block attachments with > > >>>>>>>>>>>> html inside) . > > >>>>>>>>>>>> > > >>> >> >> > >>> > >>> > > > >>> > Just untar and play > with > > >>>>>>>>>>>> your browser > > >>>>>>>>>>>> > > >>> >> >> > >>> > >>> > > > >>> > > > >>>>>>>>>>>> > > >>> >> >> > >>> > >>> > > > >>> > > > >>>>>>>>>>>> > > >>> >> >> > >>> > >>> > > > >>> > 2017-07-04 18:18 > > >>>>>>>>>>>> GMT+02:00 Dávid > > >>>>>>>>>>>> > > Szigecsán > > >>>>>>>>>>>> > > >>> < > > >>>>>>>>>>>> > > >>> >> >> > >>> > >>> sige...@gmail.com>: > > >>>>>>>>>>>> > > >>> >> >> > >>> > >>> > > > >>> > > SonarCloud is free > for > > >>>>>>>>>>>> open source. > > >>>>>>>>>>>> > > >>> >> >> > >>> > >>> > > > >>> > > > > >>>>>>>>>>>> > > >>> >> >> > >>> > >>> > > > >>> > > > > >>>>>>>>>>>> https://about.sonarcloud.io/ > > >>>>>>>>>>>> > > >>> >> >> > >>> > >>> > > > >>> > > > >>>>>>>>>>>> > > >>> >> >> > >>> > >>> > > > >>> > Yes I known > > >>>>>>>>>>>> > > >>> >> >> > >>> > >>> > > > >>> > recently we have > > >>>>>>>>>>>> activated Travis CI, I > > >>>>>>>>>>>> > > >>> think > > >>>>>>>>>>>> > > >>> >> that > > >>>>>>>>>>>> > > >>> >> >> > the > > >>>>>>>>>>>> > > >>> >> >> > >>> > trend > > >>>>>>>>>>>> > > >>> >> >> > >>> > >>> is > > >>>>>>>>>>>> > > >>> >> >> > >>> > >>> > to > > >>>>>>>>>>>> > > >>> >> >> > >>> > >>> > > > >>> > integrate with such > free > > >>>>>>>>>>>> tools in order > > >>>>>>>>>>>> > > to > > >>>>>>>>>>>> > > >>> >> achieve > > >>>>>>>>>>>> > > >>> >> >> > the > > >>>>>>>>>>>> > > >>> >> >> > >>> best > > >>>>>>>>>>>> > > >>> >> >> > >>> > >>> > quality > > >>>>>>>>>>>> > > >>> >> >> > >>> > >>> > > > >>> with > > >>>>>>>>>>>> > > >>> >> >> > >>> > >>> > > > >>> > the minimum effort > > >>>>>>>>>>>> > > >>> >> >> > >>> > >>> > > > >>> > > > >>>>>>>>>>>> > > >>> >> >> > >>> > >>> > > > >>> > > > >>>>>>>>>>>> > > >>> >> >> > >>> > >>> > > > >>> > -- Enrico > > >>>>>>>>>>>> > > >>> >> >> > >>> > >>> > > > >>> > > > >>>>>>>>>>>> > > >>> >> >> > >>> > >>> > > > >>> > > > >>>>>>>>>>>> > > >>> >> >> > >>> > >>> > > > >>> > > > > >>>>>>>>>>>> > > >>> >> >> > >>> > >>> > > > >>> > > 2017-07-04 18:15 > > >>>>>>>>>>>> GMT+02:00 Enrico > > >>>>>>>>>>>> > > >>> Olivelli < > > >>>>>>>>>>>> > > >>> >> >> > >>> > >>> > eolive...@gmail.com > > >>>>>>>>>>>> > > >>> >> >> > >>> > >>> > > >: > > >>>>>>>>>>>> > > >>> >> >> > >>> > >>> > > > >>> > > > > >>>>>>>>>>>> > > >>> >> >> > >>> > >>> > > > >>> > >> Il mar 4 lug 2017, > > >>>>>>>>>>>> 18:08 Dávid > > >>>>>>>>>>>> > > >>> Szigecsán < > > >>>>>>>>>>>> > > >>> >> >> > >>> > >>> sige...@gmail.com> > > >>>>>>>>>>>> > > >>> >> >> > >>> > >>> > ha > > >>>>>>>>>>>> > > >>> >> >> > >>> > >>> > > > >>> > scritto: > > >>>>>>>>>>>> > > >>> >> >> > >>> > >>> > > > >>> > >> > > >>>>>>>>>>>> > > >>> >> >> > >>> > >>> > > > >>> > >> > +1 > > >>>>>>>>>>>> > > >>> >> >> > >>> > >>> > > > >>> > >> > > > >>>>>>>>>>>> > > >>> >> >> > >>> > >>> > > > >>> > >> > What do you think > > >>>>>>>>>>>> about sonar? :) > > >>>>>>>>>>>> > > >>> >> >> > >>> > >>> > > > >>> > >> > > > >>>>>>>>>>>> > > >>> >> >> > >>> > >>> > > > >>> > >> > > >>>>>>>>>>>> > > >>> >> >> > >>> > >>> > > > >>> > >> I am using sonar in > > >>>>>>>>>>>> projects in my > > >>>>>>>>>>>> > > >>> company, > > >>>>>>>>>>>> > > >>> >> it > > >>>>>>>>>>>> > > >>> >> >> is > > >>>>>>>>>>>> > > >>> >> >> > >>> great. > > >>>>>>>>>>>> > > >>> >> >> > >>> > >>> > > > >>> > >> But it needs a > > server, > > >>>>>>>>>>>> so it is not > > >>>>>>>>>>>> > > >>> easy to > > >>>>>>>>>>>> > > >>> >> use > > >>>>>>>>>>>> > > >>> >> >> > for > > >>>>>>>>>>>> > > >>> >> >> > >>> > >>> everyone > > >>>>>>>>>>>> > > >>> >> >> > >>> > >>> > > > >>> > >> > > >>>>>>>>>>>> > > >>> >> >> > >>> > >>> > > > >>> > >> Enrico > > >>>>>>>>>>>> > > >>> >> >> > >>> > >>> > > > >>> > >> > > >>>>>>>>>>>> > > >>> >> >> > >>> > >>> > > > >>> > >> > > > >>>>>>>>>>>> > > >>> >> >> > >>> > >>> > > > >>> > >> > 2017-07-04 17:24 > > >>>>>>>>>>>> GMT+02:00 Enrico > > >>>>>>>>>>>> > > >>> >> Olivelli < > > >>>>>>>>>>>> > > >>> >> >> > >>> > >>> > > eolive...@gmail.com > > >>>>>>>>>>>> > > >>> >> >> > >>> > >>> > > > >: > > >>>>>>>>>>>> > > >>> >> >> > >>> > >>> > > > >>> > >> > > > >>>>>>>>>>>> > > >>> >> >> > >>> > >>> > > > >>> > >> > > Hi all, > > >>>>>>>>>>>> > > >>> >> >> > >>> > >>> > > > >>> > >> > > as we are in > the > > >>>>>>>>>>>> mood of > > >>>>>>>>>>>> > enhancing > > >>>>>>>>>>>> > > >>> >> >> automatic > > >>>>>>>>>>>> > > >>> >> >> > QA > > >>>>>>>>>>>> > > >>> >> >> > >>> and > > >>>>>>>>>>>> > > >>> >> >> > >>> > >>> > quality > > >>>>>>>>>>>> > > >>> >> >> > >>> > >>> > > of > > >>>>>>>>>>>> > > >>> >> >> > >>> > >>> > > > >>> > >> > > BookKeeper I > have > > >>>>>>>>>>>> created this > > >>>>>>>>>>>> > > >>> issue in > > >>>>>>>>>>>> > > >>> >> >> order > > >>>>>>>>>>>> > > >>> >> >> > to > > >>>>>>>>>>>> > > >>> >> >> > >>> > >>> introduce > > >>>>>>>>>>>> > > >>> >> >> > >>> > >>> > > > >>> JaCoCo in > > >>>>>>>>>>>> > > >>> >> >> > >>> > >>> > > > >>> > >> > > the build/test > > >>>>>>>>>>>> pipeline > > >>>>>>>>>>>> > > >>> >> >> > >>> > >>> > > > >>> > >> > > > > >>>>>>>>>>>> > > >>> >> >> > >>> > >>> > > > >>> > >> > > > > >>>>>>>>>>>> https://github.com/apache/ > > >>>>>>>>>>>> > > >>> >> >> > bookkeeper/issues/232 > > >>>>>>>>>>>> > > >>> >> >> > >>> > >>> > > > >>> > >> > > > > >>>>>>>>>>>> > > >>> >> >> > >>> > >>> > > > >>> > >> > > I will do some > > >>>>>>>>>>>> tests and report > > >>>>>>>>>>>> > to > > >>>>>>>>>>>> > > >>> the > > >>>>>>>>>>>> > > >>> >> >> > comunity > > >>>>>>>>>>>> > > >>> >> >> > >>> > >>> > > > >>> > >> > > > > >>>>>>>>>>>> > > >>> >> >> > >>> > >>> > > > >>> > >> > > -- Enrico > > >>>>>>>>>>>> > > >>> >> >> > >>> > >>> > > > >>> > >> > > > > >>>>>>>>>>>> > > >>> >> >> > >>> > >>> > > > >>> > >> > > > >>>>>>>>>>>> > > >>> >> >> > >>> > >>> > > > >>> > >> -- > > >>>>>>>>>>>> > > >>> >> >> > >>> > >>> > > > >>> > >> > > >>>>>>>>>>>> > > >>> >> >> > >>> > >>> > > > >>> > >> > > >>>>>>>>>>>> > > >>> >> >> > >>> > >>> > > > >>> > >> -- Enrico Olivelli > > >>>>>>>>>>>> > > >>> >> >> > >>> > >>> > > > >>> > >> > > >>>>>>>>>>>> > > >>> >> >> > >>> > >>> > > > >>> > ​ > > >>>>>>>>>>>> > > >>> >> >> > >>> > >>> > > > >>> > > > >>>>>>>>>>>> bookkeeper-server-jacoco.tar.gz > > >>>>>>>>>>>> > > > -- -- Enrico Olivelli
