[ 
https://issues.apache.org/jira/browse/DERBY-1816?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel
 ]

A B updated DERBY-1816:
-----------------------

    Attachment: d1816_recycleCleanup_v3.patch

Committed d1816_recycleCleanup_v3.patch with svn # 540740:

  URL: http://svn.apache.org/viewvc?view=rev&rev=540740

The difference between v2 and v3 is that the latter sets the time-related 
fields to "0" when getting a timestamp value from a SQL date.  This more 
accurately matches the behavior as it was is in Derby prior to the changes for 
this issue:

_v2:

< +        cal.set(year, month, day);
< +        return new java.sql.Timestamp(cal.getTimeInMillis());

---

_v3:

> +        cal.set(year, month, day, 0, 0, 0);
> +        java.sql.Timestamp ts = new 
> java.sql.Timestamp(cal.getTimeInMillis());
> +        ts.setNanos(0);
> +        return ts;

Other than that the two patches are the same.

> Client's ResultSet.getTime() on a SQL TIMESTAMP column loses the sub-second 
> resolution and always has a milli-second value of zero.
> -----------------------------------------------------------------------------------------------------------------------------------
>
>                 Key: DERBY-1816
>                 URL: https://issues.apache.org/jira/browse/DERBY-1816
>             Project: Derby
>          Issue Type: Bug
>          Components: JDBC, Network Client
>    Affects Versions: 10.1.1.0, 10.1.2.1, 10.1.3.1, 10.2.1.6, 10.3.0.0
>            Reporter: Daniel John Debrunner
>         Assigned To: A B
>            Priority: Minor
>         Attachments: d1816_recycleCleanup_v1.patch, 
> d1816_recycleCleanup_v2.patch, d1816_recycleCleanup_v2.stat, 
> d1816_recycleCleanup_v3.patch
>
>
> In embedded the java.sql.Time object returned from ResultSet.getTime() for a 
> SQL TIMESTAMP object has its millisecond value for the time portion equal to 
> that for the java.sql.Timestamp value.
> In client the millisecond time value for such a value is always set to zero.
> Note a Derby SQL TIME value has by definition resolution of only a second so 
> its millisecond  value is always zero,
> but java.sql.Time  is not a direct mapping to the SQL Type, it's a JDBC type, 
> so when converting from a SQL TIMESTAMP
> it should retain the precision.
> The new test lang.TimeHandlingTest has this assert code that shows the 
> problem, one of its calls will be commented out
> with a comment with this bug number.
>     private void assertTimeEqual(Time tv, Timestamp tsv)
>     {
>         cal.clear();
>         cal.setTime(tv);
>                 
>         int hour = cal.get(Calendar.HOUR_OF_DAY);
>         int min = cal.get(Calendar.MINUTE);
>         int sec = cal.get(Calendar.SECOND);
>         int ms = cal.get(Calendar.MILLISECOND);
>                         
>         // Check the time portion is set to the same as tv
>         cal.clear();
>         cal.setTime(tsv);
>         assertEquals(hour, cal.get(Calendar.HOUR_OF_DAY));
>         assertEquals(min, cal.get(Calendar.MINUTE));
>         assertEquals(sec, cal.get(Calendar.SECOND));
>         assertEquals(ms, cal.get(Calendar.MILLISECOND));      <<<<<<<<<<<<< 
> FAILS HERE
>     }

-- 
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