You are viewing a plain text version of this content. The canonical link for it is here.
Posted to issues@ozone.apache.org by "yandrey321 (via GitHub)" <gi...@apache.org> on 2026/08/04 16:26:12 UTC
[PR] HDDS-16043. Use EnumMap for ports in DatanodeDetails [ozone]
yandrey321 opened a new pull request, #10948:
URL: https://github.com/apache/ozone/pull/10948
## What changes were proposed in this pull request?
Use EnumMap for ports in DatanodeDetails
Previously DatanodeDetails stored its ports in a List<Port>, so every port lookup (getPort(Name), hasPort(Name)) was a linear scan. This PR replaces the backing store with an EnumMap<Port.Name, Port>, making keyed access O(1). Port.equals/hashCode are already name-based, so a name-keyed map is the natural fit.
## What is the link to the Apache JIRA
https://issues.apache.org/jira/browse/HDDS-16043
## How was this patch tested?
Unit and integration tests
--
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: issues-unsubscribe@ozone.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
---------------------------------------------------------------------
To unsubscribe, e-mail: issues-unsubscribe@ozone.apache.org
For additional commands, e-mail: issues-help@ozone.apache.org
Re: [PR] HDDS-16043. Use EnumMap for ports in DatanodeDetails [ozone]
Posted by "yandrey321 (via GitHub)" <gi...@apache.org>.
yandrey321 commented on PR #10948:
URL: https://github.com/apache/ozone/pull/10948#issuecomment-5192770565
@szetszwo could you please take a look?
--
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: issues-unsubscribe@ozone.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
---------------------------------------------------------------------
To unsubscribe, e-mail: issues-unsubscribe@ozone.apache.org
For additional commands, e-mail: issues-help@ozone.apache.org
Re: [PR] HDDS-16043. Use EnumMap for ports in DatanodeDetails [ozone]
Posted by "jojochuang (via GitHub)" <gi...@apache.org>.
jojochuang commented on PR #10948:
URL: https://github.com/apache/ozone/pull/10948#issuecomment-5199148609
Merged. 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: issues-unsubscribe@ozone.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
---------------------------------------------------------------------
To unsubscribe, e-mail: issues-unsubscribe@ozone.apache.org
For additional commands, e-mail: issues-help@ozone.apache.org
Re: [PR] HDDS-16043. Use EnumMap for ports in DatanodeDetails [ozone]
Posted by "jojochuang (via GitHub)" <gi...@apache.org>.
jojochuang merged PR #10948:
URL: https://github.com/apache/ozone/pull/10948
--
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: issues-unsubscribe@ozone.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
---------------------------------------------------------------------
To unsubscribe, e-mail: issues-unsubscribe@ozone.apache.org
For additional commands, e-mail: issues-help@ozone.apache.org