search

bdurand / seamless_database_pool

Add support for master/slave database clusters in ActiveRecord to improve performance.
http://rdoc.info/projects/bdurand/seamless_database_pool
MIT License
224 stars 58 forks source link

Making Seamless Database Pool work with Thinking Sphinx #9

Closed stijnster closed 1 year ago

stijnster commented 11 years ago

Hi there,

thanks for your wonderful gem! Up until yesterday we were struggling with a single mysql node and had a lot of headaches. But now we're running our backend processes on one node/server and the frontend on the other in a master/slave config. And it seems to work beautiful together.

One tiny issue that we had, is that we are using Thinking Sphinx to index our database for search. When I tried to integrate both TS and SDP, I ran into some problems.

It turns out TS searches for some configuration options on the database connection, and the type of datastore that is being used.

I forked your repo and made the necessary changes. The changes implemented choose some logical defaults.

If you have some time, would you mind reviewing (or maybe even consider merging) my changes?

Best regards,

Stijn

bdurand commented 11 years ago

I'm not sure what side effects there may be in reporting the underlying adapter class name for the class name of the seamless database pool adapter. I did just add support for Class.name to the latest (1.0.13) release which bases it on the underlying class name. Does that give you the compatibility you need?