[
https://issues.apache.org/jira/browse/STORM-1523?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=15146027#comment-15146027
]
ASF GitHub Bot commented on STORM-1523:
---------------------------------------
Github user redsanket commented on the pull request:
https://github.com/apache/storm/pull/1073#issuecomment-183687936
The original implementation would return -1 in any case if the port is not
bound yet. If we not changing the final outcome and unless we are concerned
about throwing an exception when the port is not bound, I guess it would be
fine. Moreover, removing if condition would keep trying to find the port
recursively until it finds it I presume. The original code only tries once as
per my understanding.
> Translate Available Ports task in util.java
> -------------------------------------------
>
> Key: STORM-1523
> URL: https://issues.apache.org/jira/browse/STORM-1523
> Project: Apache Storm
> Issue Type: Sub-task
> Components: storm-core
> Reporter: Sanket Reddy
>
--
This message was sent by Atlassian JIRA
(v6.3.4#6332)
