[
https://issues.apache.org/jira/browse/THRIFT-2390?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=13939207#comment-13939207
]
Hudson commented on THRIFT-2390:
--------------------------------
FAILURE: Integrated in Thrift #1084 (See
[https://builds.apache.org/job/Thrift/1084/])
THRIFT-2390: no close event when connection lost (jfarrell: rev
55cd520edfc2f808da08cb22f2faf53bf04de47c)
* lib/nodejs/lib/thrift/connection.js
> no close event when connection lost
> -----------------------------------
>
> Key: THRIFT-2390
> URL: https://issues.apache.org/jira/browse/THRIFT-2390
> Project: Thrift
> Issue Type: Bug
> Components: Node.js - Library
> Reporter: alexey
> Assignee: Jake Farrell
>
> thrift nodejs lib (master)
> There is no indication that connection lost if reconnect is not setup.
> (There is no "close" event from connection object or RPC error callback)
> File: connection.js:167
> Connection.prototype.connection_gone = function () {
> var self = this;
> // If a retry is already in progress, just let that happen
> if (this.retry_timer || !this.max_attempts) {
> return;
> }
> Should be changed to
> Connection.prototype.connection_gone = function () {
> var self = this;
> // If a retry is already in progress, just let that happen
> if (this.retry_timer) {
> return;
> }
> if(!this.max_attempts) {
> self.emit("close");
> return;
> }
--
This message was sent by Atlassian JIRA
(v6.2#6252)
