search

LucidDB / luciddb

DEFUNCT: See README
https://github.com/LucidDB/luciddb
Apache License 2.0
53 stars 24 forks source link

[FRG-219] FarragoUdxIterator.setDynamicParam ignores Calendar #653

Closed dynamobi-build closed 12 years ago

dynamobi-build commented 12 years ago

[reporter="jhyde", created="Mon, 9 Oct 2006 05:07:36 -0500 (GMT-05:00)"] setTimestamp(int,Timestamp) calls setDynamicParam(int,Timestamp,null), but should call setDynamicParam(int,Timestamp,Calendar.getInstance()). (JDBC says that setTimestamp with 2 parameters should do no timezone conversion, but setTimestamp with 3 parameters should convert to current local timezone if calendar is null.)

And setDynamicParam ignores the Calendar object passed in.

dynamobi-build commented 12 years ago

[author="jpham", created="Tue, 7 Nov 2006 12:31:44 -0500 (GMT-05:00)"] The fix for this seems to have made its way into the open branch as of eigenbase change 7965.

dynamobi-build commented 12 years ago

[author="jpham", created="Tue, 7 Nov 2006 12:33:13 -0500 (GMT-05:00)"] There is a test case in the farrago/expressions/udfInvocation.sql as the foreign_time Udx.