zookeeper-dev mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From Patrick Hunt <phu...@gmail.com>
Subject Re: Failed: ZOOKEEPER- PreCommit Build #1427
Date Sun, 28 Jan 2018 01:25:52 GMT
Hi Jordan. You can add it to ./src/java/test/config/findbugsExcludeFile.xml
which is used during the check.

Patrick

On Fri, Jan 26, 2018 at 8:35 AM, Jordan Zimmerman <
jordan@jordanzimmerman.com> wrote:

>
> >     [exec]     -1 findbugs.  The patch appears to introduce 1 new
> Findbugs (version 3.0.1) warnings.
>
>
> -- RV_RETURN_VALUE_IGNORED_NO_SIDE_EFFECT: Return value of method without
> side effect is ignored
>
>
> This findbugs warning is wrong. It's not noticing that there's on override
> for the method in the Enum for TTL. How do we handle this? I don't see that
> the Findbugs annotations are included in the build.
>
> -Jordan

Mime
  • Unnamed multipart/alternative (inline, None, 0 bytes)
View raw message