Supporting multiple mongo backends

[dbusch]

I've modified the plugin so that it can support configuring multiple mongo backends. You can now annotate your model classes to declare in which backend the collection lives. The value given in the annotation matches up with a set of configuration settings in the application.conf file.

Issues:

1. Even with my testing, I'm not 100% certain I haven't broken some key assumptions about what the plugin is supposed to do.
2. I had to modify the public api of the plugin. The behavior and method signature of some public static methods were change so its very likely this version is not backwards compatible for some users
   • public static Datastore ds(String name) in MorphiaPlugin was rewritten
3. The plugin has support for using GridFS, mongo's blob storage mechanism. Due to how this feature was implemented, blob storage can only be supported on a single mongo backend even though multiple instances may be configured.

[greenlaw110]

The biggest concern is every requests will go through reflection to find the db name which could impact the performance.

[dbusch]

You're absolutely right I just don't know what to do about it other than to try to build up a map of classname -> db names

On Oct 20, 2011, at 4:48 PM, Green Luo wrote:

The biggest concern is every requests will go through reflection to find the db name which could impact the performance.

[greenlaw110]

Probably code enhancement could be a solution.

[dbusch]

I'll try to take a look into these two approaches next week.

On Oct 20, 2011, at 11:02 PM, Green Luo wrote:

Probably code enhancement could be a solution.

Reply to this email directly or view it on GitHub:
#42 (comment)

[dbusch]

I've made an update to improve the performance of the datasource name lookup. Also have added a couple bugfixes and merged in the latest 1.2.4 code.

[greenlaw110]

thx, will evaluate

On Wed, Nov 2, 2011 at 8:05 AM, dbusch <
[email protected]>wrote:

I've made an update to improve the performance of the datasource name
lookup. Also have and added a couple bugfixes and merged in the latest
1.2.4 code.

Reply to this email directly or view it on GitHub:
#42 (comment)