Mailing-List: contact river-dev-help@incubator.apache.org; run by ezmlm
Precedence: bulk
Reply-To: river-dev@incubator.apache.org
Received-SPF: pass (nike.apache.org: domain of tvhobbs@googlemail.com
 designates 209.85.213.47 as permitted sender)
DomainKey-Signature: a=rsa-sha1; c=nofws;
        d=googlemail.com; s=gamma;
        h=mime-version:in-reply-to:references:date:message-id:subject:from:to
         :content-type:content-transfer-encoding;
        b=LEPJJlLMxalZaQ+zqztZocHcxD3JGx1sm2LBwzXfJ24E4qGT6lrhKuLSx1I2hb2YrV
         SODEsRm9brYkiAYXJ0yCzzajXHwYWHiTfkI4se31O3gN79zTTloZcXzeWP0bZlD7A6du
         GzEwPtjxocOry37IJwUSslc+EzsjSa7NkS5CQ=
MIME-Version: 1.0
In-Reply-To: <4CB6DC86.6030801@acm.org>
References: <AANLkTikgyPis2oBZXpQXU3jdgKV9jJgA89HqQpduP6A=@mail.gmail.com>
	<77F1E32F67C8D5479858C0C7E93EB465030A825A@WAL-MAIL.global.avidww.com>
	<4CB4CADD.6050200@acm.org>
	<77F1E32F67C8D5479858C0C7E93EB465031270C2@WAL-MAIL.global.avidww.com>
	<4CB6DC86.6030801@acm.org>
Date: Thu, 14 Oct 2010 12:17:53 +0100
Message-ID: <AANLkTimzMVJgtywjJecbvLCn1Fdoece2E4=srmPnrku2@mail.gmail.com>
Subject: Re: Java package names
From: Tom Hobbs <tvhobbs@googlemail.com>
To: river-dev@incubator.apache.org
Content-Type: text/plain; charset=ISO-8859-1
Content-Transfer-Encoding: quoted-printable

That's a good point.  We can mark those classes as deprecated for the
2.1.3 release that Peter suggested could be done in November.  The
3.0.0 release (sometime next year, I assume) can then do the actual
deprecation.

On Thu, Oct 14, 2010 at 11:33 AM, Patricia Shanahan <pats@acm.org> wrote:
> Do any of your unit tests test the Jini implementation, independent of th=
e
> rest of your own code? If so, please consider contributing them, so that
> they can be run as part of the build tests. The more unit tests, from the
> more different sources, the better.
>
> One way of minimizing pain during a package reorganization would be to go
> through a step of having both old and new versions of the classes, with t=
he
> old versions unmodified except for being deprecated. Code using the class=
es
> could be modified gradually to remove references to deprecated classes.
>
> Patricia
>
> Christopher Dolan wrote:
>>
>> It turns out that I overstated my case, so I withdraw my negative
>> vote. =A0I wrote my previous email before researching how extensively
>> my code actually uses the com.sun.jini.* classes. =A0It's not nearly as
>> bad as I thought -- I had misrecollected that some of the
>> AbstractEntry subclasses were in com.sun -- and some of it is just
>> bad code in my project. =A0Anecdotally, here are the classes my project
>> currently uses (not including my unit tests, which dig deeper):
>>
>> import com.sun.jini.config.Config; import
>> com.sun.jini.config.ConfigUtil; import
>> com.sun.jini.landlord.Landlord; import
>> com.sun.jini.landlord.LandlordLease; import
>> com.sun.jini.landlord.LeaseFactory; import
>> com.sun.jini.loader.pref.internal.PreferredResources; import
>> com.sun.jini.logging.Levels; import
>> com.sun.jini.thread.TaskManager.Task; import
>> com.sun.jini.thread.TaskManager;
>>
>> I've been trying to refactor many of those out (like TaskManager),
>> but some of them are hard to avoid, especially the landlord classes
>> and Levels. =A0PreferredResources, for example, is in my custom
>> PreferredClassLoader implementation.
>>
>> Chris
>>
>> -----Original Message----- From: Patricia Shanahan
>> [mailto:pats@acm.org] Sent: Tuesday, October 12, 2010 3:54 PM To:
>> river-dev@incubator.apache.org Subject: Re: Java package names
>>
>> This is troubling news. My proposed refactoring of TaskManager to
>> enable performance tuning depends on the assumption that, as a
>> com.sun.* class, it is only used within the project.
>>
>> In general, we will be severely limited in our ability to progress if
>> we have to treat all public com.sun.* interfaces as external
>> interfaces.
>>
>> Patricia
>>
>>
>> Christopher Dolan wrote:
>>>
>>> I vote against such an incompatible change. =A0There are a lot of class=
es
>>> under there, for example com.sun.jini.thread.TaskManager, that are util=
ity
>>> code employed by downstream developers. =A0I think
>>> all new code should go elsewhere if possible, but changing the
>>> existing com.sun.jini packages would be hard on existing users.
>>>
>>> Chris
>>>
>>>
>>> -----Original Message----- From: Benson Margulies
>>> [mailto:bimargulies@gmail.com] Sent: Tuesday, October 12, 2010
>>> 11:51 AM To: general@incubator.apache.org;
>>> river-dev@incubator.apache.org Subject: Java package names
>>>
>>> River imported packages of code from the original Sun grant under
>>> the name 'com.sun.whatever'.
>>>
>>> How important is it to change that?
>>
>
>