db-derby-dev mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From Rick Hillegas <Richard.Hille...@Sun.COM>
Subject Re: [jira] Commented: (DERBY-499) Expose BOOLEAN datatype to end users
Date Fri, 09 Dec 2005 00:15:03 GMT
Just a heads up: I'm the person who is requesting new datatype ids from 
the Open Group. Given the long lead time that this has entailed, I think 
it is best to roll all of our requests together. I am requesting a 
datatype id for XML as well as for BOOLEAN. I intend to ask for the ids 
specified in this patch. I will keep the community posted if these ids 
have to change.

Regards,
-Rick

Army wrote:

> Kathey Marsden (JIRA) wrote:
>
>> 1)
>> My biggest question about  the patch are the new types and the 
>> assignment of these temporary codes.   Will these be finalized before 
>> 10.2? Why have some other types be added to the DRDATypes but not the 
>> SQL Types.  I was wondering if it might pose any conflict with work 
>> underway for XML?
>
>
> Good question--thanks for bringing this up, Kathey.  As it turns out, 
> I've actually been using different ids for XML which conflict with 
> those specified in this patch.
>
> Since, as the comments in the code say, these values are temporary 
> until DRDA is updated to account for the new types, I don't think we 
> need to (nor want to) specify them all up front--it just makes for 
> more things we have to change down the road.  I think it'd be better 
> if we only add the constants for DRDA type and SQL type when their 
> respective datatypes are implemented.
>
> So in this case, I think it's fine for Rick's patch to include the 
> following:
>
> >     public    static final int DRDA_TYPE_BOOLEAN = 0xD0;
> >     public    static final int DRDA_TYPE_NBOOLEAN = 0xD1;
>
> >     public    static final  int DB2_SQLTYPE_BOOLEAN = 1000;     // 
> BOOLEAN
> >     public    static final  int DB2_SQLTYPE_NBOOLEAN = 1001;
>
> But it'd be best to leave the others off.  I'll add my choice of XML 
> ids as part of my XML patch, and other people who implement other 
> types can add the appropriate ids with their patches, as well.
>
> Army
>


Mime
View raw message