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 2022/10/12 09:02:47 UTC
[GitHub] [incubator-eventmesh] Alonexc opened a new issue, #1536: [Enhancement] Method uses integer based for loops to iterate over a List [Subscription]
Alonexc opened a new issue, #1536:
URL: https://github.com/apache/incubator-eventmesh/issues/1536
### Search before asking
- [X] I had searched in the [issues](https://github.com/apache/eventmesh/issues?q=is%3Aissue) and found no similar issues.
### Enhancement Request
located at: org/apache/eventmesh/common/protocol/grpc/protos/Subscription.java line 2874 and 2898
explanation:This method uses an integer-based for loop to iterate over a java.util.List, by calling List.get(i) each time through the loop. The integer is not used for other reasons. It is better to use an Iterator instead, as depending on List implementation, iterators can perform better, and they also allow for exchanging of other collection types without issue.
### Describe the solution you'd like
It is recommended to use iterators or foreach.
### Are you willing to submit PR?
- [ ] Yes I am willing to submit a 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.
To unsubscribe, e-mail: dev-unsubscribe@eventmesh.apache.org.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] karthiknatarajan-27558 commented on issue #1536: [Enhancement] Method uses integer based for loops to iterate over a List [Subscription]
Posted by GitBox <gi...@apache.org>.
karthiknatarajan-27558 commented on issue #1536:
URL: https://github.com/apache/incubator-eventmesh/issues/1536#issuecomment-1278485412
Hi Alonexc ,
I would like to work on this issue.
--
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] sclark268626 commented on issue #1536: [Enhancement] Method uses integer based for loops to iterate over a List [Subscription]
Posted by GitBox <gi...@apache.org>.
sclark268626 commented on issue #1536:
URL: https://github.com/apache/incubator-eventmesh/issues/1536#issuecomment-1279539792
Hello Alonexc,
May I work on this issue?
--
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] xwm1992 commented on issue #1536: [Enhancement] Method uses integer based for loops to iterate over a List [Subscription]
Posted by GitBox <gi...@apache.org>.
xwm1992 commented on issue #1536:
URL: https://github.com/apache/incubator-eventmesh/issues/1536#issuecomment-1278679161
This issue is for the first time contributor, issue #1538 has asigned to you.
--
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] qqeasonchen closed issue #1536: [Enhancement] Method uses integer based for loops to iterate over a List [Subscription]
Posted by GitBox <gi...@apache.org>.
qqeasonchen closed issue #1536: [Enhancement] Method uses integer based for loops to iterate over a List [Subscription]
URL: https://github.com/apache/incubator-eventmesh/issues/1536
--
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] sclark268626 commented on issue #1536: [Enhancement] Method uses integer based for loops to iterate over a List [Subscription]
Posted by GitBox <gi...@apache.org>.
sclark268626 commented on issue #1536:
URL: https://github.com/apache/incubator-eventmesh/issues/1536#issuecomment-1279828211
Thanks!
--
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