I noticed a bunch of queries being created in the function drupal_get_filepath and that's why I've created the following patch. I'm not sure where to place this actually so hopefully this issue list is a good spot.

See also: http://buytaert.net/drupal-database-interaction

CommentFileSizeAuthor
drupal_file_listing.patch1.84 KBThomasH

Comments

grendzy’s picture

grendzy commented
Version: 6.13 » 7.x-dev
Status: Active » Needs work

New features go in Drupal 7.

catch’s picture

catch
he/him
Primary language English
commented

subscribing. Any particular pages these show up on?

ThomasH’s picture

ThomasH commented

The patch above only works in 6.13

catch’s picture

catch
he/him
Primary language English
commented
Title: Performance: proposal to cache drupal_get_filepath » Cache drupal_get_filepath
Component: file system » base system
Category: feature » task
Issue tags: +Performance

Answering my own question, this shows up on every page - we'd exchange a direct query for a cache_get() - and probably for cached pages too. There's no API change necessary here (at least I hope not), so tagging ready for spit and polish.

catch’s picture

catch
he/him
Primary language English
commented
Status: Needs work » Closed (duplicate)

#626688: Add caching for system_list()