Closed
Bug 1386409
Opened 7 years ago
Closed 7 years ago
remove extra processors
Categories
(Socorro :: Processor, task)
Socorro
Processor
Tracking
(Not tracked)
RESOLVED
FIXED
People
(Reporter: willkg, Assigned: willkg)
References
Details
Attachments
(1 file)
Mozilla uses the MozillaProcessorAlgorithm2015 class for the processor. The processor app defaults to the SocorroLiteProcessorAlgorithm2015.
Given that we don't use them and they're essentially dead weight right now, the SocorroLiteProcessorAlgorithm2015 and related bits can all be removed.
Further we can remove the legacy processor and the hybrid_processor. Both of these are just sitting around as well right now.
This bug covers removing these unused extra processor bits.
Assignee | ||
Comment 1•7 years ago
|
||
What does this cover?
Roughly, I think this covers the following files:
- socorro/processor/legacy_processor.py
- socorro/processor/hybrid_processor.py
- socorro/processor/socorrolite_processor_2015.py
- socorro/unittest/processor/test_hybrid_processor.py
- socorro/unittest/processor/test_legacy_processor.py
Why do this now?
I'm in the process of reworking signature generation. All these processors touch signature generation, so that increases the size of the changes I'm making. We have the same issue for any other non-trivial processor changes we want to make.
I'm also in the process of rewriting the processor (project Jansky). Reducing the amount of stuff makes that generally easier.
Assignee | ||
Comment 2•7 years ago
|
||
I'm going to grab this to work on now since it'll make the signature generation bug (bug #1385970) easier.
Assignee: nobody → willkg
Status: NEW → ASSIGNED
Assignee | ||
Comment 3•7 years ago
|
||
Comment 4•7 years ago
|
||
Commit pushed to master at https://github.com/mozilla-services/socorro
https://github.com/mozilla-services/socorro/commit/3600d2b70f29fb31660a5aebb797136d73ee1634
fixes bug 1386409 - remove extra processors (#3877)
* fixes bug 1386409 - remove extra processors
This removes extra processors that we're not using and haven't used in some
time.
* Remove two additional files that aren't used
Updated•7 years ago
|
Status: ASSIGNED → RESOLVED
Closed: 7 years ago
Resolution: --- → FIXED
You need to log in
before you can comment on or make changes to this bug.
Description
•