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

Replace PropertySet.get with getScalar or getArray

    XMLWordPrintable

    Details

      Description

      DM-19873 is changing the API of PropertySet.get() such that it will now return a default value rather than raising KeyError. Before that can happen, all extant calls to get() must be replaced to avoid any surprises.

      The approach I will take is to change get() so that it always throws an exception and then wait for tests to fail. This may lead to some remaining usage of get() in code that is not used.

        Attachments

          Issue Links

            Activity

            No builds found.
            tjenness Tim Jenness created issue -
            tjenness Tim Jenness made changes -
            Field Original Value New Value
            Epic Link DM-16576 [ 234914 ]
            tjenness Tim Jenness made changes -
            Link This issue has to be done before DM-19873 [ DM-19873 ]
            tjenness Tim Jenness made changes -
            Component/s obs_lsst [ 16504 ]
            Component/s pipe_drivers [ 13309 ]
            Hide
            tjenness Tim Jenness added a comment -

            Russell Owen I only needed to fix three packages. Would you mind doing a quick review?

            Show
            tjenness Tim Jenness added a comment - Russell Owen I only needed to fix three packages. Would you mind doing a quick review?
            tjenness Tim Jenness made changes -
            Reviewers Russell Owen [ rowen ]
            Status To Do [ 10001 ] In Review [ 10004 ]
            Hide
            rowen Russell Owen added a comment -

            Looks great and a nice simplification in the case of using getArray.

            Show
            rowen Russell Owen added a comment - Looks great and a nice simplification in the case of using getArray.
            rowen Russell Owen made changes -
            Status In Review [ 10004 ] Reviewed [ 10101 ]
            tjenness Tim Jenness made changes -
            Resolution Done [ 10000 ]
            Status Reviewed [ 10101 ] Done [ 10002 ]
            tjenness Tim Jenness made changes -
            Story Points 1

              People

              Assignee:
              tjenness Tim Jenness
              Reporter:
              tjenness Tim Jenness
              Reviewers:
              Russell Owen
              Watchers:
              Kian-Tat Lim, Russell Owen, Tim Jenness
              Votes:
              0 Vote for this issue
              Watchers:
              3 Start watching this issue

                Dates

                Created:
                Updated:
                Resolved:

                  Jenkins Builds

                  No builds found.