Make compatible with the Queue-based Update module (#597484)

dww - October 11, 2009 - 06:57
Project:CVS deploy
Version:7.x-1.x-dev
Component:Code
Category:task
Priority:critical
Assigned:dww
Status:closed
Description

#597484: Use the Queue API to fetch available update data is going to break cvs_deploy attempting to get info about HEAD releases, since we no longer stash everything in a single cache entry in {cache_update}. Once that lands, we need to fix what we're doing in here.

The memory intensive solution is to grab everything, like we do now.

The query intensive solution is to only query individual projects that have a HEAD tag one by one as we need them.

#1

dww - October 11, 2009 - 06:59
Status:active» needs review

For people trying to test #597484, here's the simple patch for the memory-intensive option. ;) We can consider the query-intensive approach if/when #597484 lands.

AttachmentSize
601310-1.cvs_deploy_update_queue_api_all.patch 957 bytes

#2

dww - October 16, 2009 - 17:26
Priority:normal» critical
Status:needs review» fixed

Dave Reid and I both think we should probably go fine-grained and solve this problem with more queries and less RAM. However, now that #597484: Use the Queue API to fetch available update data is in, cvs_deploy is currently broken with D7. So, getting this working again is critical. ;) Therefore, I committed #1 to HEAD as a stop-gap measure. We can revisit this as we work on #238950: Reduce RAM resource consumption.

#3

System Message - October 30, 2009 - 17:30
Status:fixed» closed

Automatically closed -- issue fixed for 2 weeks with no activity.

 
 

Drupal is a registered trademark of Dries Buytaert.