Closed Bug 1272431 Opened 9 years ago Closed 9 years ago

Make TelemetryPingStore.getAllPings return pings in order – earliest to latest

Categories

(Firefox for Android Graveyard :: General, defect)

Product:
Firefox for Android Graveyard
Component:
General
Platform:
All
Android
Type:
defect
Priority:
Not set
Severity:
normal

Tracking

(firefox49 fixed)

Status:
RESOLVED FIXED
Milestone:
Firefox 49
Tracking Flags:
Tracking Status
firefox49 --- fixed

People

(Reporter: mcomella, Assigned: mcomella)

References

Details

Attachments

(3 files)

MozReview Request: Bug 1272431 - Correct order errors in maybePrunePings tests. r=ahunt
(deleted), text/x-review-board-request
ahunt
: review+
Details
MozReview Request: Bug 1272431 - Return pings from getAllPings in guaranteed order. r=ahunt
(deleted), text/x-review-board-request
ahunt
: review+
Details
MozReview Request: Bug 1272431 - Do not use API 19+ method for FileLastModifiedComparator. r=ahunt
(deleted), text/x-review-board-request
ahunt
: review+
Details
via bug 1270213 comment 24. This lets us try to offload our data before it gets deleted if we run out of space to store pings.
Comment on attachment 8753144 [details] MozReview Request: Bug 1272431 - Correct order errors in maybePrunePings tests. r=ahunt Review request updated; see interdiff: https://reviewboard.mozilla.org/r/53036/diff/1-2/
Attachment #8753144 - Attachment description: MozReview Request: Bug 1272431 - Correct order errors in maybePrunePings tests. r=sebastian → MozReview Request: Bug 1272431 - Correct order errors in maybePrunePings tests. r=ahunt
Attachment #8753145 - Attachment description: MozReview Request: Bug 1272431 - Return pings from getAllPings in guaranteed order. r=sebastian → MozReview Request: Bug 1272431 - Return pings from getAllPings in guaranteed order. r=ahunt
Attachment #8753146 - Attachment description: MozReview Request: Bug 1272431 - Do not use API 19+ method for FileLastModifiedComparator. r=sebastian → MozReview Request: Bug 1272431 - Do not use API 19+ method for FileLastModifiedComparator. r=ahunt
Attachment #8753144 - Flags: review?(s.kaspari) → review?(ahunt)
Attachment #8753145 - Flags: review?(s.kaspari) → review?(ahunt)
Attachment #8753146 - Flags: review?(s.kaspari) → review?(ahunt)
Comment on attachment 8753145 [details] MozReview Request: Bug 1272431 - Return pings from getAllPings in guaranteed order. r=ahunt Review request updated; see interdiff: https://reviewboard.mozilla.org/r/53038/diff/1-2/
Comment on attachment 8753146 [details] MozReview Request: Bug 1272431 - Do not use API 19+ method for FileLastModifiedComparator. r=ahunt Review request updated; see interdiff: https://reviewboard.mozilla.org/r/53040/diff/1-2/
Comment on attachment 8753144 [details] MozReview Request: Bug 1272431 - Correct order errors in maybePrunePings tests. r=ahunt https://reviewboard.mozilla.org/r/53036/#review50382
Attachment #8753144 - Flags: review?(ahunt) → review+
Comment on attachment 8753145 [details] MozReview Request: Bug 1272431 - Return pings from getAllPings in guaranteed order. r=ahunt https://reviewboard.mozilla.org/r/53038/#review50386
Attachment #8753145 - Flags: review?(ahunt) → review+
Attachment #8753146 - Flags: review?(ahunt) → review+
Comment on attachment 8753146 [details] MozReview Request: Bug 1272431 - Do not use API 19+ method for FileLastModifiedComparator. r=ahunt https://reviewboard.mozilla.org/r/53040/#review50406
Product: Firefox for Android → Firefox for Android Graveyard
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Creator:
Created:
Updated:
Size: