Project

General

Profile

Bug #6137

Add a new action reindexall to reindex all the solr index

Added by Jing Tao almost 6 years ago. Updated almost 6 years ago.

Status:
Resolved
Priority:
Normal
Assignee:
Category:
-
Target version:
Start date:
10/09/2013
Due date:
% Done:

0%

Estimated time:
Bugzilla-Id:

Description

Currently we have an action named reindex. If the action has a parameter pid, it will reindex the solr index of the specified pids. If there is no pid, it will reindex all solr index.

Even though only the administrator has the privilege, it is still easy to reindex all solr index accidentally, which sometimes is very expensive. We should remove the "index all feature" from reindex and add a explicit action reindexall.

History

#1 Updated by Matt Jones almost 6 years ago

I think this would be redundant and it would be better to just leave it as is, and have administrators use care. They could just as easily call reindexall as reindex inadvertantly, so there would be minimal gain.

#2 Updated by Jing Tao almost 6 years ago

I read this message after i committed the change :)

I still think we should give an expensive process a dedicated action to avoid any accident.

#3 Updated by Jing Tao almost 6 years ago

  • Status changed from New to Resolved

Just leave as it is.

#4 Updated by ben leinfelder almost 6 years ago

  • Target version changed from 2.2.1 to 2.3.0

Also available in: Atom PDF