0008070: check index lengths / message cache
authorPhilipp Schüle <p.schuele@metaways.de>
Thu, 31 Jul 2014 08:35:45 +0000 (10:35 +0200)
committerPhilipp Schüle <p.schuele@metaways.de>
Thu, 31 Jul 2014 08:59:42 +0000 (10:59 +0200)
* removes message cache update from update script because that might
take veeeery long for big cache sizes

https://forge.tine20.org/mantisbt/view.php?id=8070

Change-Id: I61b7496ebe664ea56f7e75031b1fb048ea87a7a4
Reviewed-on: http://gerrit.tine20.com/customers/918
Reviewed-by: Philipp Schüle <p.schuele@metaways.de>
Tested-by: Jenkins CI (http://ci.tine20.com/)
tine20/Felamimail/Setup/Update/Release8.php

index 134e328..d77e02f 100644 (file)
@@ -88,25 +88,31 @@ class Felamimail_Setup_Update_Release8 extends Setup_Update_Abstract
                             "localname" => "",
                             "parent" => ""
                             ),
-                        "felamimail_cache_message" => array(
-                            "content_type" => "",
-                            "body_content_type" => "",
-                            "from_email" => "",
-                            "from_name" => "",
-                            "sender" => ""
-                            ),
-                        "felamimail_cache_message_to" => array(
-                            "name" => "",
-                            "email" => ""
-                            ),
-                        "felamimail_cache_message_cc" => array(
-                            "name" => "",
-                            "email" => ""
-                            ),
-                        "felamimail_cache_message_bcc" => array(
-                            "name" => "",
-                            "email" => ""
-                            ),
+            // NOTE: we do not update the current message cache structure as this could
+            // lead to problems for large installations with lots of messages in the cache
+            // TODO: find out cache size, no problems with smaller sizes
+            // TODO: find out ways to improve alter table speed for big caches
+            
+//                         "felamimail_cache_message" => array(
+//                             "content_type" => "",
+//                             "body_content_type" => "",
+//                             "from_email" => "",
+//                             "from_name" => "",
+//                             "sender" => ""
+//                             ),
+//                         "felamimail_cache_message_to" => array(
+//                             "name" => "",
+//                             "email" => ""
+//                             ),
+//                         "felamimail_cache_message_cc" => array(
+//                             "name" => "",
+//                             "email" => ""
+//                             ),
+//                         "felamimail_cache_message_bcc" => array(
+//                             "name" => "",
+//                             "email" => ""
+//                             ),
+
                         "felamimail_sieve_rule" => array(
                             "action_type" => "",
                             "action_argument" => ""
@@ -121,10 +127,10 @@ class Felamimail_Setup_Update_Release8 extends Setup_Update_Abstract
         $this->truncateTextColumn($columns, 255);
         $this->setTableVersion('felamimail_account', 20);
         $this->setTableVersion('felamimail_folder', 14);
-        $this->setTableVersion('felamimail_cache_message', 9);
-        $this->setTableVersion('felamimail_cache_message_to', 2);
-        $this->setTableVersion('felamimail_cache_message_cc', 2);
-        $this->setTableVersion('felamimail_cache_message_bcc', 2);
+//         $this->setTableVersion('felamimail_cache_message', 9);
+//         $this->setTableVersion('felamimail_cache_message_to', 2);
+//         $this->setTableVersion('felamimail_cache_message_cc', 2);
+//         $this->setTableVersion('felamimail_cache_message_bcc', 2);
         $this->setTableVersion('felamimail_sieve_rule', 3);
         $this->setTableVersion('felamimail_sieve_vacation', 3);
         $this->setApplicationVersion('Felamimail', '8.3');