On Mon, Feb 5, 2018 at 1:04 PM, Enrico Olivelli <eolive...@gmail.com> wrote:
> 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 > just one suggestion when you are trying to restore CI jobs, please start with a separate CI job and let the CI job run for a while to ensure it doesn't have any side efforts before enforcing it on the other 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. > > > >>>>>>>>>>>> > > > > > >>>>>>>>>>>> getFunctionMetricsForSingleFil > e(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 >
