You are viewing a plain text version of this content. The canonical link for it is here.
Posted to dev@eventmesh.apache.org by GitBox <gi...@apache.org> on 2021/09/10 03:36:18 UTC

[GitHub] [incubator-eventmesh] DittoTool opened a new pull request #518: Other Cases of Inefficient Usage of JCF

DittoTool opened a new pull request #518:
URL: https://github.com/apache/incubator-eventmesh/pull/518


   Hi,
   
   This issue is supplementary to #517. We find that there are other inefficient usages of Java Collections:
   
   1. The contains method is invoked upon a list object in a loop. We recommend replacing it with a HashSet.
   Random access can occur at several LinkedList objects, which run in linear time complexity. We recommend replacing them with Arraylist objects.
   2. ArrayList is inserted before an iteration, while multiple memory reallocation might occur when the size of the list exceeds its capacity. We recommend replacing it with a LinkedList.
   3. LinkedList is not inefficient when we perform random access upon it. We recommend replacing it with an ArrayList.
   
   We discovered the above inefficient usage of containers by our tool Ditto. The patch is submitted. Could you please check and accept it? We have tested the patch on our PC. The patched program works well.
   
   Bests
   
   Ditto


-- 
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.

To unsubscribe, e-mail: dev-unsubscribe@eventmesh.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org



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

[GitHub] [incubator-eventmesh] DittoTool commented on pull request #518: Inefficient Usage of Java Collection

Posted by GitBox <gi...@apache.org>.
DittoTool commented on pull request #518:
URL: https://github.com/apache/incubator-eventmesh/pull/518#issuecomment-916735440


   > LGTM
   
   Thank you for your confirmation.


-- 
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.

To unsubscribe, e-mail: dev-unsubscribe@eventmesh.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org



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

[GitHub] [incubator-eventmesh] ruanwenjun commented on pull request #518: Inefficient Usage of Java Collection

Posted by GitBox <gi...@apache.org>.
ruanwenjun commented on pull request #518:
URL: https://github.com/apache/incubator-eventmesh/pull/518#issuecomment-1049403326


   @FastAtlas Hi, sorry to leave this pr too long and there are some conflicts, please solve this.


-- 
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.

To unsubscribe, e-mail: dev-unsubscribe@eventmesh.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org



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

[GitHub] [incubator-eventmesh] ruanwenjun commented on pull request #518: Inefficient Usage of Java Collection

Posted by GitBox <gi...@apache.org>.
ruanwenjun commented on pull request #518:
URL: https://github.com/apache/incubator-eventmesh/pull/518#issuecomment-1085469249


   Close due to no response.


-- 
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.

To unsubscribe, e-mail: dev-unsubscribe@eventmesh.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org



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

[GitHub] [incubator-eventmesh] ruanwenjun closed pull request #518: Inefficient Usage of Java Collection

Posted by GitBox <gi...@apache.org>.
ruanwenjun closed pull request #518:
URL: https://github.com/apache/incubator-eventmesh/pull/518


   


-- 
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.

To unsubscribe, e-mail: dev-unsubscribe@eventmesh.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org



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