Uploaded image for project: 'Data Management'
  1. Data Management
  2. DM-7226

Allow DbStorage to be a modern python iterator

    XMLWordPrintable

    Details

    • Type: Improvement
    • Status: Done
    • Resolution: Done
    • Fix Version/s: None
    • Component/s: daf_persistence
    • Labels:
      None
    • Story Points:
      0.1
    • Team:
      Architecture

      Description

      In DM-7069 it was noticed that the DbStorage could not be called as a modern iterator via the next function. Turns out that this is purely because there was not _next_ method. This ticket fixes that problem and updates the tests.

        Attachments

          Issue Links

          relates to

          Story - Created by Jira Software - do not edit or delete. Issue type for a user story. DM-7069 Port daf_persistence to Python 3

          • Done

          Story - Created by Jira Software - do not edit or delete. Issue type for a user story. DM-7269 Port ctrl_provenance to Python 3 and update unit tests for pytest

          • Done

          Improvement - An improvement or enhancement to an existing feature or task. DM-7323 Please rename DbStorage.next to something less confusing

          • Invalid

            Activity

            Ascending order - Click to sort in descending order
            No builds found.
            tjenness Tim Jenness created issue -
            tjenness Tim Jenness made changes -
            Field Original Value New Value
            Link This issue relates to DM-7069 [ DM-7069 ]
            Hide
            Permalink
            tjenness Tim Jenness added a comment -

            Minor fixes the iterator problem we discussed in DM-7069.

            Show
            tjenness Tim Jenness added a comment - Minor fixes the iterator problem we discussed in DM-7069 .
            tjenness Tim Jenness made changes -
            Reviewers Nate Pease [ npease ]
            Status To Do [ 10001 ] In Review [ 10004 ]
            npease Nate Pease [X] (Inactive) made changes -
            Status In Review [ 10004 ] Reviewed [ 10101 ]
            Hide
            Permalink
            tjenness Tim Jenness added a comment -

            Thanks. Merged.

            Show
            tjenness Tim Jenness added a comment - Thanks. Merged.
            tjenness Tim Jenness made changes -
            Resolution Done [ 10000 ]
            Status Reviewed [ 10101 ] Done [ 10002 ]
            rowen Russell Owen made changes -
            Link This issue relates to DM-7269 [ DM-7269 ]
            tjenness Tim Jenness made changes -
            Link This issue relates to DM-7323 [ DM-7323 ]

              People

              Assignee:
              tjenness Tim Jenness
              Reporter:
              tjenness Tim Jenness
              Reviewers:
              Nate Pease [X] (Inactive)
              Watchers:
              Nate Pease [X] (Inactive), Tim Jenness
              Votes:
              0 Vote for this issue
              Watchers:
              2 Start watching this issue

                Dates

                Created:
                Updated:
                Resolved:

                  Jenkins

                  No builds found.