Ok I work on a patch.

One Problem is that oracle needs an explicid datepattern. 
In the SQL-Encoder are given 3.
Can I reduce this for Oracle to yyyy-mm-dd only?

- Tim

-----Ursprüngliche Nachricht-----
Von: Andrea Aime [mailto:[EMAIL PROTECTED]
Gesendet: Donnerstag, 21. Juni 2007 11:52
An: Tim Englich
Cc: [EMAIL PROTECTED]; [email protected]
Betreff: Re: OracleDataStore querying Date-Columns


Tim Englich ha scritto:

> I habe found a problem in the OracleDatastore when I try to query a
> Date-Column.
> I got an Oracle-Exception.
> I take a look at the use SQL-Stament an saw that there is used the function
> date(...) inteatd of to_date() which is unknown in Oracle DBMS.
> It is not possible to create this Function in the Oracle DBMS because it is
> an protected keyword.
> 
> So I take a look at the Sourcecode and noticed that the Method
> org.geotools.filter.SQLEncoder.visit(LiteralExpression expression)  has to
be
> modified to work with Oracle.
> 
> I think about creating an Extention of SQLEncoder  for the OracleDatastore
> which pays attention to the specific Syntax of Oracle.
> When I looked at the JDBC1Datastore I noticed that there is an Method
called
> getSqlBuilder(...).
> I think we have to override it in the OracleDataStore an put there the
> Extention of SQLEncoder to the SqlBuilder. 
> Am I right?

Tim, you're right, the extension should be placed in SqlEncoderOracle (a 
class that's already there).
Can you cook up a fix, and do the usual stuff? (open a jira issue, 
attach a patch, if you have time, add a unit test as well?).

Cheers
Andrea

-------------------------------------------------------------------------
This SF.net email is sponsored by DB2 Express
Download DB2 Express C - the FREE version of DB2 express and take
control of your XML. No limits. Just data. Click to get it now.
http://sourceforge.net/powerbar/db2/
_______________________________________________
Geotools-gt2-users mailing list
[email protected]
https://lists.sourceforge.net/lists/listinfo/geotools-gt2-users

Reply via email to