From dev-return-948-archive-asf-public=cust-asf.ponee.io@fluo.apache.org Wed Dec 18 19:30:42 2019 Return-Path: X-Original-To: archive-asf-public@cust-asf.ponee.io Delivered-To: archive-asf-public@cust-asf.ponee.io Received: from mail.apache.org (hermes.apache.org [207.244.88.153]) by mx-eu-01.ponee.io (Postfix) with SMTP id 141B718065B for ; Wed, 18 Dec 2019 20:30:41 +0100 (CET) Received: (qmail 15996 invoked by uid 500); 18 Dec 2019 19:30:41 -0000 Mailing-List: contact dev-help@fluo.apache.org; run by ezmlm Precedence: bulk List-Help: List-Unsubscribe: List-Post: List-Id: Reply-To: dev@fluo.apache.org Delivered-To: mailing list dev@fluo.apache.org Received: (qmail 15984 invoked by uid 99); 18 Dec 2019 19:30:41 -0000 Received: from pnap-us-west-generic-nat.apache.org (HELO spamd2-us-west.apache.org) (209.188.14.142) by apache.org (qpsmtpd/0.29) with ESMTP; Wed, 18 Dec 2019 19:30:41 +0000 Received: from localhost (localhost [127.0.0.1]) by spamd2-us-west.apache.org (ASF Mail Server at spamd2-us-west.apache.org) with ESMTP id A50EC1A33C5 for ; Wed, 18 Dec 2019 19:30:40 +0000 (UTC) X-Virus-Scanned: Debian amavisd-new at spamd2-us-west.apache.org X-Spam-Flag: NO X-Spam-Score: 0.25 X-Spam-Level: X-Spam-Status: No, score=0.25 tagged_above=-999 required=6.31 tests=[DKIM_SIGNED=0.1, DKIM_VALID=-0.1, DKIM_VALID_AU=-0.1, DKIM_VALID_EF=-0.1, HTML_MESSAGE=0.2, KAM_LOTSOFHASH=0.25, RCVD_IN_DNSWL_NONE=-0.0001, SPF_HELO_NONE=0.001, SPF_PASS=-0.001] autolearn=disabled Authentication-Results: spamd2-us-west.apache.org (amavisd-new); dkim=pass (2048-bit key) header.d=gmail.com Received: from mx1-he-de.apache.org ([10.40.0.8]) by localhost (spamd2-us-west.apache.org [10.40.0.9]) (amavisd-new, port 10024) with ESMTP id vOMiT2ly-b3e for ; Wed, 18 Dec 2019 19:30:38 +0000 (UTC) Received-SPF: Pass (mailfrom) identity=mailfrom; client-ip=2a00:1450:4864:20::344; helo=mail-wm1-x344.google.com; envelope-from=glorifiedcalculator@gmail.com; receiver= Received: from mail-wm1-x344.google.com (mail-wm1-x344.google.com [IPv6:2a00:1450:4864:20::344]) by mx1-he-de.apache.org (ASF Mail Server at mx1-he-de.apache.org) with ESMTPS id 4360F7DDAC for ; Wed, 18 Dec 2019 19:30:38 +0000 (UTC) Received: by mail-wm1-x344.google.com with SMTP id m24so3049616wmc.3 for ; Wed, 18 Dec 2019 11:30:38 -0800 (PST) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=gmail.com; s=20161025; h=mime-version:references:in-reply-to:from:date:message-id:subject:to; bh=t5D4yQonkWxy/xnIH7hwaTeeXL8BLKCoqmk21EMfFHc=; b=ll4f7cg2Ws9v3XYl7zi6zyd1Af4nRi4Cf1nA1Rhz3wwKIRh2OpnqtmzHFd0zrbdFEj 5V08+S0kF0x32ClLctl7imERnAx875+KRP3ffEYZh7tC8K7ca6OlPNOOgIqIX1gGyStD WbiWqHbdlXrO8ybI9AAkgUD0WbXiR95z3A7Jl3BueFqxWg7DWxoZeiQ+yxF8FEKxmBiQ 0LpUhUJ9EjQAEM5yK7xA5ubTjX98QSwpqLlF4GInhhaHFAfrW6Ttnl7E+HEfwji1uvBZ ty/IVyA4jJ0pFHK8Qn8q1ZB0kE015n8TzFAq2y7KNVxfgurE9ev2m6Rc9OBr5TZnP982 QR0w== X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20161025; h=x-gm-message-state:mime-version:references:in-reply-to:from:date :message-id:subject:to; bh=t5D4yQonkWxy/xnIH7hwaTeeXL8BLKCoqmk21EMfFHc=; b=b+ahFgXhzEwrfDPhHnc76vg8FeBHALbvQU7ic/PAH62Bv15Sqdogop4XuXkRPzQPnC Nkc2i0lsU6lLCsuDW+CP41MCNuwRrH9J/RckfotFcS76FngrXtdh7WcgzqiRk4gWUjQU ietRGHK89wCfspv0z8YlvgmWJmMI5YYxetxXVS6FTxhCIfqPlDSaeqPxWqbyppTKb6yS ESAAXqmrWF9Ao09jQqOoUmguRc7rfn6NKha65Sa+ZqoAEmnG8RrB21Bew3Y8h/+wG+Lq mxBBDTQhKyAW+ef5NH0v5rDqgldAFOxPEBepZOL06Og6yMH3NfO/toa1dhzi+cU5aBKj v+3w== X-Gm-Message-State: APjAAAXHAe6bgd9SCrr/IPABDKEaFYXgE/ePiO3XMxoo51Ud/QhdMwGm u1GjocEYgFoyc0BLKXnIM+lGa4GK0cmYdHju20esbA== X-Google-Smtp-Source: APXvYqyB4H9WDO08vVLSxU5PE+FXAjFMC3n8/4FdBJG0IxIhAkvi6yvV6/oZzCeaAYMqO9jDD8wTG+PJHuS1C/DR7Qc= X-Received: by 2002:a1c:67c3:: with SMTP id b186mr5031132wmc.36.1576697437521; Wed, 18 Dec 2019 11:30:37 -0800 (PST) MIME-Version: 1.0 References: In-Reply-To: From: Kenneth McFarland Date: Wed, 18 Dec 2019 11:30:25 -0800 Message-ID: Subject: Re: OracleIT and Rat check failures To: dev@fluo.apache.org Content-Type: multipart/alternative; boundary="000000000000ae8ab50599ff7a66" --000000000000ae8ab50599ff7a66 Content-Type: text/plain; charset="UTF-8" @Christopher I'll defer to your expertise. Since the test passes and everyone is ok with it, it's probably better to just leave it. Happy holidays On Tue, Dec 17, 2019, 10:48 PM Christopher wrote: > I'm not necessarily opposed, but I'm not sure it will help above and > beyond the fact that the test passes regardless of those messages. If > possible, it'd probably be better to modify the logging configuration > during the execution of that test to make it less spammy, rather than > add more debug messages. > > On Sat, Dec 14, 2019 at 12:51 PM Kenneth McFarland > wrote: > > > > Does anyone have any issue with the idea of putting some debug message > that > > informs the user the OracleIT errors are expected? > > > > I'm in favor due to the limitations of my memory. I got confused. Maybe > > just wrap the output. Any objections? I feel it's good documentation. > > > > On Mon, Dec 2, 2019, 3:00 PM Christopher wrote: > > > > > I wouldn't worry about documenting it. It's more of a tooling (and > > > tooling familiarity) issue than anything pertaining to Fluo, so I > > > would consider it out of scope of Fluo's own documentation. Besides, > > > documenting everything that can go wrong with build tooling could > > > easily overwhelm the docs. It's better to just work through it in the > > > community forums like this email list, when somebody encounters an > > > issue. > > > > > > One reason this might not have been more obvious is that the > > > `.gitignore` file in Fluo is too overly-broad. It matches all files > > > and directories named `target` anywhere in the project (even those no > > > longer part of the project). So, `git status` wouldn't show them... > > > they'd be hidden instead. In Accumulo, we addressed this by making the > > > `.gitignore` file only match on `/target/` (this pattern ignores only > > > directories that are at the same depth as the `.gitignore` file > > > itself), and having a separate `.gitignore` file for each module with > > > the same contents. That way, if we removed a directory, no > > > `.gitignore` pattern would match on it any more. We could do the same > > > in Fluo. > > > > > > Another option would be that somebody could submit a patch for the > > > apache-rat-plugin that fixes its default excludes pattern matching to > > > use the same logic as that of the `.gitignore` pattern matching. It > > > would fix at least this plugin... but other plugins could still behave > > > badly when the workspace is "dirty" in this way. > > > > > > On Thu, Nov 28, 2019 at 3:26 PM Kenneth McFarland > > > wrote: > > > > > > > > Thank you Christopher, that worked perfectly. Issue resolved, and I > > > learned > > > > something. Thanks again! > > > > > > > > I'd be happy to add this to some documentation but not sure where it > > > would > > > > belong. Might be a niche thing. > > > > > > > > Kenny > > > > > > > > On Thu, Nov 28, 2019, 11:30 AM Christopher > wrote: > > > > > > > > > Okay, so it looks like the failures you're seeing are the result > of an > > > old > > > > > modules/integration directory that was removed or renamed. Try > > > resetting > > > > > your clone with `git clean -fdx` > > > > > > > > > > On Thu, Nov 28, 2019, 00:39 Kenneth McFarland < > > > > > glorifiedcalculator@gmail.com> > > > > > wrote: > > > > > > > > > > > @Christopher : Good, I thought that race condition was fixed in a > > > > > previous > > > > > > issue so I was concerned and should have looked closer. Thank you > > > for the > > > > > > catch sir! > > > > > > > > > > > > I'm still confused as to why the RAT check is throwing errors. I > am > > > > > > working off a clean master with the same hash as the one Joe > > > mentioned > > > > > > (commit 549d645addb330f4ae2e074447428cb86b5a9a3f). > > > > > > > > > > > > Here is some environment info: > > > > > > > > > > > > *MAVEN* > > > > > > Apache Maven 3.5.2 (138edd61fd100ec658bfa2d307c43b76940a5d7d; > > > > > > 2017-10-18T00:58:13-07:00) > > > > > > Maven home: /opt/apache-maven-3.5.2 > > > > > > Java version: 1.8.0_161, vendor: Oracle Corporation > > > > > > Java home: /usr/lib/jvm/jdk1.8.0_161/jre > > > > > > Default locale: en_US, platform encoding: UTF-8 > > > > > > OS name: "linux", version: "4.15.0-70-generic", arch: "amd64", > > > family: > > > > > > "unix" > > > > > > > > > > > > *JAVA* > > > > > > openjdk 11.0.5 2019-10-15 > > > > > > OpenJDK Runtime Environment (build > > > 11.0.5+10-post-Ubuntu-2ubuntu116.04) > > > > > > OpenJDK 64-Bit Server VM (build > 11.0.5+10-post-Ubuntu-2ubuntu116.04, > > > > > mixed > > > > > > mode, sharing) > > > > > > > > > > > > I also have a few other java versions installed like the Oracle > 1.8.0 > > > > > that > > > > > > maven reports above. > > > > > > > > > > > > *Operating System* > > > > > > Linux ubuntu 4.15.0-70-generic #79~16.04.1-Ubuntu SMP Tue Nov 12 > > > 14:01:10 > > > > > > UTC 2019 x86_64 x86_64 x86_64 GNU/Linux > > > > > > > > > > > > I will try this on my MacI notice the apache-rat > setDefaultExcludes > > > > > > variable is set to true. I'm currently baffled. The errors are > the > > > same > > > > > if > > > > > > I run 'mvn apache-rat:check' or if I run 'mvn verify' or 'mvn > clean > > > > > > verify''. The logs are kinda large so I simply piped it into a > text > > > file, > > > > > > as well as included the rat file. > > > > > > > > > > > > > > > > > > > > > > > > On Wed, Nov 27, 2019 at 7:06 PM Christopher > > > > wrote: > > > > > > > > > > > >> I think the stack traces in the logs/test output are intended > for > > > that > > > > > >> test, because it's specifically checking error handling for a > > > running > > > > > >> server (if I remember correctly). > > > > > >> > > > > > >> On Wed, Nov 27, 2019, 18:42 Joseph Koshakow > > > wrote: > > > > > >> > > > > > >> > I'm running `mvn clean verify` on the master branch > > > > > >> > (549d645addb330f4ae2e074447428cb86b5a9a3f) with OpenJDK 1.8 on > > > Ubuntu > > > > > >> 18.04 > > > > > >> > x86_64. I don't see any issues related to Rat. OracleIT passes > > > and the > > > > > >> > entire build is successful, but OracleIT does print the > following > > > > > >> errors in > > > > > >> > the logs > > > > > >> > > > > > > >> > Running org.apache.fluo.integration.impl.OracleIT > > > > > >> > 2019-11-27 18:18:24,087 [thrift.ProcessFunction] ERROR: > Internal > > > error > > > > > >> > processing getTimestamps > > > > > >> > java.lang.IllegalStateException: Received timestamp request > but > > > Oracle > > > > > >> is > > > > > >> > not leader > > > > > >> > at > > > > > >> > > > > > > >> > > > > > > >> > > > > > > > > > org.apache.fluo.core.oracle.OracleServer.getTimestampsImpl(OracleServer.java:244) > > > > > >> > at > > > > > >> > > > > > > >> > > > > > > >> > > > > > > > > > org.apache.fluo.core.oracle.OracleServer.getTimestamps(OracleServer.java:224) > > > > > >> > at > > > > > >> > > > > > > >> > > > > > > >> > > > > > > > > > org.apache.fluo.core.thrift.OracleService$Processor$getTimestamps.getResult(OracleService.java:279) > > > > > >> > at > > > > > >> > > > > > > >> > > > > > > >> > > > > > > > > > org.apache.fluo.core.thrift.OracleService$Processor$getTimestamps.getResult(OracleService.java:257) > > > > > >> > at > > > > > >> > > > > > > >> > > > > > > >> > > > > > > > > > org.apache.fluo.core.shaded.thrift.ProcessFunction.process(ProcessFunction.java:38) > > > > > >> > at > > > > > >> > > > > > > >> > > > > > > >> > > > > > > > > > org.apache.fluo.core.shaded.thrift.TBaseProcessor.process(TBaseProcessor.java:39) > > > > > >> > at > > > > > >> > > > > > > >> > > > > > > >> > > > > > > > > > org.apache.fluo.core.shaded.thrift.server.AbstractNonblockingServer$FrameBuffer.invoke(AbstractNonblockingServer.java:518) > > > > > >> > at > > > > > >> > > > > > > >> > > > > > > >> > > > > > > > > > org.apache.fluo.core.shaded.thrift.server.Invocation.run(Invocation.java:18) > > > > > >> > at > > > > > >> > > > > > > >> > > > > > > >> > > > > > > > > > java.util.concurrent.ThreadPoolExecutor.runWorker(ThreadPoolExecutor.java:1149) > > > > > >> > at > > > > > >> > > > > > > >> > > > > > > >> > > > > > > > > > java.util.concurrent.ThreadPoolExecutor$Worker.run(ThreadPoolExecutor.java:624) > > > > > >> > at java.lang.Thread.run(Thread.java:748) > > > > > >> > 2019-11-27 18:18:24,091 [oracle.OracleClient] ERROR: > TException > > > > > >> occurred in > > > > > >> > doWork() method > > > > > >> > org.apache.fluo.core.shaded.thrift.TApplicationException: > Internal > > > > > error > > > > > >> > processing getTimestamps > > > > > >> > at > > > > > >> > > > > > > >> > > > > > > >> > > > > > > > > > org.apache.fluo.core.shaded.thrift.TServiceClient.receiveBase(TServiceClient.java:79) > > > > > >> > at > > > > > >> > > > > > > >> > > > > > > >> > > > > > > > > > org.apache.fluo.core.thrift.OracleService$Client.recv_getTimestamps(OracleService.java:86) > > > > > >> > at > > > > > >> > > > > > > >> > > > > > > >> > > > > > > > > > org.apache.fluo.core.thrift.OracleService$Client.getTimestamps(OracleService.java:73) > > > > > >> > at > > > > > >> > > > > > > >> > > > > > > >> > > > > > > > > > org.apache.fluo.core.oracle.OracleClient$TimestampRetriever.doWork(OracleClient.java:182) > > > > > >> > at > > > > > >> > > > > > > >> > > > > > > >> > > > > > > > > > org.apache.fluo.core.oracle.OracleClient$TimestampRetriever.run(OracleClient.java:116) > > > > > >> > at java.lang.Thread.run(Thread.java:748) > > > > > >> > Tests run: 6, Failures: 0, Errors: 0, Skipped: 0, Time > elapsed: > > > 22.911 > > > > > >> sec > > > > > >> > - in org.apache.fluo.integration.impl.OracleIT > > > > > >> > > > > > > >> > -Joe > > > > > >> > > > > > > >> > On Wed, Nov 27, 2019 at 5:18 PM Christopher < > ctubbsii@apache.org> > > > > > >> wrote: > > > > > >> > > > > > > >> > > I have not seen any errors with `mvn clean verify` locally, > but > > > > > >> > > occasionally, there seems to be errors on Travis. > > > > > >> > > I'm building the master branch (currently > > > > > >> > > 549d645addb330f4ae2e074447428cb86b5a9a3f) with OpenJDK 11 on > > > Fedora > > > > > 31 > > > > > >> > > x86_64 with no problems. > > > > > >> > > > > > > > >> > > On Wed, Nov 27, 2019 at 2:31 PM Kenneth McFarland > > > > > >> > > wrote: > > > > > >> > > > > > > > > >> > > > Hello, > > > > > >> > > > > > > > > >> > > > This is just a quick inquiry if anyone else running mvn > > > verify is > > > > > >> > getting > > > > > >> > > > rat errors that prevent maven from completing. > > > > > >> > > > > > > > > >> > > > I'm using -Drat.skip=true to bypass but is this normal? > > > > > >> > > > > > > > > >> > > > Last thing is OracleIT is throwing errors again. I've > checked > > > out > > > > > >> the > > > > > >> > new > > > > > >> > > > code and am building on 16.10 Ubuntu x64. > > > > > >> > > > > > > > > >> > > > If this is something others are experiencing maybe we can > see > > > if > > > > > >> there > > > > > >> > is > > > > > >> > > > an open ticket for OracleIT (thought we closed it with > switch > > > on > > > > > >> lock > > > > > >> > > > implementation in Curator). Rat check is new to me. > > > > > >> > > > > > > > > >> > > > Thanks a bunch. This is informal, if I'm not insane I'll > do a > > > > > formal > > > > > >> > > ticket > > > > > >> > > > or something. Again, thanks! > > > > > >> > > > > > > > > >> > > > Happy holidays also. > > > > > >> > > > > > > > > >> > > > I > > > > > >> > > > > > > > >> > > > > > > >> > > > > > > > > > > > > > > > --000000000000ae8ab50599ff7a66--