[ 
https://issues.apache.org/jira/browse/DERBY-4625?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=12904225#action_12904225
 ] 

Knut Anders Hatlen commented on DERBY-4625:
-------------------------------------------

Thanks, Nirmal.

Perhaps the change in DataTypeUtilities is better to leave out for now to make 
the impact on the tests smaller. If I understand correctly, it doesn't seem to 
be strictly necessary for getting the TIMESTAMP function to accept nanoseconds, 
it helps IJ display the entire value. Maybe that change could be made as part 
of DERBY-4614 instead?

The patch seems to make an unintended change to the license header in 
SQLTimestamp.java.

The constant SQLTimestamp.FRACTION_TO_NANO is used in 
SQLChar.setValue(Timestamp,Calendar). I think changing it from 1000 to 1 may 
make the calculations in SQLChar become wrong. (If we fix DERBY-4626 that code 
will probably go away, though.)

It would also be good to have a test case (for example in DateTimeTest) that 
verifies that the function now accepts nanosecond resolution.

> TIMESTAMP function doesn't accept nanoseconds
> ---------------------------------------------
>
>                 Key: DERBY-4625
>                 URL: https://issues.apache.org/jira/browse/DERBY-4625
>             Project: Derby
>          Issue Type: Bug
>          Components: SQL
>    Affects Versions: 10.5.3.0
>            Reporter: Knut Anders Hatlen
>            Assignee: C.S. Nirmal J. Fernando
>            Priority: Minor
>         Attachments: derby-4625-1.diff
>
>
> The TIMESTAMP function fails if the string argument specifies the number of 
> nanoseconds. It works if the argument is limited to microsecond resolution.
> ij> values timestamp('2010-04-21 12:00:00.123456');
> 1                         
> --------------------------
> 2010-04-21 12:00:00.123456
> 1 row selected
> ij> values timestamp('2010-04-21 12:00:00.123456789');
> ERROR 22008: '2010-04-21 12:00:00.123456789' is an invalid argument to the 
> timestamp function.
> Since Derby (and JDBC) supports nanosecond resolution, the TIMESTAMP function 
> should also support it.

-- 
This message is automatically generated by JIRA.
-
You can reply to this email to add a comment to the issue online.

Reply via email to