Elasticsearch plugin is not reading the queue in order

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

Bug Description

There's a minor syntax error in the elasticsearch code that pulls out data from the queue for sending to the server.

The 3rd and 4th arguments to get_records_array() are switched, which means that it pulls records in arbitrary database order, instead of specifically ordered by ID.

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

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

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

commit e475f524106371b351351fdc6eb6ac0cb76161ab
Author: Aaron Wells <email address hidden>
Date: Thu May 28 18:28:48 2015 +1200

Correct the order of arguments in a call to get_records_array()

Bug 1457710

Change-Id: I0ae1e2264e6f88af215d6bb769cdbfe8bd489584

Robert Lyon (robertl-9)
Changed in mahara:
status: In Progress → Fix Committed
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.