GeoTools
  1. GeoTools
  2. GEOT-2232

Have PreparedFilterToSql delegate to the dialect for geometry placeholder generation

    Details

    • Type: Improvement Improvement
    • Status: Closed Closed
    • Priority: Major Major
    • Resolution: Fixed
    • Affects Version/s: 2.5.2
    • Fix Version/s: 2.5.3
    • Component/s: jdbc
    • Labels:
      None

      Description

      If a function must be used to encode the placeholder, the dialect is the one that knows how

        Activity

        Hide
        Andrea Aime added a comment -
        Christian, I believe this one is of interest for you too, I guess you can avoid some of the code in your DB2 FilterToSql class
        Show
        Andrea Aime added a comment - Christian, I believe this one is of interest for you too, I guess you can avoid some of the code in your DB2 FilterToSql class
        Hide
        Andrea Aime added a comment -
        Committed on 2.5.x and trunk, keeping open waiting for the DB2 land feedback
        Show
        Andrea Aime added a comment - Committed on 2.5.x and trunk, keeping open waiting for the DB2 land feedback
        Hide
        Christian Mueller added a comment -
        Works for DB2 land, I could remove some code from the DB2 FilterToSql class. But I am still waiting for http://jira.codehaus.org/browse/GEOT-2212 , this issue forces me to do nasty things in the dialect.

        Anyway, you can close this issue
        Show
        Christian Mueller added a comment - Works for DB2 land, I could remove some code from the DB2 FilterToSql class. But I am still waiting for http://jira.codehaus.org/browse/GEOT-2212 , this issue forces me to do nasty things in the dialect. Anyway, you can close this issue
        Hide
        Andrea Aime added a comment -
        Roger about closing the issue. You should ask again Justin about the other issue ;)
        Show
        Andrea Aime added a comment - Roger about closing the issue. You should ask again Justin about the other issue ;)

          People

          • Assignee:
            Andrea Aime
            Reporter:
            Andrea Aime
          • Votes:
            0 Vote for this issue
            Watchers:
            1 Start watching this issue

            Dates

            • Created:
              Updated:
              Resolved: