-
Notifications
You must be signed in to change notification settings - Fork 27
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
performance improvement: add redis Connection pool #2
Comments
+1 i'm agree with you - opening and closing connexions on redis it's a perf killer for both mysql & redis |
Well, as my opinion, I think MySQL2Redis is not a good choice to dump data from MySQL to Redis very frequently. And, the hot pot of performance may be not in the redis server. If we can prepare all the data firstly then dump them into redis with one or N (could be any number less than 1024) tcp connection from udp to redis, the performance could be improved a lot. I am busy not for my job, but I can try to fix it in my part-time, thanks for your suggestion. |
Hi, jackey I have implemented a similar project based on this project. See https://github.com/dawnbreaks/mysql2redis Thanks & Best Regards. |
hi dawnbreaks, thank you for your information. I am so glad that you do a better improvement for this project. May I have a little suggestion? Can you provide a function that the data in a MySQL server can dump into more than one redis server? The "More" may be 2 or 3, sometimes, we may have some backup of redis. but, people can sync data between redises with the mechanism of redis. |
hi, jackey Also you could consider abandon mysql and use redis as a main storage server. Best Regards! |
dawnbreaks, you are right, The solution of MySQL2Redis is odd, but it is really occurred. Why I created such a project? That is came from an odd requirement of one of my web site projects. We was unable to change our architecture at that time, so MySQL2Redis such a temp solution came. |
Hi, all
I think there is no need to make a tcp connection to redis server everytime you execute a udf function, a connection pool would be great for performance.
anybody could provide a patch for this enhancement? Or give me your opinions.
The text was updated successfully, but these errors were encountered: