Use List/Set injection into configuration

[wendigo]

This makes configuration more explicit in regard to expected types. Mostly a cleanup

Description

Additional context and related issues

Release notes

( ) This is not user-visible or is docs only, and no release notes are required.
( ) Release notes are required. Please propose a release note for me.
( ) Release notes are required, with the following suggested text:

# Section
* Fix some things. ({issue}`issuenumber`)

[wendigo]

Configuration classes should get expected types, rather than parsing them. This aligns better with the idea of introduction of alternate formats to Java properties (like TOML)

[losipiuk]

Seems mostly fine - but not 100% backward compatible.

[wendigo]

@losipiuk it is, string.toUpperCase was needed to call Enum.valueOf but this is handled by airlift.

[wendigo]

@losipiuk there are tests in both Trino and Airlift that are setting configuration properties mapped to enum with a lower case strings

[losipiuk]

ok

[wendigo]

@losipiuk i.e. here:

trino/plugin/trino-openlineage/src/test/java/io/trino/plugin/openlineage/TestOpenLineageListenerConfig.java

Line 64 in 44a8277

.put("openlineage-event-listener.disabled-facets", "trino_metadata,trino_query_statistics")

as this maps to:

OpenLineageTrinoFacet
{
    TRINO_METADATA,
    TRINO_QUERY_STATISTICS,
    TRINO_QUERY_CONTEXT;
}

[colebow]

Does this need a release note? @wendigo

[wendigo]

@colebow no.