fix reconnect issue for nodejs
CHANGES:
- remove /lib/nodejs/lib/thrift/connection.js#L233-L236, it cause thrift cannot do reconnect
- add
forceClosevariable for indicatingcloseaction caseued by manual, and give up reconnecting
Anybody who could doublecheck this patch?
Anybody who could doublecheck this patch?
We have used this patch in production
This issue has been automatically marked as stale because it has not had recent activity. It will be closed in 7 days if no further activity occurs. Thank you for your contributions.
Generally looks good to me. Could you kindly rebase on latest master and push again?
This issue is no longer stale. Thank you for your contributions.
@emmenlau rebased on latest master and pushed