db-derby-dev mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From Army <qoz...@sbcglobal.net>
Subject Re: [PATCH] Derby-107, Phase II
Date Tue, 15 Feb 2005 18:34:41 GMT
Army wrote:
 >
 > Attached is a patch for Phase II of my proposal for fixing Derby-107:
 >
 > "Phase II) Where it is SAFE to do, modify the values of non-numeric rows
 > in the metadata result sets to be NULL when the columns in question don't
 > apply.  For example, "SCALE/DECIMAL_DIGITS" and "NUM_PREC_RADIX" are two
 > columns that wouldn't apply to a row that is for CHAR, and thus the values
 > in those two columns (for that specific row) would be NULL."
 >   -- Pasted from my first email referenced above.
 >

As I was glancing over the Phase II patch for DERBY-107, I realized that I'm not entirely
certain whether or not the 
SCALE/DECIMAL_DIGITS and NUM_PREC_RADIX columns should apply to date, time, and timestamp
columns...Anyone know?

In my version of the Phase II patch, I assumed they did NOT; now, since I'm not so sure of
that, I'm changing the Phase 
II patch so that date, time, and timestamp columns will return the same values as they always
have for those columns in 
Derby (instead of the NULL values that my original Phase II patch was returning).

Please find the RE-patch for Phase II attached to this email.  Since none of the patches have
been applied so far, I 
hope this isn't too much of an inconvenience.

NOTE: This patch is STILL with respect to the Phase I patch, found here:

http://mail-archives.eu.apache.org/mod_mbox/db-derby-dev/200502.mbox/%3c4202699E.3000009@golux.com%3e

So committers, please commit the Phase I patch before committing this one.

I hope to post the third and final patch for Derby-107 later today.  In the meantime, any
feedback regarding the first 
two patches would be appreciated--and if there isn't any, then perhaps a committer could go
ahead and commit them to the 
trunk...?

Thanks,
Army

Mime
View raw message