@CrazyHZM, would you mind to take a look at my comment? I think the root cause 
is 

```java
            if ((flag & FLAG_EVENT) != 0) {
                res.setEvent(Response.HEARTBEAT_EVENT);
            }
```

once we change it to 

```java
            if ((flag & FLAG_EVENT) != 0) {
                res.setEvent(true);
            }
```

then we should not need the following logic any longer:

```java
if (!req.isBroken()) {
                if ((flag & FLAG_EVENT) != 0) {
                    if (data != null && data.equals(Request.READONLY_EVENT)) {
                        req.setEvent(Request.READONLY_EVENT);
                    } else {
                        req.setEvent(Request.HEARTBEAT_EVENT);
                    }
                }
                req.setData(data);
            } else {
                if ((flag & FLAG_EVENT) != 0) {
                    req.setEvent(true);
                }
            }
```

And, in your change, you consider request only, pls. take care response too.

[ Full content available at: 
https://github.com/apache/incubator-dubbo/pull/3043 ]
This message was relayed via gitbox.apache.org for 
[email protected]

---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]

Reply via email to