[ 
https://issues.apache.org/jira/browse/HTTPCLIENT-2204?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel
 ]

Robert Kennedy updated HTTPCLIENT-2204:
---------------------------------------
    Description: 
Similar to HTTPCLIENT-2202 
 
There appears to be a regression from 4.x to 5.x in which 
[MemcachedHttpAsyncCacheStorage|https://github.com/apache/httpcomponents-client/blob/master/httpclient5-cache/src/main/java/org/apache/hc/client5/http/impl/cache/memcached/MemcachedHttpAsyncCacheStorage.java]
 now requires the MemcachedClient object instead of the interface 
MemcachedClientIF. The class doesn't use any functionality outside of the 
interface, so why require anything higher than the interface?

Changing to use MemcachedClientIF would be backward compatible since 
MemcachedClient implements this interface already.

  was:
Similar to 
[HTTPCLIENT-2202|https://issues.apache.org/jira/browse/HTTPCLIENT-2202] 
 
There appears to be a regression from 4.x to 5.x in which 
MemcachedHttpAsyncCacheStorage now requires the MemcachedClient object instead 
of the interface MemcachedClientIF. The class doesn't use any functionality 
outside of the interface, so why require anything higher than the interface?

Changing to use MemcachedClientIF would be backward compatible since 
MemcachedClient implements this interface already.


>  MemcachedAsyncHttpCacheStorage should use MemcachedClientIF interface
> ----------------------------------------------------------------------
>
>                 Key: HTTPCLIENT-2204
>                 URL: https://issues.apache.org/jira/browse/HTTPCLIENT-2204
>             Project: HttpComponents HttpClient
>          Issue Type: Bug
>          Components: HttpCache
>    Affects Versions: 5.0, 5.1, 5.1.1, 5.1.2, 5.1.3
>            Reporter: Robert Kennedy
>            Priority: Minor
>             Fix For: 5.2-alpha2
>
>
> Similar to HTTPCLIENT-2202 
>  
> There appears to be a regression from 4.x to 5.x in which 
> [MemcachedHttpAsyncCacheStorage|https://github.com/apache/httpcomponents-client/blob/master/httpclient5-cache/src/main/java/org/apache/hc/client5/http/impl/cache/memcached/MemcachedHttpAsyncCacheStorage.java]
>  now requires the MemcachedClient object instead of the interface 
> MemcachedClientIF. The class doesn't use any functionality outside of the 
> interface, so why require anything higher than the interface?
> Changing to use MemcachedClientIF would be backward compatible since 
> MemcachedClient implements this interface already.



--
This message was sent by Atlassian Jira
(v8.20.1#820001)

---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]

Reply via email to