Hi Mike, Your comment is accepted. A general solution for not placing extra "( )" for DISTINCT/ALL/ANY is filed under OPENJPA-607 and resolved.
Catalina On 5/21/08, Michael Dick (JIRA) <[EMAIL PROTECTED]> wrote: > > > [ > https://issues.apache.org/jira/browse/OPENJPA-606?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=12598604#action_12598604] > > Michael Dick commented on OPENJPA-606: > -------------------------------------- > > It seems that we can move some of the logic in Distinct.java into the > DBDictionary. Or at least add a flag in the DBDictionary that indicates > whether ( ) are needed. Leaving : > if > (sel.getConfiguration().getDBDictionaryInstance().platform.indexOf("Informix") > > -1) { > > in the code is rather ugly. > > > InformixDictionary default setting beaks many testcases > > ------------------------------------------------------- > > > > Key: OPENJPA-606 > > URL: https://issues.apache.org/jira/browse/OPENJPA-606 > > Project: OpenJPA > > Issue Type: Bug > > Components: sql > > Affects Versions: 1.2.0 > > Reporter: Catalina Wei > > > > In testing Informix backend store, there are many query test string > failed: > > 1. parameter of a boolean value does not work, should use 't for true, > 'f' for false > > WHERE (t1.isManager = ?) [params=(boolean) false] > > > > should generate WHERE (t1.isManager = ?) [params=(String) f] > > 2. SELECT COUNT(DISTINCT(t1.name)) reported syntax error [code=-201, > state=42000] > > should generate COUNT(DISTINCT t1.name ) > > 3. CROSS JOIN syntax error > > > > should generate JOIN with ON 1 = 1 > > > > 4. There is no equivalent function for LOCATE. Informix users must > create INSTR as a user defined function. > > 3. CONCAT function in '||' does not take parameter markers > > WHERE ((?||t0.name) LIKE ? ESCAPE '\') > > should generate CONCAT(?, t0.name) > > > -- > This message is automatically generated by JIRA. > - > You can reply to this email to add a comment to the issue online. > >
