Unnecessary generic type in BeanValidationRecordValidator

[fmbenhassine]

The following example shows incoherence in API design:

Job job = new JobBuilder()
    .reader(new FlatFileRecordReader("tweets.csv"))
    .mapper(new DelimitedRecordMapper(Tweet.class, "id", "name", "message"))
    .validator(new BeanValidationRecordValidator<Tweet>())
    .marshaller(new XmlRecordMarshaller(Tweet.class))
    .build();

The DelimitedRecordMapper and XmlRecordMarshaller take the target type as parameter, while the BeanValidationRecordValidator is generic and is parametrized with the target type.

Looking at the implementation of BeanValidationRecordValidator, ConstraintViolation is the root origin of genericity needs, but methods called on it does not use the generic type. Hence, the generic type can be removed form BeanValidationRecordValidator.