You are viewing a plain text version of this content. The canonical link for it is here.
Posted to issues@commons.apache.org by GitBox <gi...@apache.org> on 2019/12/23 18:33:26 UTC

[GitHub] [commons-beanutils] melloware opened a new pull request #21: [BEANUTILS-509] Wrap WeakHashMap in Collections.synchronizedMap

melloware opened a new pull request #21: [BEANUTILS-509] Wrap WeakHashMap in Collections.synchronizedMap
URL: https://github.com/apache/commons-beanutils/pull/21
 
 
   

----------------------------------------------------------------
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
 
For queries about this service, please contact Infrastructure at:
users@infra.apache.org


With regards,
Apache Git Services

[GitHub] [commons-beanutils] melloware commented on issue #21: [BEANUTILS-509] Wrap WeakHashMap in Collections.synchronizedMap

Posted by GitBox <gi...@apache.org>.
melloware commented on issue #21: [BEANUTILS-509] Wrap WeakHashMap in Collections.synchronizedMap
URL: https://github.com/apache/commons-beanutils/pull/21#issuecomment-574834874
 
 
   No idea if there is a perfomance measurement suite I was just reading the ticket and it seems the Jenkins team definitely has this bug and use case.  You are more than welcome to reject this PR.

----------------------------------------------------------------
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
 
For queries about this service, please contact Infrastructure at:
users@infra.apache.org


With regards,
Apache Git Services

[GitHub] [commons-beanutils] garydgregory commented on issue #21: [BEANUTILS-509] Wrap WeakHashMap in Collections.synchronizedMap

Posted by GitBox <gi...@apache.org>.
garydgregory commented on issue #21: [BEANUTILS-509] Wrap WeakHashMap in Collections.synchronizedMap
URL: https://github.com/apache/commons-beanutils/pull/21#issuecomment-574687350
 
 
   There are a few more clean ups IIRC, I will try to take a look over the weekend.

----------------------------------------------------------------
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
 
For queries about this service, please contact Infrastructure at:
users@infra.apache.org


With regards,
Apache Git Services

[GitHub] [commons-beanutils] melloware commented on issue #21: [BEANUTILS-509] Wrap WeakHashMap in Collections.synchronizedMap

Posted by GitBox <gi...@apache.org>.
melloware commented on issue #21: [BEANUTILS-509] Wrap WeakHashMap in Collections.synchronizedMap
URL: https://github.com/apache/commons-beanutils/pull/21#issuecomment-574713276
 
 
   Thank you very much.

----------------------------------------------------------------
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
 
For queries about this service, please contact Infrastructure at:
users@infra.apache.org


With regards,
Apache Git Services

[GitHub] [commons-beanutils] melloware commented on issue #21: [BEANUTILS-509] Wrap WeakHashMap in Collections.synchronizedMap

Posted by GitBox <gi...@apache.org>.
melloware commented on issue #21: [BEANUTILS-509] Wrap WeakHashMap in Collections.synchronizedMap
URL: https://github.com/apache/commons-beanutils/pull/21#issuecomment-574676791
 
 
   @garydgregory any chance of a Beanutils2 release anytime soon?  I think it just needs  the whole codebase to be formatted and i think its worthy of a release.

----------------------------------------------------------------
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
 
For queries about this service, please contact Infrastructure at:
users@infra.apache.org


With regards,
Apache Git Services

[GitHub] [commons-beanutils] melloware commented on issue #21: [BEANUTILS-509] Wrap WeakHashMap in Collections.synchronizedMap

Posted by GitBox <gi...@apache.org>.
melloware commented on issue #21: [BEANUTILS-509] Wrap WeakHashMap in Collections.synchronizedMap
URL: https://github.com/apache/commons-beanutils/pull/21#issuecomment-583375515
 
 
   @garydgregory Just pinging you again about a potential BeanUtils2 release?

----------------------------------------------------------------
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
 
For queries about this service, please contact Infrastructure at:
users@infra.apache.org


With regards,
Apache Git Services

[GitHub] [commons-beanutils] garydgregory commented on issue #21: [BEANUTILS-509] Wrap WeakHashMap in Collections.synchronizedMap

Posted by GitBox <gi...@apache.org>.
garydgregory commented on issue #21: [BEANUTILS-509] Wrap WeakHashMap in Collections.synchronizedMap
URL: https://github.com/apache/commons-beanutils/pull/21#issuecomment-574834449
 
 
   What are the performance consequences of this PR?

----------------------------------------------------------------
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
 
For queries about this service, please contact Infrastructure at:
users@infra.apache.org


With regards,
Apache Git Services