hc-dev mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From sebb <seb...@gmail.com>
Subject Re: svn commit: r1057745
Date Tue, 11 Jan 2011 22:21:58 GMT
On 11 January 2011 20:43, Oleg Kalnichevski <olegk@apache.org> wrote:
> On Tue, 2011-01-11 at 20:14 +0000, Moore, Jonathan wrote:
>> Well, it didn't take me that long to do, and if we aren't just going to
>> remove the deprecated method, I wanted to be able to sleep at night. :)
>>
>> If we don't want to include code like this then I think we should just
>> remove the deprecated method, because even if we remain binary
>> backwards-compatible for one of these folks, the caching module won't work
>> for them properly (they'll get UnsupportedOperationExceptions) and they'll
>> have to upgrade to the new API anyway. I'd rather have that discovery
>> happen for them at compile time than at runtime, to be frank.
>>
>> So I guess I'm coming around to: I think we should remove this method
>> which only appeared publicly in a beta release and for which I think there
>> is a high probability there are exactly zero people who will actually have
>> stuff break if we remove it.

+1

>>
>
> That is a lot of unnecessary code. Let's just remove it along with the
> deprecated method, and move on.

There's also a corresponding deprecated ctor.
Also the variantURIs private field.

> Oleg
>
>
>
> ---------------------------------------------------------------------
> To unsubscribe, e-mail: dev-unsubscribe@hc.apache.org
> For additional commands, e-mail: dev-help@hc.apache.org
>
>

---------------------------------------------------------------------
To unsubscribe, e-mail: dev-unsubscribe@hc.apache.org
For additional commands, e-mail: dev-help@hc.apache.org


Mime
View raw message