[jira] [Work logged] (LANG-1547) fix code smells; fix typos; performance refine(trying to)

2020-05-31 Thread ASF GitHub Bot (Jira)
[ https://issues.apache.org/jira/browse/LANG-1547?focusedWorklogId=439363&page=com.atlassian.jira.plugin.system.issuetabpanels:worklog-tabpanel#worklog-439363 ] ASF GitHub Bot logged work on LANG-1547: Author: ASF GitHub Bot

[GitHub] [commons-lang] XenoAmess commented on pull request #533: [LANG-1547] fix code smells; fix typos

2020-05-31 Thread GitBox
XenoAmess commented on pull request #533: URL: https://github.com/apache/commons-lang/pull/533#issuecomment-636646635 > > I'd find it reasonable to consider your changes in batches of, say, ten apiece. (That would make it possible to carefully inspect one by one.) But that many in one go?

[jira] [Work logged] (LANG-1547) fix code smells; fix typos; performance refine(trying to)

2020-05-31 Thread ASF GitHub Bot (Jira)
[ https://issues.apache.org/jira/browse/LANG-1547?focusedWorklogId=439360&page=com.atlassian.jira.plugin.system.issuetabpanels:worklog-tabpanel#worklog-439360 ] ASF GitHub Bot logged work on LANG-1547: Author: ASF GitHub Bot

[GitHub] [commons-lang] kinow commented on pull request #533: [LANG-1547] fix code smells; fix typos

2020-05-31 Thread GitBox
kinow commented on pull request #533: URL: https://github.com/apache/commons-lang/pull/533#issuecomment-636643768 > I'd find it reasonable to consider your changes in batches of, say, ten apiece. (That would make it possible to carefully inspect one by one.) But that many in one go? Defini

[jira] [Work logged] (LANG-1406) StringIndexOutOfBoundsException in StringUtils.replaceIgnoreCase()

2020-05-31 Thread ASF GitHub Bot (Jira)
[ https://issues.apache.org/jira/browse/LANG-1406?focusedWorklogId=439356&page=com.atlassian.jira.plugin.system.issuetabpanels:worklog-tabpanel#worklog-439356 ] ASF GitHub Bot logged work on LANG-1406: Author: ASF GitHub Bot

[jira] [Work logged] (LANG-1406) StringIndexOutOfBoundsException in StringUtils.replaceIgnoreCase()

2020-05-31 Thread ASF GitHub Bot (Jira)
[ https://issues.apache.org/jira/browse/LANG-1406?focusedWorklogId=439357&page=com.atlassian.jira.plugin.system.issuetabpanels:worklog-tabpanel#worklog-439357 ] ASF GitHub Bot logged work on LANG-1406: Author: ASF GitHub Bot

[jira] [Work logged] (LANG-1406) StringIndexOutOfBoundsException in StringUtils.replaceIgnoreCase()

2020-05-31 Thread ASF GitHub Bot (Jira)
[ https://issues.apache.org/jira/browse/LANG-1406?focusedWorklogId=439358&page=com.atlassian.jira.plugin.system.issuetabpanels:worklog-tabpanel#worklog-439358 ] ASF GitHub Bot logged work on LANG-1406: Author: ASF GitHub Bot

[jira] [Work logged] (LANG-1406) StringIndexOutOfBoundsException in StringUtils.replaceIgnoreCase()

2020-05-31 Thread ASF GitHub Bot (Jira)
[ https://issues.apache.org/jira/browse/LANG-1406?focusedWorklogId=439355&page=com.atlassian.jira.plugin.system.issuetabpanels:worklog-tabpanel#worklog-439355 ] ASF GitHub Bot logged work on LANG-1406: Author: ASF GitHub Bot

[GitHub] [commons-lang] kinow commented on pull request #340: [LANG-1406] StringIndexOutOfBoundsException in StringUtils.replaceIgnoreCase

2020-05-31 Thread GitBox
kinow commented on pull request #340: URL: https://github.com/apache/commons-lang/pull/340#issuecomment-636642377 Oh, actually the issue has been closed in #423! Closing the PR, but thanks for your contribution @HiuKwok ! T

[GitHub] [commons-lang] kinow commented on pull request #381: LANG-1406 StringIndexOutOfBoundsException in StringUtils.replaceIgnor…

2020-05-31 Thread GitBox
kinow commented on pull request #381: URL: https://github.com/apache/commons-lang/pull/381#issuecomment-636642239 Oh, actually the issue has been closed in #423! Closing the PR, but thanks for your contribution @drajakumar !

[GitHub] [commons-lang] kinow closed pull request #340: [LANG-1406] StringIndexOutOfBoundsException in StringUtils.replaceIgnoreCase

2020-05-31 Thread GitBox
kinow closed pull request #340: URL: https://github.com/apache/commons-lang/pull/340 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

[GitHub] [commons-lang] kinow closed pull request #381: LANG-1406 StringIndexOutOfBoundsException in StringUtils.replaceIgnor…

2020-05-31 Thread GitBox
kinow closed pull request #381: URL: https://github.com/apache/commons-lang/pull/381 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

[jira] [Commented] (LANG-1326) StringUtils#isAllEmpty and #isAllBlank should return false for an empty array

2020-05-31 Thread Bruno P. Kinoshita (Jira)
[ https://issues.apache.org/jira/browse/LANG-1326?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=17120773#comment-17120773 ] Bruno P. Kinoshita commented on LANG-1326: -- Oh, that is a good point I think. [~p

[jira] [Work logged] (LANG-1490) Create a more generic/string representation of a DiffResult and a Diff

2020-05-31 Thread ASF GitHub Bot (Jira)
[ https://issues.apache.org/jira/browse/LANG-1490?focusedWorklogId=439349&page=com.atlassian.jira.plugin.system.issuetabpanels:worklog-tabpanel#worklog-439349 ] ASF GitHub Bot logged work on LANG-1490: Author: ASF GitHub Bot

[GitHub] [commons-lang] kinow commented on a change in pull request #457: WIP [LANG-1490] - Creates DiffResultView and DiffView classes

2020-05-31 Thread GitBox
kinow commented on a change in pull request #457: URL: https://github.com/apache/commons-lang/pull/457#discussion_r433050850 ## File path: src/main/java/org/apache/commons/lang3/builder/DiffView.java ## @@ -0,0 +1,64 @@ +package org.apache.commons.lang3.builder; Review comment

[GitHub] [commons-lang] kinow commented on pull request #530: LANG-1542: ToStringBuilder.reflectionToString - Wrong JSON format for List

2020-05-31 Thread GitBox
kinow commented on pull request #530: URL: https://github.com/apache/commons-lang/pull/530#issuecomment-636621642 BTW, @TranNgocKhoa if you could rebase it, the build should pass. There were checkstyle issues on master until recently. --

[jira] [Work logged] (LANG-1542) ToStringBuilder.reflectionToString - Wrong JSON format when object has a List of Enum

2020-05-31 Thread ASF GitHub Bot (Jira)
[ https://issues.apache.org/jira/browse/LANG-1542?focusedWorklogId=439345&page=com.atlassian.jira.plugin.system.issuetabpanels:worklog-tabpanel#worklog-439345 ] ASF GitHub Bot logged work on LANG-1542: Author: ASF GitHub Bot

[jira] [Work logged] (LANG-1542) ToStringBuilder.reflectionToString - Wrong JSON format when object has a List of Enum

2020-05-31 Thread ASF GitHub Bot (Jira)
[ https://issues.apache.org/jira/browse/LANG-1542?focusedWorklogId=439344&page=com.atlassian.jira.plugin.system.issuetabpanels:worklog-tabpanel#worklog-439344 ] ASF GitHub Bot logged work on LANG-1542: Author: ASF GitHub Bot

[GitHub] [commons-lang] kinow commented on pull request #530: LANG-1542: ToStringBuilder.reflectionToString - Wrong JSON format for List

2020-05-31 Thread GitBox
kinow commented on pull request #530: URL: https://github.com/apache/commons-lang/pull/530#issuecomment-636620127 Will leave it to @garydgregory to do the final review as he had requested changes before :+1: This is an auto

[jira] [Updated] (COMPRESS-529) Tar decompression fails with runtime exceptions

2020-05-31 Thread Peter Lee (Jira)
[ https://issues.apache.org/jira/browse/COMPRESS-529?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ] Peter Lee updated COMPRESS-529: --- Assignee: Peter Lee > Tar decompression fails with runtime exceptions > ---

[jira] [Commented] (GEOMETRY-95) CSG Examples

2020-05-31 Thread Matt Juntunen (Jira)
[ https://issues.apache.org/jira/browse/GEOMETRY-95?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=17120702#comment-17120702 ] Matt Juntunen commented on GEOMETRY-95: --- bq. Did you remove the "uv-sphere" implem

[jira] [Work logged] (LANG-1548) split regionMatches for better performance

2020-05-31 Thread ASF GitHub Bot (Jira)
[ https://issues.apache.org/jira/browse/LANG-1548?focusedWorklogId=439297&page=com.atlassian.jira.plugin.system.issuetabpanels:worklog-tabpanel#worklog-439297 ] ASF GitHub Bot logged work on LANG-1548: Author: ASF GitHub Bot

[jira] [Work logged] (LANG-1548) split regionMatches for better performance

2020-05-31 Thread ASF GitHub Bot (Jira)
[ https://issues.apache.org/jira/browse/LANG-1548?focusedWorklogId=439298&page=com.atlassian.jira.plugin.system.issuetabpanels:worklog-tabpanel#worklog-439298 ] ASF GitHub Bot logged work on LANG-1548: Author: ASF GitHub Bot

[GitHub] [commons-lang] XenoAmess edited a comment on pull request #534: [LANG-1548] split regionMatches for better performance

2020-05-31 Thread GitBox
XenoAmess edited a comment on pull request #534: URL: https://github.com/apache/commons-lang/pull/534#issuecomment-636539583 also want to split StringUtils.startswith, endwith, prependIfMissing... now the call flow is like this: 1. user call startsWith(final CharSequence str, final Cha

[GitHub] [commons-lang] XenoAmess commented on pull request #534: [LANG-1548] split regionMatches for better performance

2020-05-31 Thread GitBox
XenoAmess commented on pull request #534: URL: https://github.com/apache/commons-lang/pull/534#issuecomment-636539583 also want to split StringUtils.startswith, endwith, prependIfMissing... now the call flow is like this: 1. user call startsWith(final CharSequence str, final CharSequen

[jira] [Work logged] (LANG-1549) implement TODO in CharSequenceUtils.lastIndexOf : remake it.

2020-05-31 Thread ASF GitHub Bot (Jira)
[ https://issues.apache.org/jira/browse/LANG-1549?focusedWorklogId=439296&page=com.atlassian.jira.plugin.system.issuetabpanels:worklog-tabpanel#worklog-439296 ] ASF GitHub Bot logged work on LANG-1549: Author: ASF GitHub Bot

[GitHub] [commons-lang] coveralls commented on pull request #535: [LANG-1549] implement TODO in CharSequenceUtils.lastIndexOf : remake it.

2020-05-31 Thread GitBox
coveralls commented on pull request #535: URL: https://github.com/apache/commons-lang/pull/535#issuecomment-636537530 [![Coverage Status](https://coveralls.io/builds/31147261/badge)](https://coveralls.io/builds/31147261) Coverage decreased (-0.03%) to 95.01% when pulling **2def8

[jira] [Work logged] (LANG-1548) split regionMatches for better performance

2020-05-31 Thread ASF GitHub Bot (Jira)
[ https://issues.apache.org/jira/browse/LANG-1548?focusedWorklogId=439292&page=com.atlassian.jira.plugin.system.issuetabpanels:worklog-tabpanel#worklog-439292 ] ASF GitHub Bot logged work on LANG-1548: Author: ASF GitHub Bot

[GitHub] [commons-lang] coveralls commented on pull request #534: [LANG-1548] split regionMatches for better performance

2020-05-31 Thread GitBox
coveralls commented on pull request #534: URL: https://github.com/apache/commons-lang/pull/534#issuecomment-636531146 [![Coverage Status](https://coveralls.io/builds/31146797/badge)](https://coveralls.io/builds/31146797) Coverage decreased (-0.008%) to 95.033% when pulling **52e

[GitHub] [commons-beanutils] garydgregory commented on pull request #27: [BEANUTILS-539] use ConcurrentHashMap insteadof WeakFastHashMap

2020-05-31 Thread GitBox
garydgregory commented on pull request #27: URL: https://github.com/apache/commons-beanutils/pull/27#issuecomment-636531069 Just wait. I have a query on the legal mailing list to check to see if we can reuse the Infinispan version of its concurrent weak key hash map, and since it is co-aut

[GitHub] [commons-lang] garydgregory commented on pull request #532: reformat Locks and LocksTest for passing Checkstyle

2020-05-31 Thread GitBox
garydgregory commented on pull request #532: URL: https://github.com/apache/commons-lang/pull/532#issuecomment-636530486 @jochenw glad it worked for you. This is an automated message from the Apache Git Service. To respond to

[jira] [Work logged] (LANG-1547) fix code smells; fix typos; performance refine(trying to)

2020-05-31 Thread ASF GitHub Bot (Jira)
[ https://issues.apache.org/jira/browse/LANG-1547?focusedWorklogId=439290&page=com.atlassian.jira.plugin.system.issuetabpanels:worklog-tabpanel#worklog-439290 ] ASF GitHub Bot logged work on LANG-1547: Author: ASF GitHub Bot

[GitHub] [commons-lang] coveralls commented on pull request #533: [LANG-1547] fix code smells; fix typos

2020-05-31 Thread GitBox
coveralls commented on pull request #533: URL: https://github.com/apache/commons-lang/pull/533#issuecomment-636529000 [![Coverage Status](https://coveralls.io/builds/31146722/badge)](https://coveralls.io/builds/31146722) Coverage decreased (-0.004%) to 95.036% when pulling **62f

[GitHub] [commons-beanutils] XenoAmess commented on pull request #27: [BEANUTILS-539] use ConcurrentHashMap insteadof WeakFastHashMap

2020-05-31 Thread GitBox
XenoAmess commented on pull request #27: URL: https://github.com/apache/commons-beanutils/pull/27#issuecomment-636524987 my bad. I found the reason why it can't pass, it is my mistake. Now we can use `org.apache.commons.logging.impl.WeakHashtable ` to replace `FastWeakHashMap`. We

[GitHub] [commons-lang] asfgit closed pull request #532: reformat Locks and LocksTest for passing Checkstyle

2020-05-31 Thread GitBox
asfgit closed pull request #532: URL: https://github.com/apache/commons-lang/pull/532 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 t

[GitHub] [commons-lang] jochenw commented on pull request #532: reformat Locks and LocksTest for passing Checkstyle

2020-05-31 Thread GitBox
jochenw commented on pull request #532: URL: https://github.com/apache/commons-lang/pull/532#issuecomment-636522760 Thanks, Gary, that helped. One more thing, though: I do not feel responsible for this one: [INFO] There is 1 error reported by Checkstyle 8.32 with src/site/reso

[jira] [Commented] (GEOMETRY-95) CSG Examples

2020-05-31 Thread Gilles Sadowski (Jira)
[ https://issues.apache.org/jira/browse/GEOMETRY-95?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=17120633#comment-17120633 ] Gilles Sadowski commented on GEOMETRY-95: - bq. "icosphere" Did you remove the "

[jira] [Work logged] (LANG-1547) fix code smells; fix typos; performance refine(trying to)

2020-05-31 Thread ASF GitHub Bot (Jira)
[ https://issues.apache.org/jira/browse/LANG-1547?focusedWorklogId=439281&page=com.atlassian.jira.plugin.system.issuetabpanels:worklog-tabpanel#worklog-439281 ] ASF GitHub Bot logged work on LANG-1547: Author: ASF GitHub Bot

[GitHub] [commons-lang] XenoAmess commented on pull request #533: [LANG-1547] fix code smells; fix typos

2020-05-31 Thread GitBox
XenoAmess commented on pull request #533: URL: https://github.com/apache/commons-lang/pull/533#issuecomment-636512322 the performance refines I said has being splited to [LANG-1548] and [LANG-1549] This is an automated messa

[jira] [Created] (LANG-1549) implement TODO in CharSequenceUtils.lastIndexOf : remake it.

2020-05-31 Thread JIN XU (Jira)
JIN XU created LANG-1549: Summary: implement TODO in CharSequenceUtils.lastIndexOf : remake it. Key: LANG-1549 URL: https://issues.apache.org/jira/browse/LANG-1549 Project: Commons Lang Issue Type:

[jira] [Created] (LANG-1548) split regionMatches for better performance

2020-05-31 Thread JIN XU (Jira)
JIN XU created LANG-1548: Summary: split regionMatches for better performance Key: LANG-1548 URL: https://issues.apache.org/jira/browse/LANG-1548 Project: Commons Lang Issue Type: Improvement

[GitHub] [commons-lang] XenoAmess opened a new pull request #535: implement TODO in CharSequenceUtils.lastIndexOf : remake it.

2020-05-31 Thread GitBox
XenoAmess opened a new pull request #535: URL: https://github.com/apache/commons-lang/pull/535 Hi. If this change is accectable then I will start CharSequenceUtils.indexOf using same way. Though, need there be more tests about this function? -

[GitHub] [commons-codec] coveralls edited a comment on pull request #46: Base16 Input and Output Streams

2020-05-31 Thread GitBox
coveralls edited a comment on pull request #46: URL: https://github.com/apache/commons-codec/pull/46#issuecomment-616146181 [![Coverage Status](https://coveralls.io/builds/31145020/badge)](https://coveralls.io/builds/31145020) Coverage increased (+0.1%) to 93.982% when pulling *

[GitHub] [commons-codec] XenoAmess commented on pull request #48: fix travis-ci scripts

2020-05-31 Thread GitBox
XenoAmess commented on pull request #48: URL: https://github.com/apache/commons-codec/pull/48#issuecomment-636498741 > I wonder if we need both openjdk11 and oraclejdk11 ? I thought it must be a reason to contain both in original travis-ci scripts, so I didn't remove any of them. -

[GitHub] [commons-codec] XenoAmess edited a comment on pull request #48: fix travis-ci scripts

2020-05-31 Thread GitBox
XenoAmess edited a comment on pull request #48: URL: https://github.com/apache/commons-codec/pull/48#issuecomment-636498741 > I wonder if we need both openjdk11 and oraclejdk11 ? I thought there must be a reason to contain both in original travis-ci scripts, so I didn't remove any of

[jira] [Work logged] (LANG-1542) ToStringBuilder.reflectionToString - Wrong JSON format when object has a List of Enum

2020-05-31 Thread ASF GitHub Bot (Jira)
[ https://issues.apache.org/jira/browse/LANG-1542?focusedWorklogId=439260&page=com.atlassian.jira.plugin.system.issuetabpanels:worklog-tabpanel#worklog-439260 ] ASF GitHub Bot logged work on LANG-1542: Author: ASF GitHub Bot

[GitHub] [commons-lang] coveralls edited a comment on pull request #530: LANG-1542: ToStringBuilder.reflectionToString - Wrong JSON format for List

2020-05-31 Thread GitBox
coveralls edited a comment on pull request #530: URL: https://github.com/apache/commons-lang/pull/530#issuecomment-629603951 [![Coverage Status](https://coveralls.io/builds/31144338/badge)](https://coveralls.io/builds/31144338) Coverage increased (+0.0009%) to 95.092% when pullin

[jira] [Commented] (GEOMETRY-95) CSG Examples

2020-05-31 Thread Matt Juntunen (Jira)
[ https://issues.apache.org/jira/browse/GEOMETRY-95?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=17120577#comment-17120577 ] Matt Juntunen commented on GEOMETRY-95: --- I just added a PR containing the update {

[GitHub] [commons-geometry] darkma773r opened a new pull request #78: GEOMETRY-95: Improving Sphere.toTree()

2020-05-31 Thread GitBox
darkma773r opened a new pull request #78: URL: https://github.com/apache/commons-geometry/pull/78 - Improving Sphere.toTree() - Removing use of HyperplaneSubset.Builder interface to improve performance and numerical robustness - Other updates to improve robustness with large BSP trees

[GitHub] [commons-lang] garydgregory commented on pull request #532: reformat Locks and LocksTest for passing Checkstyle

2020-05-31 Thread GitBox
garydgregory commented on pull request #532: URL: https://github.com/apache/commons-lang/pull/532#issuecomment-636481961 Do this: ``` git config core.autocrlf input git checkout release git checkout master mvn checkstyle:check ``` -

[GitHub] [commons-beanutils] XenoAmess edited a comment on pull request #27: [BEANUTILS-539] use ConcurrentHashMap insteadof WeakFastHashMap

2020-05-31 Thread GitBox
XenoAmess edited a comment on pull request #27: URL: https://github.com/apache/commons-beanutils/pull/27#issuecomment-636474232 also here is some guy contributed a concurrent weak hash map too : https://issues.apache.org/jira/browse/HARMONY-6434?jql=text%20~%20%22concurrent%20weak%20hash%20

[jira] [Updated] (COLLECTIONS-700) Add a ConcurrentWeakHashMap

2020-05-31 Thread Gary D. Gregory (Jira)
[ https://issues.apache.org/jira/browse/COLLECTIONS-700?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ] Gary D. Gregory updated COLLECTIONS-700: Assignee: Gary D. Gregory > Add a ConcurrentWeakHashMap >

[GitHub] [commons-beanutils] XenoAmess commented on pull request #27: [BEANUTILS-539] use ConcurrentHashMap insteadof WeakFastHashMap

2020-05-31 Thread GitBox
XenoAmess commented on pull request #27: URL: https://github.com/apache/commons-beanutils/pull/27#issuecomment-636474232 also here is some guy contributed a concurrent weak hash map too : https://issues.apache.org/jira/browse/HARMONY-6434?jql=text%20~%20%22concurrent%20weak%20hash%20map%22

[GitHub] [commons-beanutils] XenoAmess edited a comment on pull request #27: [BEANUTILS-539] use ConcurrentHashMap insteadof WeakFastHashMap

2020-05-31 Thread GitBox
XenoAmess edited a comment on pull request #27: URL: https://github.com/apache/commons-beanutils/pull/27#issuecomment-636470709 > IIRC I think someone contributed a concurrent weak hash map either in this component or in commons-collections, there should be a JIRA ticket... AFK ATM...

[GitHub] [commons-beanutils] XenoAmess commented on pull request #27: [BEANUTILS-539] use ConcurrentHashMap insteadof WeakFastHashMap

2020-05-31 Thread GitBox
XenoAmess commented on pull request #27: URL: https://github.com/apache/commons-beanutils/pull/27#issuecomment-636470709 > IIRC I think someone contributed a concurrent weak hash map either in this component or in commons-collections, there should be a JIRA ticket... AFK ATM... well

[GitHub] [commons-beanutils] garydgregory commented on pull request #27: [BEANUTILS-539] use ConcurrentHashMap insteadof WeakFastHashMap

2020-05-31 Thread GitBox
garydgregory commented on pull request #27: URL: https://github.com/apache/commons-beanutils/pull/27#issuecomment-636466018 IIRC I think someone contributed a concurrent weak hash map either in this component or in commons-collections, there should be a JIRA ticket... AFK ATM... -

[GitHub] [commons-beanutils] XenoAmess edited a comment on pull request #27: [BEANUTILS-539] use ConcurrentHashMap insteadof WeakFastHashMap

2020-05-31 Thread GitBox
XenoAmess edited a comment on pull request #27: URL: https://github.com/apache/commons-beanutils/pull/27#issuecomment-636465013 > If your app does not need synchronization, you can use the map in "fast" mode. @garydgregory Thanks. usually we need synchronization, and even when I

[GitHub] [commons-beanutils] XenoAmess edited a comment on pull request #27: [BEANUTILS-539] use ConcurrentHashMap insteadof WeakFastHashMap

2020-05-31 Thread GitBox
XenoAmess edited a comment on pull request #27: URL: https://github.com/apache/commons-beanutils/pull/27#issuecomment-636465013 > If your app does not need synchronization, you can use the map in "fast" mode. @garydgregory Thanks. usually we need synchronization, and even when I

[GitHub] [commons-beanutils] XenoAmess commented on pull request #27: [BEANUTILS-539] use ConcurrentHashMap insteadof WeakFastHashMap

2020-05-31 Thread GitBox
XenoAmess commented on pull request #27: URL: https://github.com/apache/commons-beanutils/pull/27#issuecomment-636465013 > If your app does not need synchronization, you can use the map in "fast" mode. Thanks. usually we need synchronization, and even when I need weak reference I wi

[GitHub] [commons-beanutils] XenoAmess commented on pull request #27: [BEANUTILS-539] use ConcurrentHashMap insteadof WeakFastHashMap

2020-05-31 Thread GitBox
XenoAmess commented on pull request #27: URL: https://github.com/apache/commons-beanutils/pull/27#issuecomment-636464477 @garydgregory is org.apache.commons.logging.impl.WeakHashtable a better replacement of WeakFastHashMap in this cases? --

[GitHub] [commons-beanutils] garydgregory commented on pull request #27: [BEANUTILS-539] use ConcurrentHashMap insteadof WeakFastHashMap

2020-05-31 Thread GitBox
garydgregory commented on pull request #27: URL: https://github.com/apache/commons-beanutils/pull/27#issuecomment-636463974 If your app does not need synchronization, you can use the map in "fast" mode. This is an automated

[jira] [Resolved] (IO-669) fix code smells; fix typos

2020-05-31 Thread Gary D. Gregory (Jira)
[ https://issues.apache.org/jira/browse/IO-669?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ] Gary D. Gregory resolved IO-669. Fix Version/s: 2.7.1 Resolution: Fixed Please review and close.   > fix code smells; fix typos

[jira] [Work logged] (IO-669) fix code smells; fix typos

2020-05-31 Thread ASF GitHub Bot (Jira)
[ https://issues.apache.org/jira/browse/IO-669?focusedWorklogId=439224&page=com.atlassian.jira.plugin.system.issuetabpanels:worklog-tabpanel#worklog-439224 ] ASF GitHub Bot logged work on IO-669: - Author: ASF GitHub Bot Cre

[GitHub] [commons-io] garydgregory merged pull request #115: [IO-669] Fix code smells; fix typos

2020-05-31 Thread GitBox
garydgregory merged pull request #115: URL: https://github.com/apache/commons-io/pull/115 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

[GitHub] [commons-beanutils] XenoAmess closed pull request #27: [BEANUTILS-539] use ConcurrentHashMap insteadof WeakFastHashMap

2020-05-31 Thread GitBox
XenoAmess closed pull request #27: URL: https://github.com/apache/commons-beanutils/pull/27 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 t

[GitHub] [commons-beanutils] XenoAmess commented on pull request #27: [BEANUTILS-539] use ConcurrentHashMap insteadof WeakFastHashMap

2020-05-31 Thread GitBox
XenoAmess commented on pull request #27: URL: https://github.com/apache/commons-beanutils/pull/27#issuecomment-636462475 > -1. Do you understand the difference between a weak and strong reference in a map? Do you you know what will happen if you install this change in an application that k

[jira] [Resolved] (COMPRESS-532) Decompression fails with IllegalArgumentException(BinaryTree.java:72)

2020-05-31 Thread Stefan Bodewig (Jira)
[ https://issues.apache.org/jira/browse/COMPRESS-532?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ] Stefan Bodewig resolved COMPRESS-532. - Fix Version/s: 1.21 Resolution: Fixed fixed with commit 6bbc7cb1 > Decompressio

[jira] [Resolved] (COMPRESS-531) Tar decompression fails with NullPointerException

2020-05-31 Thread Stefan Bodewig (Jira)
[ https://issues.apache.org/jira/browse/COMPRESS-531?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ] Stefan Bodewig resolved COMPRESS-531. - Fix Version/s: 1.21 Resolution: Fixed > Tar decompression fails with NullPointer

[jira] [Commented] (COMPRESS-531) Tar decompression fails with NullPointerException

2020-05-31 Thread Stefan Bodewig (Jira)
[ https://issues.apache.org/jira/browse/COMPRESS-531?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=17120514#comment-17120514 ] Stefan Bodewig commented on COMPRESS-531: - Thanks to the fix of COMPRESS-530 th

[GitHub] [commons-beanutils] melloware commented on pull request #27: [BEANUTILS-539] use ConcurrentHashMap insteadof WeakFastHashMap

2020-05-31 Thread GitBox
melloware commented on pull request #27: URL: https://github.com/apache/commons-beanutils/pull/27#issuecomment-636458210 @garydgregory I am having a hard time resyncing my fork all the sudden. I am getting this error "! refs/heads/master:refs/heads/master [remote rejected] (refusing to all

[jira] [Commented] (COMPRESS-528) Tar decompression - Inconsistent entry size

2020-05-31 Thread Stefan Bodewig (Jira)
[ https://issues.apache.org/jira/browse/COMPRESS-528?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=17120508#comment-17120508 ] Stefan Bodewig commented on COMPRESS-528: - I'm not sure your expectation is cor

[GitHub] [commons-beanutils] garydgregory commented on pull request #27: [BEANUTILS-539] use ConcurrentHashMap insteadof WeakFastHashMap

2020-05-31 Thread GitBox
garydgregory commented on pull request #27: URL: https://github.com/apache/commons-beanutils/pull/27#issuecomment-636457473 -1. Do you understand the difference between a weak and strong reference in a map? Do you you know what will happen if you install this change in an application that

[GitHub] [commons-beanutils] XenoAmess commented on pull request #27: [BEANUTILS-539] use ConcurrentHashMap insteadof WeakFastHashMap

2020-05-31 Thread GitBox
XenoAmess commented on pull request #27: URL: https://github.com/apache/commons-beanutils/pull/27#issuecomment-636457266 > Looks like you have some merge conflicts? Sorry for that. will fix it immediately. > Also I think this will resolve BEANUTILS-509: #21 I had no expe

[GitHub] [commons-beanutils] XenoAmess commented on pull request #27: [BEANUTILS-539] use ConcurrentHashMap insteadof WeakFastHashMap

2020-05-31 Thread GitBox
XenoAmess commented on pull request #27: URL: https://github.com/apache/commons-beanutils/pull/27#issuecomment-636457321 detailed output of Jprofile is attached at link : https://issues.apache.org/jira/browse/BEANUTILS-539 T

[jira] [Resolved] (COMPRESS-530) Tar decompression fails with NegativeArraySizeException

2020-05-31 Thread Stefan Bodewig (Jira)
[ https://issues.apache.org/jira/browse/COMPRESS-530?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ] Stefan Bodewig resolved COMPRESS-530. - Fix Version/s: 1.21 Resolution: Fixed > Tar decompression fails with NegativeArr

[jira] [Commented] (COMPRESS-529) Tar decompression fails with runtime exceptions

2020-05-31 Thread Stefan Bodewig (Jira)
[ https://issues.apache.org/jira/browse/COMPRESS-529?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=17120506#comment-17120506 ] Stefan Bodewig commented on COMPRESS-529: - At least the two of us share the sam

[GitHub] [commons-beanutils] melloware commented on pull request #23: Formatted source code using Apache Commons standards.

2020-05-31 Thread GitBox
melloware commented on pull request #23: URL: https://github.com/apache/commons-beanutils/pull/23#issuecomment-636456327 Closing this PR. Will create a new one This is an automated message from the Apache Git Service. To res

[GitHub] [commons-beanutils] melloware closed pull request #23: Formatted source code using Apache Commons standards.

2020-05-31 Thread GitBox
melloware closed pull request #23: URL: https://github.com/apache/commons-beanutils/pull/23 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 t

[GitHub] [commons-beanutils] melloware commented on pull request #27: [BEANUTILS-539] use ConcurrentHashMap insteadof WeakFastHashMap

2020-05-31 Thread GitBox
melloware commented on pull request #27: URL: https://github.com/apache/commons-beanutils/pull/27#issuecomment-636456188 Looks like you have some merge conflicts? Also I think this will resolve BEANUTILS-509: https://github.com/apache/commons-beanutils/pull/21 -

[GitHub] [commons-lang] jochenw commented on pull request #532: reformat Locks and LocksTest for passing Checkstyle

2020-05-31 Thread GitBox
jochenw commented on pull request #532: URL: https://github.com/apache/commons-lang/pull/532#issuecomment-636442299 Hi, could someone please advise me, what I am doing wring here? Trying to pull this in: git clone https://gitbox.apache.org/repos/asf/commons-lang.git

[GitHub] [commons-lang] XenoAmess commented on pull request #532: reformat Locks and LocksTest for passing Checkstyle

2020-05-31 Thread GitBox
XenoAmess commented on pull request #532: URL: https://github.com/apache/commons-lang/pull/532#issuecomment-636434614 > > > I got a build error when update my pull request today. What should I do? > > > Link: #530 > > > > > > your build passed, but check style failed. > > a

[GitHub] [commons-lang] XenoAmess opened a new pull request #534: split regionMatches for better performance

2020-05-31 Thread GitBox
XenoAmess opened a new pull request #534: URL: https://github.com/apache/commons-lang/pull/534 This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL abov

[GitHub] [commons-lang] TranNgocKhoa commented on pull request #532: reformat Locks and LocksTest for passing Checkstyle

2020-05-31 Thread GitBox
TranNgocKhoa commented on pull request #532: URL: https://github.com/apache/commons-lang/pull/532#issuecomment-636433876 > > I got a build error when update my pull request today. What should I do? > > Link: #530 > > your build passed, but check style failed. > and check style f

[GitHub] [commons-beanutils] XenoAmess commented on pull request #27: [BEANUTILS-539] use ConcurrentHashMap insteadof WeakFastHashMap

2020-05-31 Thread GitBox
XenoAmess commented on pull request #27: URL: https://github.com/apache/commons-beanutils/pull/27#issuecomment-636433256 feel free to ask for changes about this test, I will help to run it. Then we decide whether change usage of WeakHashMap to CuncurrentHashMap, or remain what it looks l

[jira] [Updated] (BEANUTILS-539) use ConcurrentHashMap insteadof WeakFastHashMap

2020-05-31 Thread JIN XU (Jira)
[ https://issues.apache.org/jira/browse/BEANUTILS-539?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ] JIN XU updated BEANUTILS-539: - Description: Hi. throughtout my performance test (using Jprofiler), I found out WeakHashMap is far s

[GitHub] [commons-beanutils] XenoAmess opened a new pull request #27: [BEANUTILS-539] use ConcurrentHashMap insteadof WeakFastHashMap

2020-05-31 Thread GitBox
XenoAmess opened a new pull request #27: URL: https://github.com/apache/commons-beanutils/pull/27 throughtout my performance test (using Jprofiler), I found out WeakHashMap is far slower than ConcurrentHashMap. add tests to show how slow it is. please run it using Jprofiler or other t

[jira] [Updated] (BEANUTILS-539) use ConcurrentHashMap insteadof WeakFastHashMap

2020-05-31 Thread JIN XU (Jira)
[ https://issues.apache.org/jira/browse/BEANUTILS-539?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ] JIN XU updated BEANUTILS-539: - Description: Hi. throughtout my performance test (using Jprofiler), I found out WeakHashMap is far s

[GitHub] [commons-lang] XenoAmess commented on pull request #532: reformat Locks and LocksTest for passing Checkstyle

2020-05-31 Thread GitBox
XenoAmess commented on pull request #532: URL: https://github.com/apache/commons-lang/pull/532#issuecomment-636432421 > I got a build error when update my pull request today. What should I do? > Link: #530 your build passed, but check style failed. and check style failed because

[jira] [Commented] (EMAIL-193) use ConcurrentHashMap insteadof WeakHashMap

2020-05-31 Thread JIN XU (Jira)
[ https://issues.apache.org/jira/browse/EMAIL-193?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=17120440#comment-17120440 ] JIN XU commented on EMAIL-193: -- Sorry this is a misoperation. this bug shall be commited to

[jira] [Created] (BEANUTILS-539) use ConcurrentHashMap insteadof WeakFastHashMap

2020-05-31 Thread JIN XU (Jira)
JIN XU created BEANUTILS-539: Summary: use ConcurrentHashMap insteadof WeakFastHashMap Key: BEANUTILS-539 URL: https://issues.apache.org/jira/browse/BEANUTILS-539 Project: Commons BeanUtils Issue

[jira] [Created] (EMAIL-193) use ConcurrentHashMap insteadof WeakHashMap

2020-05-31 Thread JIN XU (Jira)
JIN XU created EMAIL-193: Summary: use ConcurrentHashMap insteadof WeakHashMap Key: EMAIL-193 URL: https://issues.apache.org/jira/browse/EMAIL-193 Project: Commons Email Issue Type: Improvement

[jira] [Closed] (EMAIL-193) use ConcurrentHashMap insteadof WeakHashMap

2020-05-31 Thread JIN XU (Jira)
[ https://issues.apache.org/jira/browse/EMAIL-193?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ] JIN XU closed EMAIL-193. Resolution: Not A Problem > use ConcurrentHashMap insteadof WeakHashMap > --