horizonzy opened a new pull request, #3557:
URL: https://github.com/apache/bookkeeper/pull/3557
Descriptions of the changes in this PR:
When we config as follow:
```
conf.setDisableServerSocketBind(true);
conf.setEnableLocalTransport(true);
```
https://github.com/apache/bookkeeper/blob/255416a8552f34e5fd50231bbeed77e0945b63e3/bookkeeper-server/src/main/java/org/apache/bookkeeper/proto/BookieNettyServer.java#L374-L428
At line_426, it use bookieId to bind, there is a flag about
`BookKeeperClusterTestCase#useUUIDasBookieId`, it set uuid to bookieId.
When the client connects to the server, the remoteAddress
`local:a4367111-5740-40dd-a8b2-0fcb2995398b` is strange, the resolve logic
throw java.lang.ArrayIndexOutOfBoundsException.
https://github.com/apache/bookkeeper/blob/255416a8552f34e5fd50231bbeed77e0945b63e3/bookkeeper-server/src/main/java/org/apache/bookkeeper/proto/PerChannelBookieClient.java#L1502-L1515
The master branch looks good, The pr #2723 remove logic.
```
// use a random BookieId
if (useUUIDasBookieId) {
conf.setBookieId(UUID.randomUUID().toString());
}
```
--
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: [email protected]
For queries about this service, please contact Infrastructure at:
[email protected]
