Github user zentol commented on a diff in the pull request:

    https://github.com/apache/flink/pull/5062#discussion_r153797256
  
    --- Diff: 
flink-queryable-state/flink-queryable-state-client-java/src/main/java/org/apache/flink/queryablestate/network/AbstractServerBase.java
 ---
    @@ -251,34 +263,81 @@ private boolean attemptToBind(final int port) throws 
Throwable {
                        throw future.cause();
                } catch (BindException e) {
                        log.debug("Failed to start {} on port {}: {}.", 
serverName, port, e.getMessage());
    -                   shutdown();
    +                   try {
    +                           shutdownServer(Time.seconds(10L)).get();
    +                   } catch (Exception r) {
    +
    +                           // Here we were seeing this problem:
    +                           // https://github.com/netty/netty/issues/4357 
if we do a get().
    +                           // this is why we now simply wait a bit so that 
everything is
    +                           // shut down and then we check
    --- End diff --
    
    we don't check anything here.


---

Reply via email to