nkuprins commented on issue #989:
URL: https://github.com/apache/fesod/issues/989#issuecomment-5284264551
Hi @delei you raised the compatibility point originally, so probably worth
your call here. As I see it, there are three steps:
1. Keep `FIELD_CACHE`, `CLASS_CONTENT_CACHE` and `CONTENT_CACHE` fields
exactly as they are. Either the resolvers cache into the existing maps, or the
maps live behind the resolvers and `ClassUtils` just assigns them to the same
fields. **The refactor stays internal, nothing downstream changes.**
2. Add read-only accessors and mark the fields `@Deprecated` - starts the
transition window. Strictly, isn't zero risk: a downstream project building
using `-Xlint:deprecation -Werror` would fail.
3. Remove the fields - the **breaking change**, for a later release.
**My opinion:**
The **1** solves the problem I raised without any break, so I lean towards
it.
**2** commits us to **3**, and I'd leave both for a future migration rather
than this PR.
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: [email protected]
For queries about this service, please contact Infrastructure at:
[email protected]
---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]
