elasticsearch - reset and indexing

Bug #1422232 reported by Robert Lyon
6
This bug affects 1 person
Affects Status Importance Assigned to Milestone
Mahara
Fix Released
Low
Robert Lyon
1.10
Fix Released
Low
Unassigned
1.8
Fix Released
Low
Unassigned
1.9
Fix Released
Low
Unassigned
15.04
Fix Released
Low
Robert Lyon

Bug Description

When one resets the index via extensions -> elasticsearch configuration form part of the process is to index the elasticsearch

If the site is large the 'Cron record limit' value should be set so that not too much of the queue is slurped up to be handled at one time.

One thing I noticed from doing some tests was that when I hit reset index button it goes off and does it's thing which calls the index_queued_items() to index some items but then the cron rolls around and calls it too.

When we hit reset index we should set the cron flag to stop cron from doing the indexing at same time and unset the flag once the reindex step is done by the form.

Robert Lyon (robertl-9)
Changed in mahara:
assignee: nobody → Robert Lyon (robertl-9)
status: Confirmed → In Progress
Revision history for this message
Mahara Bot (dev-mahara) wrote : A patch has been submitted for review

Patch for "master" branch: https://reviews.mahara.org/4279

Revision history for this message
Jinelle Foley-Barnes (jinelleb) wrote :

Hi,

Unless someone can be close by while I install elastic search, this one might be best for a dev to test. There seems to be quite a few moving parts :)

Thanks,
Jinelle

Revision history for this message
Mahara Bot (dev-mahara) wrote :

Patch for "15.04_STABLE" branch: https://reviews.mahara.org/4647

Revision history for this message
Mahara Bot (dev-mahara) wrote :

Patch for "1.10_STABLE" branch: https://reviews.mahara.org/4648

Revision history for this message
Mahara Bot (dev-mahara) wrote :

Patch for "1.9_STABLE" branch: https://reviews.mahara.org/4649

Revision history for this message
Mahara Bot (dev-mahara) wrote :

Patch for "1.8_STABLE" branch: https://reviews.mahara.org/4650

Revision history for this message
Mahara Bot (dev-mahara) wrote : A change has been merged

Reviewed: https://reviews.mahara.org/4648
Committed: http://gitorious.org/mahara/mahara/commit/41d56eabba1cb44eca75403120abbf0239a2cd87
Submitter: Robert Lyon (<email address hidden>)
Branch: 1.10_STABLE

commit 41d56eabba1cb44eca75403120abbf0239a2cd87
Author: Robert Lyon <email address hidden>
Date: Tue Feb 17 08:11:51 2015 +1300

Stopping the elasticsearch cron running at same time as reset index

Bug #1422232

Change-Id: Ia8fd7d074db3be027e1318a07d062a9ed1bb2ad8
Signed-off-by: Robert Lyon <email address hidden>

Revision history for this message
Mahara Bot (dev-mahara) wrote :

Reviewed: https://reviews.mahara.org/4649
Committed: http://gitorious.org/mahara/mahara/commit/11734cf6eb73e529e512036d788756ebf3d6edf1
Submitter: Robert Lyon (<email address hidden>)
Branch: 1.9_STABLE

commit 11734cf6eb73e529e512036d788756ebf3d6edf1
Author: Robert Lyon <email address hidden>
Date: Tue Feb 17 08:11:51 2015 +1300

Stopping the elasticsearch cron running at same time as reset index

Bug #1422232

Change-Id: Ia8fd7d074db3be027e1318a07d062a9ed1bb2ad8
Signed-off-by: Robert Lyon <email address hidden>

Revision history for this message
Mahara Bot (dev-mahara) wrote :

Reviewed: https://reviews.mahara.org/4650
Committed: http://gitorious.org/mahara/mahara/commit/2901622e9f36a5c38f3e96c6ccc62ae0d0d7b67d
Submitter: Robert Lyon (<email address hidden>)
Branch: 1.8_STABLE

commit 2901622e9f36a5c38f3e96c6ccc62ae0d0d7b67d
Author: Robert Lyon <email address hidden>
Date: Tue Feb 17 08:11:51 2015 +1300

Stopping the elasticsearch cron running at same time as reset index

Bug #1422232

Change-Id: Ia8fd7d074db3be027e1318a07d062a9ed1bb2ad8
Signed-off-by: Robert Lyon <email address hidden>

Revision history for this message
Mahara Bot (dev-mahara) wrote :

Reviewed: https://reviews.mahara.org/4279
Committed: http://gitorious.org/mahara/mahara/commit/6722bcfa85019f4ea44c4b336d7a621ab34213f7
Submitter: Robert Lyon (<email address hidden>)
Branch: master

commit 6722bcfa85019f4ea44c4b336d7a621ab34213f7
Author: Robert Lyon <email address hidden>
Date: Tue Feb 17 08:11:51 2015 +1300

Stopping the elasticsearch cron running at same time as reset index

Bug #1422232

Change-Id: Ia8fd7d074db3be027e1318a07d062a9ed1bb2ad8
Signed-off-by: Robert Lyon <email address hidden>

Revision history for this message
Mahara Bot (dev-mahara) wrote :

Reviewed: https://reviews.mahara.org/4647
Committed: http://gitorious.org/mahara/mahara/commit/f8b90169c50de2e6264d647cef8833c045fe08cc
Submitter: Robert Lyon (<email address hidden>)
Branch: 15.04_STABLE

commit f8b90169c50de2e6264d647cef8833c045fe08cc
Author: Robert Lyon <email address hidden>
Date: Tue Feb 17 08:11:51 2015 +1300

Stopping the elasticsearch cron running at same time as reset index

Bug #1422232

Change-Id: Ia8fd7d074db3be027e1318a07d062a9ed1bb2ad8
Signed-off-by: Robert Lyon <email address hidden>

Robert Lyon (robertl-9)
Changed in mahara:
status: Fix Committed → Fix Released
To post a comment you must log in.
This report contains Public information  
Everyone can see this information.

Other bug subscribers

Remote bug watches

Bug watches keep track of this bug in other bug trackers.