Remove auto-service plugin from mapper processor

Description

The auto-service plugin doesn't buy us much: it just generates a one-line descriptor file

that will rarely, if ever, change.

On the downside, it pulls a transitive dependency to Guava that is quite error-prone: it's easy to accidentally import non-shaded Guava types in the processor code, and that will results in runtime errors if the auto-service dependency is kept optional (as is intended).

On a side note, we also runs all processors in the build, which means the mapper processor runs on itself. This is not really a problem right now, but we have a few annotated classes in the test classpath (

,

) that currently get processed unnecessarily. It will also be a blocker if we ever need sample files that contain configuration errors.

Suggestion:

  • remove the dependency to auto-service, maintain the descriptor file manually instead.

  • add proc:none to the mapper-processor build

Environment

None

Pull Requests

None

Status

Assignee

Olivier Michallat

Reporter

Olivier Michallat

Labels

None

PM Priority

None

Affects versions

None

Fix versions

Pull Request

None

Doc Impact

None

Size

None

External issue ID

None

External issue ID

None

Priority

Minor
Configure