Replace Class.forName with Module Check and Direct Load

[SentryMan]

Also check module-path first

[rbygrave]

I'm looking to do a check with GraalVM build time initialisation to see if this should adjust to better support that (native image built time initialisation of the YamlParser). As in, if we pull out this initialisation into another class and build time initialise it, is that a positive feature to look to support or is it not worth it (as the initialisation cascades etc).

[SentryMan]

I'm looking to do a check with GraalVM build time initialisation

I feel like this works better as a fast follow-up rather than to delay the immediate benefits of this PR's improvements. It does sound interesting though

[rbygrave]

Did I miss the immediate benefit? Are you needing an avaje-config RC release? To me it looks like a minor performance improvement if Snake YAML is on the classpath. Is there another benefit I missed? (As in, I'll look to sort out the graal thing in a couple of days. I don't think it's great to churn the avaje-config version too frequently because LOTS of things depend on it, so a couple of days to sort this is imo a good thing to not churn avaje-config releases).

[SentryMan]

I don't think it's great to churn the avaje-config version too frequently

I see, it can wait then