Mailing-List: contact dev-help@harmony.apache.org; run by ezmlm
Precedence: bulk
Reply-To: dev@harmony.apache.org
Received-SPF: pass (herse.apache.org: domain of smallsmallorgan@gmail.com
 designates 64.233.162.229 as permitted sender)
DomainKey-Signature: a=rsa-sha1; c=nofws;
        d=gmail.com; s=beta;
        h=received:message-id:date:from:to:subject:in-reply-to:mime-version:content-type:references;
        b=bdn3vAk0a11zaiicR0aYxJVmqB2oLOYD7iZ2s2DrkKJT48x2kdbDgItUdACWzOGQmzNMvmmsICxwWtTlT3rDXE92FLWdCYlgb1EyUHxXAqAcjR1++KGPUnUaMANmgH/ImYWXKFggjnrh98z03G5S0/2yMMASh0Kk+glyMZuu67Q=
Message-ID: <473c46620707042017j5ddd258blc99e9e5d6251c27d@mail.gmail.com>
Date: Thu, 5 Jul 2007 11:17:01 +0800
From: "Spark Shen" <smallsmallorgan@gmail.com>
To: dev@harmony.apache.org
Subject: Re: [classlib][beans] Current java6 bean implementation does not
 persist some class properly [was: There are some JDK 6 feature not
 implemented]
In-Reply-To: <e66844de0707041956wfd5b2a2q4b259ae1be3d485f@mail.gmail.com>
MIME-Version: 1.0
Content-Type: multipart/alternative;
	boundary="----=_Part_144897_28200917.1183605421349"
References: <473c46620707030357u38241ac8kc4d544cc4b9573cb@mail.gmail.com>
	 <211709bc0707030542t668630f2o2b7eb9c375b1b81b@mail.gmail.com>
	 <2c9597b90707040819x1e911e0eu1a2e9ae3af38b663@mail.gmail.com>
	 <f982cec50707041919jb29d8dci4234bbadbdc1f713@mail.gmail.com>
	 <473c46620707041931u3ddee6ffvb4a99e4791ec4b07@mail.gmail.com>
	 <f982cec50707041944t314991fh2867a69015ff8d38@mail.gmail.com>
	 <e66844de0707041956wfd5b2a2q4b259ae1be3d485f@mail.gmail.com>

------=_Part_144897_28200917.1183605421349
Content-Type: text/plain; charset=ISO-8859-1; format=flowed
Content-Transfer-Encoding: 7bit
Content-Disposition: inline

2007/7/5, Leo Li <liyilei1979@gmail.com>:
>
> Agree.
> More testcases shall be added and maybe we need an estimation and a plan
> before we start. Since the spec of beans module is so unclear that it will
> give us benefits if we can grasp the global image of the problem and where
> to refractor and what function shall be added.


Hmm, agree.
I am now working on this issues for several days, and the 2 big problems
related to beans module are:

1. Some test cases fails on RI while some others fail on Harmony.

2. PeresistenceDelegate does not persist object properly, for example
current implementation does not persist collection, awt.Choice etc.

Is some one else also interested, or could provide more information on this
module?

On 7/5/07, Yang Paulex <paulex.yang@gmail.com> wrote:
> >
> > 2007/7/5, Spark Shen <smallsmallorgan@gmail.com>:
> > >
> > > 2007/7/5, Yang Paulex <paulex.yang@gmail.com>:
> > > >
> > > > 2007/7/4, Alexei Zakharov <alexei.zakharov@gmail.com>:
> > > > >
> > > > > Spark Chen wrote:
> > > > > > > If no objection, I will go to implement those missing
> > persistence
> > > > > > > functionality.
> > > > >
> > > > > FYI we already have number of persistence delegates classes
> located
> > in
> > > > > org.apache.harmony.beans package, and FieldPersistenceDelegate is
> > one
> > > > > of them.
> > >
> > >
> > > I agree to utilize existing implementation. But there are still others
> > > missing. For example, java.util.Collection and its subclasses[1].
> > >
> > > [1]http://issues.apache.org/jira/browse/HARMONY-4327
> > >
> > > It contains the algorithm that is very close to one you've
> > > > > implemented in the patch for HARMONY-4321. So I don't think we
> > should
> > > > > add new FieldPersistenceDelegate, let's fix the existing one
> instead
> > > > > (if it needs fixing). BTW, we also have a special folder
> > > > > (src/test/java-internal/org/apache/harmony/beans) where tests for
> > > > > persistence delegates are located. Shouldn't we create something
> > like
> > > > > FieldPersistenceDelegateTest there?
> > > >
> > > >
> > > > Seems the tests in src/test/java-internal/org/apache/harmony/beans
> are
> > > > implementation tests, all of which fail on RI with message like "
> > > > java.lang.NoClassDefFoundError:
> > > > org/apache/harmony/beans/ArrayPersistenceDelegate ". But I think the
> > > > persistent delegate mechanism are very similar with serialization,
> on
> > > > which
> > > > we need to be compatible with other Java SE implementation, so it
> > makes
> > > > sense to make them the API tests which pass on RI. Did I miss
> > something?
> > >
> > >
> > > I agree to regard these test cases as  API tests.  And then these
> > > persistenceDelegate implementations better be placed at
> > > src/main/java/java/beans corresponding to test case layout.
> >
> >
> > IMO, test case layout and  implementation class layout are different
> > thing.
> > What I meant is the test cases may need to be written against API
> factory
> > method instead of construct implementation classes directly via
> > implementation specific constructor/class, and the compatibility tests
> > like
> > serialization tests are needed. I have no preference in where the
> > implementation classes locate.  :)
> >
> > Thanks,
> > > > >
> > > > > 2007/7/3, Tony Wu <wuyuehao@gmail.com>:
> > > > > > On 7/3/07, Spark Shen <smallsmallorgan@gmail.com> wrote:
> > > > > > > I find beans.XMLEncoder does not persist
> > > > java.lang.reflect.Fieldproperly.
> > > > > > > And I suspect there are more classes not properly handled.
> > > > > > >
> > > > > >
> > > > > > Yes, I think so.
> > > > > > > I have reported a JIRA:
> > > > > > > https://issues.apache.org/jira/browse/HARMONY-4321
> > > > > > >
> > > > > > > If no objection, I will go to implement those missing
> > persistence
> > > > > > > functionality.
> > > > > > >
> > > > > >
> > > > > > please go head:)
> > > > >
> > > > >
> > > > >
> > > > > --
> > > > > Alexei Zakharov,
> > > > > Intel ESSD
> > > > >
> > > >
> > > >
> > > >
> > > > --
> > > > Paulex Yang
> > > > China Software Development laboratory
> > > > IBM
> > > >
> > >
> > >
> > >
> > > --
> > > Spark Shen
> > > China Software Development Lab, IBM
> > >
> >
> >
> >
> > --
> > Paulex Yang
> > China Software Development laboratory
> > IBM
> >
>
>
>
> --
> Leo Li
> China Software Development Lab, IBM
>


-- 
Spark Shen
China Software Development Lab, IBM

------=_Part_144897_28200917.1183605421349--