[#731 state:resolved] SCHEMA VERSION BUMP - UPGRADE AHEAD! Restored m…

…t_comment_blog_ip_date index which, before it's removal, was critical in streamlining performance of the comment app when comment throttling is enabled. Also removed mt_comment_parent_id index which was redundant and also removed from MT 5 for performance reasons. Source case: http://bugs.movabletype.org/?103831
openmelody · Feb 3, 2011 · cdffa35 · cdffa35
1 parent 104816a
commit cdffa35
Show file tree

Hide file tree

Showing 2 changed files with 3 additions and 2 deletions.
diff --git a/lib/MT.pm b/lib/MT.pm
@@ -49,7 +49,7 @@ BEGIN {
 
         $PRODUCT_VERSION = $VERSION;    # The rightful resting place
         $VERSION         = '4.35';      # The true API version
-        $SCHEMA_VERSION  = '4.0077';
+        $SCHEMA_VERSION  = '4.0078';
         $PRODUCT_NAME    = 'Melody';
         $PRODUCT_CODE    = 'OM';
         $VERSION_ID      = '1.0.0b3 (build 32)';

diff --git a/lib/MT/Comment.pm b/lib/MT/Comment.pm
@@ -37,7 +37,6 @@ __PACKAGE__->install_properties( {
            author        => 1,
            email         => 1,
            commenter_id  => 1,
-           parent_id     => 1,
            last_moved_on => 1,    # used for junk expiration
 
            # For URL lookups to aid spam filtering
@@ -48,6 +47,8 @@ __PACKAGE__->install_properties( {
              { columns => [ 'blog_id', 'visible', 'created_on', 'id' ], },
            dd_coment_vis_mod => { columns => [ 'visible', 'modified_on' ], },
            visible_date      => { columns => [ 'visible', 'created_on' ], },
+           # To help determine whether a comment should be throttled
+           blog_ip_date => { columns => [ 'blog_id', 'ip', 'created_on'], },
        },
        meta => 1,
        defaults =>