OpenJPA
  1. OpenJPA
  2. OPENJPA-606

InformixDictionary default setting beaks many testcases

    Details

    • Type: Bug Bug
    • Status: Closed
    • Priority: Major Major
    • Resolution: Fixed
    • Affects Version/s: 1.2.0
    • Fix Version/s: 1.0.4, 1.2.0
    • Component/s: sql
    • Labels:
      None

      Description

      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)

        Issue Links

          Activity

          Hide
          Michael Dick added a comment -

          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.

          Show
          Michael Dick added a comment - 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.
          Hide
          Catalina Wei added a comment -

          resolved in r659731

          Show
          Catalina Wei added a comment - resolved in r659731

            People

            • Assignee:
              Unassigned
              Reporter:
              Catalina Wei
            • Votes:
              0 Vote for this issue
              Watchers:
              0 Start watching this issue

              Dates

              • Created:
                Updated:
                Resolved:

                Development