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

修复多个连接下的问题 #1

Open
wants to merge 1 commit into
base: master
Choose a base branch
from
Open

Conversation

ahkimkoo
Copy link

@ahkimkoo ahkimkoo commented Jun 3, 2014

上一版本的代码不支持同时连接多个ssdb,原因是因为net连接的部分作为一个module的普通方法,被进程中的其他js共用了,因此申明多个连接时前面的net连接会被后面的覆盖。

解决:
参照redis nodejs client,将net连接及监听的代码独立出来,每次connect的时候new 一个实例,同时支持多个连接,直接在SSDB.js上修改,调用方式不变。

上一版本的代码不支持同时连接多个ssdb,原因是因为net连接的部分作为一个module的普通方法,被进程中的其他js共用了,因此申明多个连接时前面的net连接会被后面的覆盖。

解决:
参照redis nodejs client,将net连接及监听的代码独立出来,每次connect的时候new 一个实例,同时支持多个连接,直接在SSDB.js上修改,调用方式不变。
@ideawu
Copy link
Contributor

ideawu commented Jun 3, 2014

为什么整个文件都被改动了, 相当于完全重写? 真是这样吗? 如果是这样, 你可以创建一个新的 client.

@iflamed
Copy link

iflamed commented Jun 23, 2014

可能是windows ,unix 换行符得问题吧。

@ideawu
Copy link
Contributor

ideawu commented Jun 23, 2014

换一个文本编辑器, 重新修改一下吧. 这样的改动无法被合并.

@iflamed
Copy link

iflamed commented Jun 23, 2014

按照他得修改,我来提交一份看看。

@ideawu
Copy link
Contributor

ideawu commented Jun 24, 2014

Hi, 请问这个多连接问题是什么问题? 如何重现?

@ahkimkoo
Copy link
Author

连接几个ssdb,不关闭的情况下,操作,影响的是最后连接上的那个。其实代码里就比较明显,socket连接是模块级的变量,整个vm共用的,所以每次连接new一个socket。可能,需要同时连接多个ssdb的应用场景不会很多吧,我是一个爬虫的项目:
http://git.oschina.net/dreamidea/neocrawler
。因为以前使用redis,有多个库的应用,现在加入了ssdb的支持,在选择ssdb的时候就变成多主机了。
2014-6-24 上午9:08于 "ideawu" notifications@github.com写道:

Hi, 请问这个多连接问题是什么问题? 如何重现?


Reply to this email directly or view it on GitHub
#1 (comment).

@iflamed
Copy link

iflamed commented Jun 24, 2014

对,其实上面的说的是正确的。如果不想贡献的话,还可以用var ssdb = new SSDB.connect();的方法来新建新的实例。

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.

3 participants