Title: RE: Issues with beta 2 - does it need more work?

> -----Original Message-----
> From: Russell Butek [mailto:[EMAIL PROTECTED]]
> Sent: Monday, April 29, 2002 10:45 AM
> To: [EMAIL PROTECTED]
> Subject: Issues with beta 2 - does it need more work?
>
>
> Over the weekend some folks played with beta 2 rc 1.  I've
> seen a number of
> issues related to it.
>
> 1.  Glen found and fixed a problem with bean field names -
> capitalized vs
> lowercase (in the head branch - NOT in the beta 2 branch).  Glen and I
> determined that this should not go into beta2, but we should
> ask the wider
> community.  Should this fix go into beta 2?
>
> 2.  There's a problem with our mapping of xsd:date (thanks to
> Tom Mikalsen
> for finding it).  It used to work in beta 1.  It doesn't in
> beta 2.  Since
> it's a regression, the fix is a good candidate for inclusion
> in beta 2.
> Here's the relevant code in
> org/apache/axis/encoding/DefaultTypeMappingImpl.java:
>
>         myRegister(Constants.XSD_DATE1,      
> java.util.Calendar.class,
>                    new DateSerializerFactory(java.util.Date.class,
>                                              Constants.XSD_DATE1),
>                    new DateDeserializerFactory(java.util.Date.class,
>                                                Constants.XSD_DATE1),
>                    true);

> Note that we're mixing Calendar and Date in this code.  It
> should be all
> Calendar or all Date.  I suggest it should be all Date because
> CalendarDeserializer assumes a time portion as well, which
> doesn't exist in
> xsd:date.  (And after beta2 we should write a test for this as well!)
>
> 3.  The C++ code is in the source distribution.  Do we want that to be
> there?  It implies that we're distributing a C++
> implementation which we're
> not yet.  I think we should remove the C++ files from the beta 2
> distribution.
>
> Any other issues?
>
> At this point my votes for changing the beta 2 for these issues are:
>
> 1.  -0
> 2.  +1
> 3.  +0

My votes are
1. +1 to fixing bean field names for beta 2
2. +1 to fixing xsd:date mapping for beta 2
3. -1 to not including C++ code in beta 2

I'd prefer to never include the C++ code in any release of the Java version of Axis.  We should have separate downloads just like there are Java and C versions of Xerces that are downloaded separately.



***********************************************************************************
WARNING: All e-mail sent to and from this address will be received or
otherwise recorded by the A.G. Edwards corporate e-mail system and is
subject to archival, monitoring or review by, and/or disclosure to,
someone other than the recipient.
************************************************************************************

Reply via email to