At http://svn.apache.org/viewvc?view=revision&revision=r1476296 Adrian noted:
I updated the Delegator JavaDocs to warn against skipping the cache clearing step - which shouldn't be done
After I proposed to rather refactor it, Paul Foxworthy suggested:
If doCacheClear will have no effect, I suggest deprecating the method
signature with the doCacheClear. It can call another overloaded variant
without that parameter. All calls in trunk should be changed to the second.
We can leave the deprecated version for backwards compatibility for a time.
I agree in the release branches that all we should do is add some JavaDoc
and log a warning.
I agree with Paul, and here is how I will do it:
- Deprecate in trunk
- Backport in releases
- Remove useless deprecated in trunk