Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Connection modification #31

Closed
wants to merge 8 commits into from
Closed

Connection modification #31

wants to merge 8 commits into from

Conversation

aamadeo27
Copy link

Before, I was having a problem with the callback keep getting called (#11 (comment)) that was ocurring because the Connection.flusher was keep getting because its interval was never cleared, and that produced an invocation of the callback because the callback was associated with the socket.on('error').

I changed that and some other things.

amadeoa and others added 8 commits April 27, 2018 16:44
…tants.SESSION_EVENT.CONNECTION_ERROR). Close connection when socket.destroyed
Added reconnect capability to the connections.
Modified connect() results with result as
	{ connected: boolean, errors: [Error] }.
Added Authentication Error control.
Added Connection Erorr description.
Added Volttable Support.
Added adHoc and updateClasses sugar functions for Client.
Linted lib/* and test/*.
Modify test to adapt to new connections.
Modify test to use not localhost connection to voltdb.
Modify testrunner to run a specific test.
Added test to Volttable support.
Added test to Reconnect support.
Added tests to connections test. BadAuth. Reconnect. Mixed.
@aamadeo27 aamadeo27 closed this Aug 25, 2018
@aamadeo27
Copy link
Author

This pull request was made by changing the master branch. I will create a new one with all the changes in a different branch as it says in the CONTRIBUTING file.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant