nodece opened a new pull request, #4217:
URL: https://github.com/apache/bookkeeper/pull/4217

   ### Motivation
   
   - The PTR result will end with a `.`, we need to trim that, please see 
https://datatracker.ietf.org/doc/html/rfc1035#autoid-37
   - In testing, we use the loopback net interface, which doesn't need to do 
PTR parse: 
   
   ```
   java.lang.RuntimeException: java.net.UnknownHostException: Unable to resolve 
default hostname: 1.0.0.127.in-addr.arpa. for interface: lo0
   
    at 
org.apache.bookkeeper.proto.SimpleBookieServiceInfoProvider.<init>(SimpleBookieServiceInfoProvider.java:41)
    at 
org.apache.bookkeeper.bookie.TestBookieImpl.<init>(TestBookieImpl.java:71)
    at 
org.apache.bookkeeper.bookie.TestBookieImpl.<init>(TestBookieImpl.java:46)
    at 
org.apache.bookkeeper.bookie.BookieWriteToJournalTest.testForceLedger(BookieWriteToJournalTest.java:167)
   ```
   
   ### Changes
   
   - Trim the PTR result.
   - Skip parse PTR parsing of loopback address.
   


-- 
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]

Reply via email to