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

Make "immediate=True" the default for butler.get()

    XMLWordPrintable

    Details

    • Story Points:
      0.5
    • Sprint:
      Alert Production S17 - 2
    • Team:
      Alert Production

      Description

      Considering how problematic readProxy objects are to deal with, I feel it might be better to default to immediate=True for butler.get() calls. For any test code or interactive code, we want to get an immediately useable object, while inside the pipeline we can be more explicit about immediate=False when we realize that we're waiting on I/O.

        Attachments

          Issue Links

            Activity

            Hide
            rowen Russell Owen added a comment -

            A trivial change. One unit test required updating. I performed two Jenkins runs: 20990 (py2 including lsst_ci and ci_hsc) and 20991 (py3 lsst_py) and both passed.

            Show
            rowen Russell Owen added a comment - A trivial change. One unit test required updating. I performed two Jenkins runs: 20990 (py2 including lsst_ci and ci_hsc) and 20991 (py3 lsst_py) and both passed.
            Hide
            Parejkoj John Parejko added a comment -

            Looks fine, but is there any documentation that we need to change to reflect this new default?

            Show
            Parejkoj John Parejko added a comment - Looks fine, but is there any documentation that we need to change to reflect this new default?
            Hide
            rowen Russell Owen added a comment - - edited

            I found one doc string that needed updating, but nothing else. I also announced the change on community: https://community.lsst.org/t/butler-get-now-retrieves-immediately-by-default/1620

            Show
            rowen Russell Owen added a comment - - edited I found one doc string that needed updating, but nothing else. I also announced the change on community: https://community.lsst.org/t/butler-get-now-retrieves-immediately-by-default/1620

              People

              Assignee:
              rowen Russell Owen
              Reporter:
              Parejkoj John Parejko
              Reviewers:
              John Parejko
              Watchers:
              Jim Bosch, John Parejko, John Swinbank, Kian-Tat Lim, Krzysztof Findeisen, Russell Owen, Simon Krughoff
              Votes:
              1 Vote for this issue
              Watchers:
              7 Start watching this issue

                Dates

                Created:
                Updated:
                Resolved:

                  Jenkins Builds

                  No builds found.