@himanshug I think we can modify current code:
```java
private int chooseNext(int start)
{
// up to unsigned short max (65535)
for (int i = start; i <= 0xFFFF; i++) {
if (!usedPorts.contains(i)) {
return i;
}
}
throw new ISE("All ports are Used..");
}
```
to like this:
```java
private int chooseNext(int start)
{
// up to endPort (which default value is 65535)
for (int i = start; i <= endPort; i++) {
if (!usedPorts.contains(i)) {
return i;
}
}
throw new ISE("All ports are Used..");
}
```
And `endPort` can be defined like this:
```java
@JsonProperty
@Min(1024)
@Max(65535)
private int endPort = 65535;
```
which is similar to current `startPort` definition：
```java
@JsonProperty
@Min(1024)
@Max(65535)
private int startPort = 8100;
```
In this way, I think users don't have to think about technicality of possible
port numbers.
[ Full content available at:
https://github.com/apache/incubator-druid/pull/6263 ]
This message was relayed via gitbox.apache.org for [email protected]
