Refactor Bean property introspection

[cowtowncoder]

This PR is for (roughly) first half of work for #4515.

What it does is rewrite Creator (constructor, factory-method) introspection to happen mostly in POJOPropertiesCollector (early, along with Field/Getter/Setter introspection); and rely less on BasicDeserializerFactory handling which occurs much later on in the process.

But this first PR does not eliminate redundancy yet, leaving BasicDeserializerFactory to do sort of double processing; old methods in POJOPropertiesCollector are left (mostly) as-is as well (but they will be removed ASAP after merging to 3.0).

[cowtowncoder]

Some progress has been made, rewriting logic of POJOPropertiesCollector; leaving about 12 failures, not including Record-tests (more work for Records).

[JooHyukKim]

This may be personal preference.

From here to line 766 seems like we can just end everything with...

if  explicit do explcit handling
else do implicit 

And not mix up two different handling. So that is easier to make changes per case

[cowtowncoder]

Yeah I am sure there are ways to clean things up: at first I am trying to replicate existing logic. So once things pass will focus more on clean up.

Exciting thing is that right now I am 1 test fail away from passing all Java 8 tests -- then need to tackle record handling.

And once that happens, then start making use of Creators detected: right now all I have changed is finding properties-based Creators to link properties in POJOPropertiesCollector, but BasicDeserializerFactory then re-scans everything. But that won't be necessary for long.

... and need to figure out at what point to merge things to master. Probably half-way through, to keep diff manageable.

But at least I am FINALLY making progress!

[cowtowncoder]

Forgot to mention that this part of code will definitely be rewritten as right now it is strictly duplicating existing handling, but once everything works as well as it used to, can and will start simplifying & cleaning up.

[cowtowncoder]

And with that, ALL tests pass! On to merging to 2.18, then try to tackle merge to master (3.0)

[JooHyukKim]

Great work! @cowtowncoder 🔥

[cowtowncoder]

Thanks @JooHyukKim ! Hope to continue with follow-up refactoring to remove Creator selection from BasicDeserializerFactory -- and then maybe improve logic in POJOPropertiesCollector by:

1. Better support for actual annotation-less auto-detection (finally possible now)
2. Add hook(s) for Kotlin and Scala module to indicate canonical constructor (or maybe just creator, allowing factory methods) to use, f.ex for Kotlin Data classes (and Scala has something similar too I think)

With (2) I'll need @pjfanning 's and @k163377 's help; but I am not quite there yet.
Hook(s) to call would probably be via AnnotationIntrospector, called from POJOPropertiesCollector "_addCreators()" method.