The modules implementing will be the same on every call unless a new module has been enabled. We should create a cache object service:servers. This should be a serialised array structure:

array(array('#name' => 'service_name', '#path' => service_path'));

Comments

gdd’s picture

Status: Active » Postponed

I'm marking this postponed since I don't really consider a release blocker, and from a performance standpoint I don't think it buys us much. That said if you want to write a patch and reopen within the next couple weeks that's totally cool. Just trying to wean down the open issues.

marcingy’s picture

I agree I have some thoughts on this and it makes more sense in the D7 branch. I'd like to throw out the idea of allowing for servers and services to become entities and hence become fieldable....

gdd’s picture

Version: 6.x-2.x-dev » 6.x-3.x-dev
Status: Postponed » Active

For the record, marcingy and I have talked about making Services entities and fieldable but agree it should wait for a 4.x version. In the meantime, setting the original issue for consideration in 3.x.

marcingy’s picture

Title: Convert module_implements('server_info') calls to run against a cache » Convert services and servers into entities
Version: 6.x-3.x-dev » 7.x-3.x-dev

Renaming this issue and setting to head - the server cache seems like a micro optimisation.

If we go down the route of entities and a site is using entity cache then the caching comes for free, I'll try and think this idea through further in the new year.

ygerasimov’s picture

I am not sure but if servers and resources will be implemented via ctools plugins #855404: Use CTools' plugin system to declare resources. woudn't ctools take care about caching?

gdd’s picture

Status: Active » Postponed

Trying to clear out the 7.3 issue queue, will move these to 7.4 when it opens

marcingy’s picture

Status: Postponed » Closed (won't fix)

Closing again because there is no real benefit to this as I can after using the module in the real world.