castor
  1. castor
  2. CASTOR-3086

SQLRelationLoader: persistence.sql.query classes, CastorStatement and CastorConnection

    Details

    • Type: Improvement Improvement
    • Status: Resolved Resolved
    • Priority: Major Major
    • Resolution: Fixed
    • Affects Version/s: None
    • Fix Version/s: 1.3.3rc1
    • Component/s: JDO queries
    • Labels:
      None
    • Number of attachments :
      4

      Description

      Refactoring of SQLRelationLoader:

      • Use of persistence.sql.query classes to represent constructed queries.
      • Introduce CastorStatement and CastorConnection classes to handle query construction (query visitors).
      1. 3086_19052001.txt
        18 kB
        Dennis Butterstein
      2. patch-SQLRelationLoader-20110513-alpha.txt
        14 kB
        Johannes Venzke
      3. patch-SQLRelationLoader-20110516-beta.txt
        15 kB
        Johannes Venzke
      4. patch-SQLRelationLoader-20110516-SQLColumnInfo.txt
        17 kB
        Johannes Venzke

        Activity

        Hide
        Johannes Venzke added a comment -

        Used persistence.sql.query classes to represent constructed queries and used "name-binded" parameters in a first step.

        Show
        Johannes Venzke added a comment - Used persistence.sql.query classes to represent constructed queries and used "name-binded" parameters in a first step.
        Hide
        Johannes Venzke added a comment -

        Replaced e.printStackTrace() with LOG entries

        Show
        Johannes Venzke added a comment - Replaced e.printStackTrace() with LOG entries
        Hide
        Johannes Venzke added a comment -

        I had the idea to use SQLColumnInfo class.
        I would change the constructor signature, if issue 3073 would have been resolved.

        Show
        Johannes Venzke added a comment - I had the idea to use SQLColumnInfo class. I would change the constructor signature, if issue 3073 would have been resolved.
        Hide
        Dennis Butterstein added a comment -

        Merged patch with recent changes to SQLRelationLoader. Removed parameter PersistenceFactory from SQLRelationLoader.

        Show
        Dennis Butterstein added a comment - Merged patch with recent changes to SQLRelationLoader. Removed parameter PersistenceFactory from SQLRelationLoader.

          People

          • Assignee:
            Johannes Venzke
            Reporter:
            Johannes Venzke
          • Votes:
            0 Vote for this issue
            Watchers:
            0 Start watching this issue

            Dates

            • Created:
              Updated:
              Resolved: