youzan / nsqJavaSDK

nsq client for java
37 stars 22 forks source link

NSQConnection should be invalidated when it is closed in producer exception handler #2

Closed DoraALin closed 7 years ago

DoraALin commented 7 years ago

in 2.2, when there is an unexpected exception in publishing, NSQConenction will be closed, however that closed connection should NOT be returned to connection pool. That will fail following publishing in a high concurrent situation.

DoraALin commented 7 years ago

commit in https://github.com/youzan/nsqJavaSDK/commit/3f026bf846c573302b64c71bd0dd91b5dc2aaa4b