ASF GitHub Bot commented on IVY-612:

Github user twogee commented on the pull request:

    What else must be done to remove it? Deprecate the class?
    Here's the [relevant 
which refers to a [JIRA issue](https://issues.apache.org/jira/browse/IVY-612) 
attempting to document it. In particular, Xavier Hanin stated
    CacheResolver has been introduced quite a long time ago and never
    documented because it wasn't supposed to be public. I'm still not sure
    it's a good idea to use it... IMO, Ivy caching mechanism should be
    strong enough to avoid the requirement for a cache resolver, at least
    at resolve time. With the addition of dynamic revisions caching in
    beta 2, now Ivy cache has all information necessary to be able to
    perform a resolve from the cache, without actually requiring to use
    the cache resolver. This may require some improvement, like accepting
    to use dynamic revision resolution cached data even when they have
    exceeded their TTL. This could be a special resolve mode or refresh
    mode. With this improvement, the last thing for which the cache
    resolver would still be useful is publishing. But even in this case,
    I'd prefer using a local repository with useOrigin="true": the
    behavior is sligthly different, but I think this should address all
    the needs covered by cache resolver which only has its root in bad
    caching support and is a design flaw.

> Cache Resolver isn't documented
> -------------------------------
>                 Key: IVY-612
>                 URL: https://issues.apache.org/jira/browse/IVY-612
>             Project: Ivy
>          Issue Type: Improvement
>          Components: Documentation
>    Affects Versions: 2.0.0-alpha-2
>            Reporter: Maarten Coene
>              Labels: patch
>         Attachments: crdoc.patch
> The Cache Resolver isn't listed/documented in the available resolvers here:
> http://incubator.apache.org/ivy/history/trunk/configuration/resolvers.html

This message was sent by Atlassian JIRA

Reply via email to