diff options
author | Brion Vibber <brion@pobox.com> | 2010-04-01 11:09:45 -0700 |
---|---|---|
committer | Brion Vibber <brion@pobox.com> | 2010-04-01 11:09:45 -0700 |
commit | d844e6bde511595ae05c60406a58b864d2607d8b (patch) | |
tree | 4f368d81fea69603b8cbe669b6ea4eda613c80ed | |
parent | 5f19a94646a94119fe7be435c33fcf0bda5bdfb9 (diff) | |
parent | 9f2cfd20f80fbeadd59b0f9a153be0b9f0799fae (diff) |
Merge branch 'testing' of git@gitorious.org:statusnet/mainline into 0.9.x
-rw-r--r-- | classes/Profile.php | 63 |
1 files changed, 47 insertions, 16 deletions
diff --git a/classes/Profile.php b/classes/Profile.php index 5de35c191..54f557ea7 100644 --- a/classes/Profile.php +++ b/classes/Profile.php @@ -225,31 +225,62 @@ class Profile extends Memcached_DataObject { $notice = new Notice(); - $notice->profile_id = $this->id; + // Temporary hack until notice_profile_id_idx is updated + // to (profile_id, id) instead of (profile_id, created, id). + // It's been falling back to PRIMARY instead, which is really + // very inefficient for a profile that hasn't posted in a few + // months. Even though forcing the index will cause a filesort, + // it's usually going to be better. + if (common_config('db', 'type') == 'mysql') { + $index = ''; + $query = + "select id from notice force index (notice_profile_id_idx) ". + "where profile_id=" . $notice->escape($this->id); + + if ($since_id != 0) { + $query .= " and id > $since_id"; + } - $notice->selectAdd(); - $notice->selectAdd('id'); + if ($max_id != 0) { + $query .= " and id < $max_id"; + } - if ($since_id != 0) { - $notice->whereAdd('id > ' . $since_id); - } + $query .= ' order by id DESC'; - if ($max_id != 0) { - $notice->whereAdd('id <= ' . $max_id); - } + if (!is_null($offset)) { + $query .= " LIMIT $limit OFFSET $offset"; + } + + $notice->query($query); + } else { + $index = ''; - $notice->orderBy('id DESC'); + $notice->profile_id = $this->id; - if (!is_null($offset)) { - $notice->limit($offset, $limit); + $notice->selectAdd(); + $notice->selectAdd('id'); + + if ($since_id != 0) { + $notice->whereAdd('id > ' . $since_id); + } + + if ($max_id != 0) { + $notice->whereAdd('id <= ' . $max_id); + } + + $notice->orderBy('id DESC'); + + if (!is_null($offset)) { + $notice->limit($offset, $limit); + } + + $notice->find(); } $ids = array(); - if ($notice->find()) { - while ($notice->fetch()) { - $ids[] = $notice->id; - } + while ($notice->fetch()) { + $ids[] = $notice->id; } return $ids; |