The patch is ok I merged it. Great, thanks! Now master is clean Enrico
Il sab 10 giu 2017, 05:49 Sijie Guo <guosi...@gmail.com> ha scritto: > I found the problem. I think it is a merge issue. there were two same > lines merged. sending a pull request out now. > > - Sijie > > On Fri, Jun 9, 2017 at 7:02 PM, Sijie Guo <guosi...@gmail.com> wrote: > >> Yes, I am working on it. Enrico pointed it out earlier. >> >> Sijie >> >> On Jun 9, 2017 6:23 PM, "Charan Reddy G" <reddychara...@gmail.com> wrote: >> >>> Hey Sijie, >>> >>> It seems one of your commit >>> >>> https://github.com/apache/bookkeeper/commit/5fe86525a9c823f79b3e97fd82ea4aa1c75c79eb >>> has broken CompactionTests tests. For me locally consistently some of the >>> tests in CompactionTest.java are failing with test timeout exceptions. >>> Can >>> you please look into the failures. >>> >>> Thanks, >>> Charan >>> >> > -- -- Enrico Olivelli
