Remove -Xlint exclusions in the ingest-common module.

The generics in AbstractStringProcessor is not needed
(IngestDocument#setFieldValue(...) accepts Object) and
therefor it was removed. Otherwise the corresponding
factory implementations would need typing too.

Relates to #40366

Author: martijnvg

modules/ingest-common/build.gradle

@@ -29,9 +29,6 @@ dependencies {
     compile project(':libs:dissect')
 }
 
-compileJava.options.compilerArgs << "-Xlint:-unchecked,-rawtypes"
-compileTestJava.options.compilerArgs << "-Xlint:-unchecked,-rawtypes"
-
 integTestCluster {
     module project(':modules:lang-painless')
 }

modules/ingest-common/src/main/java/org/elasticsearch/ingest/common/AbstractStringProcessor.java

@@ -29,10 +29,8 @@
 /**
  * Base class for processors that manipulate source strings and require a single "fields" array config value, which
  * holds a list of field names in string format.
- *
- * @param <T> The resultant type for the target field
  */
-abstract class AbstractStringProcessor<T> extends AbstractProcessor {
+abstract class AbstractStringProcessor extends AbstractProcessor {
     private final String field;
     private final boolean ignoreMissing;
     private final String targetField;
@@ -70,7 +68,7 @@ public final IngestDocument execute(IngestDocument document) {
         return document;
     }
 
-    protected abstract T process(String value);
+    protected abstract Object process(String value);
 
     abstract static class Factory implements Processor.Factory {
         final String processorType;

modules/ingest-common/src/test/java/org/elasticsearch/ingest/common/AbstractStringProcessorTestCase.java

@@ -31,18 +31,18 @@
 import static org.hamcrest.Matchers.containsString;
 import static org.hamcrest.Matchers.equalTo;
 
-public abstract class AbstractStringProcessorTestCase<T> extends ESTestCase {
+public abstract class AbstractStringProcessorTestCase extends ESTestCase {
 
     protected abstract AbstractStringProcessor newProcessor(String field, boolean ignoreMissing, String targetField);
 
     protected String modifyInput(String input) {
         return input;
     }
 
-    protected abstract T expectedResult(String input);
+    protected abstract Object expectedResult(String input);
 
-    protected Class<T> expectedResultType(){
-        return (Class<T>) String.class;  // most results types are Strings
+    protected Class<?> expectedResultType(){
+        return String.class;  // most results types are Strings
     }
 
     public void testProcessor() throws Exception {