Set a default record limit for the elasticsearch cron job

Bug #1464052 reported by Aaron Wells
6
This bug affects 1 person
Affects Status Importance Assigned to Milestone
Mahara
Fix Released
Low
Aaron Wells
1.10
Fix Released
Low
Unassigned
1.9
Fix Released
Low
Unassigned
15.04
Fix Released
Low
Unassigned
15.10
Fix Released
Low
Aaron Wells

Bug Description

We added a setting to the elasticsearch cron job to limit how many records it will attempt to index on each cron run. But by default, we set this to "unlimited".

Every time I set up a new site, I have to manually remember to set it to a reasonable number (which is around 10,000). It would make my life easier if this were in there by default, and it would also be helpful to our users so that they don' have to figure out what's a reasonable number for this value.

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/4829

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

Reviewed: https://reviews.mahara.org/4829
Committed: https://git.nzoss.org.nz/mahara/mahara/commit/0b1d3d68f9ff6ce0ff232ccc853eab5606eed498
Submitter: Robert Lyon (<email address hidden>)
Branch: master

commit 0b1d3d68f9ff6ce0ff232ccc853eab5606eed498
Author: Aaron Wells <email address hidden>
Date: Thu Jun 11 12:51:22 2015 +1200

Set elasticsearch cronlimit to 50000 by default.

Bug 1464052: Giving it a default of 50000 instead of unlimited.

Change-Id: I30210d3416d70d96bec40ebb578c5302e07ff63f

Revision history for this message
Mahara Bot (dev-mahara) wrote : A patch has been submitted for review

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

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

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

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

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

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

Reviewed: https://reviews.mahara.org/5114
Committed: https://git.nzoss.org.nz/mahara/mahara/commit/174e5363cf4052aa2d54c6013b3efa4aeebdfa18
Submitter: Robert Lyon (<email address hidden>)
Branch: 1.9_STABLE

commit 174e5363cf4052aa2d54c6013b3efa4aeebdfa18
Author: Aaron Wells <email address hidden>
Date: Thu Jun 11 12:51:22 2015 +1200

Set elasticsearch cronlimit to 50000 by default.

Bug 1464052: Giving it a default of 50000 instead of unlimited.

Change-Id: I30210d3416d70d96bec40ebb578c5302e07ff63f
(cherry picked from commit 0b1d3d68f9ff6ce0ff232ccc853eab5606eed498)

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

Reviewed: https://reviews.mahara.org/5113
Committed: https://git.nzoss.org.nz/mahara/mahara/commit/50c3a53f8e44104802f36f42923b4d0ada2b6e1f
Submitter: Robert Lyon (<email address hidden>)
Branch: 1.10_STABLE

commit 50c3a53f8e44104802f36f42923b4d0ada2b6e1f
Author: Aaron Wells <email address hidden>
Date: Thu Jun 11 12:51:22 2015 +1200

Set elasticsearch cronlimit to 50000 by default.

Bug 1464052: Giving it a default of 50000 instead of unlimited.

Change-Id: I30210d3416d70d96bec40ebb578c5302e07ff63f
(cherry picked from commit 0b1d3d68f9ff6ce0ff232ccc853eab5606eed498)

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

Reviewed: https://reviews.mahara.org/5112
Committed: https://git.nzoss.org.nz/mahara/mahara/commit/2048414c7d741019c63102440147fbcb9a691b71
Submitter: Robert Lyon (<email address hidden>)
Branch: 15.04_STABLE

commit 2048414c7d741019c63102440147fbcb9a691b71
Author: Aaron Wells <email address hidden>
Date: Thu Jun 11 12:51:22 2015 +1200

Set elasticsearch cronlimit to 50000 by default.

Bug 1464052: Giving it a default of 50000 instead of unlimited.

Change-Id: I30210d3416d70d96bec40ebb578c5302e07ff63f
(cherry picked from commit 0b1d3d68f9ff6ce0ff232ccc853eab5606eed498)

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.