presence.beings looking for "" documents

Bug #1718340 reported by John A Meinel
6
This bug affects 1 person
Affects Status Importance Assigned to Milestone
Canonical Juju
Fix Released
High
John A Meinel
2.2
Fix Released
High
John A Meinel

Bug Description

Turns out we had a bug in building the list of IDs to search for. We were allocating a list, and then appending to it, rather than filling in the part we just allocated.

                docIds := make([]string, len(batch))
- for _, seq := range batch {
- docIds = append(docIds, docIDInt64(w.modelUUID, seq))
+ for i, seq := range batch {
+ docIds[i] = docIDInt64(w.modelUUID, seq)

Tags: presence
John A Meinel (jameinel)
Changed in juju:
assignee: nobody → John A Meinel (jameinel)
Changed in juju:
milestone: 2.3-beta1 → 2.3-beta2
Changed in juju:
milestone: 2.3-beta2 → none
Revision history for this message
Anastasia (anastasia-macmood) wrote :

@John A Meinel,

Are you currently actively working on this?

Revision history for this message
John A Meinel (jameinel) wrote :

The fix was landed in juju 2.2.5 and 2.3.0.
The PR was
https://github.com/juju/juju/pull/7863

Changed in juju:
status: In Progress → 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.