Closed Bug 1431167 Opened 2 years ago Closed 2 years ago

Add ReleaseData to the prefix signature list

Categories

(Socorro :: Signature, task)

task
Not set

Tracking

(Not tracked)

RESOLVED FIXED

People

(Reporter: mccr8, Assigned: mccr8)

References

(Blocks 1 open bug)

Details

ReleaseData is a generic function used to destroy string data. I'll add it to the prefix list to split up the different call sites into different signatures.

Here's an example of a crash report with the most common protosignature with this signature (making up 46% of the total crashes):
  bp-cac1d6a0-ed50-4959-a431-321c10180117

After this change, I think the signature will be more like:
  ReleaseData | nsTArray_Impl<T>::DestructRange | nsTArray_Impl<T>::ReplaceElementsAt<T> | mozilla::net::nsHttpRequestHead::SetHeaders
Blocks: 1145613
Commits pushed to master at https://github.com/mozilla-services/socorro

https://github.com/mozilla-services/socorro/commit/96919c4505c9074d7fe86985902a47a2cd7ebcc7
Bug 1431167 - Add ReleaseData to the prefix list

ReleaseData is a generic function used to destroy string data.

https://github.com/mozilla-services/socorro/commit/9f7b7ce4b50895c89da12165ff0e24ca9859daa5
Merge pull request #4296 from amccreight/patch-14

Bug 1431167 - Add ReleaseData to the prefix list
Andrew: There are 3500 or so crash reports with this signature. When this goes to -prod, would you want me to reprocess them?
(In reply to Will Kahn-Greene [:willkg] ET needinfo? me from comment #3)
> Andrew: There are 3500 or so crash reports with this signature. When this
> goes to -prod, would you want me to reprocess them?

I don't think it matters too much. The related crash doesn't seem to be very high priority.
Ok. IF that ever changes, we can always reprocess later.

This is fixed now in master branch, so I'm going to mark it FIXED. It should go out in the next deploy today or tomorrow.
Status: NEW → RESOLVED
Closed: 2 years ago
Resolution: --- → FIXED
You need to log in before you can comment on or make changes to this bug.