commons-dev mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From "Bruno P. Kinoshita" <brunodepau...@yahoo.com.br>
Subject Re: [VOTE] Release pool 2.4 based on RC1
Date Sat, 23 May 2015 04:58:29 GMT
Thanks for clarifying Phil!
 [ X ] +1 Release these artifacts (not binding)
Bruno

 
      From: Phil Steitz <phil.steitz@gmail.com>
 To: Commons Developers List <dev@commons.apache.org> 
 Sent: Saturday, May 23, 2015 4:22 PM
 Subject: Re: [VOTE] Release pool 2.4 based on RC1
   
On 5/22/15 8:10 PM, Bruno P. Kinoshita wrote:
> Build works fine in 
>
> Apache Maven 3.2.5 (12a6b3acb947671f09b81f49094c53f426d8cea1; 2014-12-15T06:29:23+13:00)
> Maven home: /opt/apache-maven-3.2.5
> Java version: 1.8.0_45, vendor: Oracle Corporation
> Java home: /usr/lib/jvm/java-8-oracle/jre
> Default locale: en_US, platform encoding: UTF-8
> OS name: "linux", version: "3.16.0-38-generic", arch: "amd64", family: "unix"
> Rat reports 1 binary, against 2 binaries from the previous web site. I think it is because
the previous version included site.tgz in the report. One new error in checkstyle, but that
doesn't seem relevant.
> This version introduces five errors in FindBugs, all related to the same type [1] 
>
> I've changed locally a part of the code reported by FindBugs (GenericKeyedObjectPool
L755). From
>        for (K k : poolMap.keySet()) {
>            ObjectDeque<T> queue = poolMap.get(k);
> To
>
>        for (Map.Entry<K, ObjectDeque<T>> entry : poolMap.entrySet())
{
>            ObjectDeque<T> queue = entry.getValue();
> Running the build again, all tests pass and the FindBugs error is gone. But I'm not sure
if it wasn't intentional to avoid stale reads from the map.
> Could anyone confirm if the errors reported by FindBugs can be ignored, please?

Yes, "fixing" this would be a bad idea.  It is a false positive.  If
there is another RC, I will add a suppression filter for this "bug"
that must be due to findbugs version upgrade.  Otherwise, this can
be added post-release as part of the site update.

Thanks for the review!

Phil
> Thanks!Bruno
>
> http://findbugs.sourceforge.net/bugDescriptions.html#WMI_WRONG_MAP_ITERATOR
>
>  
>      From: Phil Steitz <phil.steitz@gmail.com>
>  To: Commons Developers List <dev@commons.apache.org> 
>  Sent: Saturday, May 23, 2015 3:51 AM
>  Subject: [VOTE] Release pool 2.4 based on RC1
>    
> This is a vote to release version 2.4 of Apache Commons Pool based
> on RC1.
>
> Here is the release candidate:
> https://dist.apache.org/repos/dist/dev/commons/pool/
> (r9083)
>
> Built from:
> https://svn.apache.org/repos/asf/commons/proper/pool/tags/POOL_2_4_RC1/
> (r1680963)
>
> Release notes:
> https://dist.apache.org/repos/dist/dev/commons/pool/RELEASE-NOTES.txt
>
> Maven artifacts:
> http://s.apache.org/Va6
>
> site:
> http://people.apache.org/~psteitz/pool/pool-2.4-rc1
>
> keys:
>
> https://www.apache.org/dist/commons/KEYS
>
> Votes, please.
>
> This vote will close no sooner that 72 hours from now (15:50 GMT
> 25-May-2015).
>
>  [ ] +1 Release these artifacts
>  [ ] +0 OK, but...
>  [ ] -0 OK, but really should fix...
>  [ ] -1 I oppose this release because...
>
> Thanks!
>
> Phil
>
>
>
> ---------------------------------------------------------------------
> To unsubscribe, e-mail: dev-unsubscribe@commons.apache.org
> For additional commands, e-mail: dev-help@commons.apache.org


>
>
>    
>



---------------------------------------------------------------------
To unsubscribe, e-mail: dev-unsubscribe@commons.apache.org
For additional commands, e-mail: dev-help@commons.apache.org


   

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