Closed Bug 1386409 Opened 7 years ago Closed 7 years ago

remove extra processors

Categories

(Socorro :: Processor, task)

task
Not set
normal

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.
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.
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
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
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.

Attachment

General

Created:
Updated:
Size: