Details
-
Improvement
-
Status: To Do
-
Minor
-
Resolution: Unresolved
-
0.3.0
-
None
-
None
Description
We have a lot of Javac warnings (several hundred) in the build right now. The primary cause of these is unchecked types (for reasons including org.simple.json uses raw types under the hood). These should be labelled with @SuppressWarnings("unchecked") where appropriate. In addition, things like @SafeVarargs should be added where appropriate, etc.
Given the number of changes in this ticket is significantly larger than even the Error Prone warnings (and may require tinkering with generics at appropriate points), I'm going to split this up by module (Analytics and Platform), and then split platform as necessary (parsers and enrichment in particular have a lot). Otherwise, anything resulting is likely to be too large and unmanageable to be properly tested and reviewed.
Attachments
1.
|
Eliminate Javac Warnings in metron-analytics |
|
Done | Justin Leet |
2.
|
Eliminate Javac warnings in metron-parsers |
|
To Do | Unassigned |
3.
|
Eliminate Javac warnings in metron-enrichment |
|
To Do | Unassigned |
4.
|
Eliminate Javac warnings in metron-platform outside of parsers and enrichment |
|
To Do | Unassigned |
5.
|
Eliminate Javac Warnings in metron-interface |
|
To Do | Unassigned |