Details

    • Type: Sub-task Sub-task
    • Status: Resolved Resolved
    • Priority: Major Major
    • Resolution: Fixed
    • Affects Version/s: 1.3.2
    • Fix Version/s: 1.3.3rc1
    • Component/s: None
    • Labels:
      None
    • Number of attachments :
      9

      Description

      Use ColumnInfo classes to bind parameters in SQLRelationLoader.

      1. patch-20110527.txt
        15 kB
        Johannes Venzke
      2. patch-C3096-20110607-createRelationResolver.txt
        18 kB
        Ralf Joachim
      3. patch-C3096-20110615.txt
        55 kB
        Ralf Joachim
      4. patch-SQLRelationLoader-ColumnInfo-110530.txt
        19 kB
        Johannes Venzke
      5. patch-SQLRelationLoader-ColumnInfo-110531.txt
        24 kB
        Johannes Venzke
      6. patch-SQLRelationLoader-ColumnInfo-110601.txt
        26 kB
        Johannes Venzke
      7. patch-SQLRelationLoader-ColumnInfo-110603.txt
        29 kB
        Johannes Venzke
      8. patch-SQLRelationLoader-ColumnInfo-110606.txt
        29 kB
        Johannes Venzke
      9. patch-SQLRelationLoader-ColumnInfo-110610-final.txt
        39 kB
        Johannes Venzke

        Activity

        Hide
        Johannes Venzke added a comment -

        Recreated patch for current SVN state and advanced it.

        Show
        Johannes Venzke added a comment - Recreated patch for current SVN state and advanced it.
        Hide
        Ralf Joachim added a comment -

        Patch that shows how SQLRelationResolver should be created in SQLEngine.createSQLRelationResolver().
        I've check debug output of cpactf run which showed that all informations of RelationTableInfo are correct.

        Show
        Ralf Joachim added a comment - Patch that shows how SQLRelationResolver should be created in SQLEngine.createSQLRelationResolver(). I've check debug output of cpactf run which showed that all informations of RelationTableInfo are correct.
        Hide
        Johannes Venzke added a comment -

        Everything is done and works now. This seems to be the final patch.
        One thing one could do is to let Identity implement Iterable which would make the code more elegant. But this would be a new issue.

        Show
        Johannes Venzke added a comment - Everything is done and works now. This seems to be the final patch. One thing one could do is to let Identity implement Iterable which would make the code more elegant. But this would be a new issue.
        Hide
        Johannes Venzke added a comment -

        Date pf patch file was wrong.

        Show
        Johannes Venzke added a comment - Date pf patch file was wrong.
        Hide
        Ralf Joachim added a comment -

        Final patch with some minor modifications to the one from Johannes.

        Show
        Ralf Joachim added a comment - Final patch with some minor modifications to the one from Johannes.

          People

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

            Dates

            • Created:
              Updated:
              Resolved: