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

Move Record printing to C++

    XMLWordPrintable

    Details

    • Type: Story
    • Status: Done
    • Resolution: Done
    • Fix Version/s: None
    • Component/s: afw
    • Labels:
      None
    • Story Points:
      0.5
    • Sprint:
      DRP S18-4
    • Team:
      Data Release Production

      Description

      Current implementation is apparently super slow, and looking at what it's doing (constructing a new dictionary of all fields via extract just to get one element from it, for every field), I can see why.

      That could be cleaned up in Python but a lot of it is more naturally and definitely more efficient in C++, and doing that will give us a C++ operator<<, too, which is nice.

        Attachments

          Activity

          Hide
          jbosch Jim Bosch added a comment -

          Nate Lust, mind looking at this small move of code from Python to C++?

           

          Show
          jbosch Jim Bosch added a comment - Nate Lust , mind looking at this small move of code from Python to C++?  
          Hide
          nlust Nate Lust added a comment -

          comment up on github, curious about your take, will mark as reviewed, but please respond to the comment before merge.

          Show
          nlust Nate Lust added a comment - comment up on github, curious about your take, will mark as reviewed, but please respond to the comment before merge.
          Hide
          jbosch Jim Bosch added a comment -

          Merged after implementing Nate Lust's suggestion.

          Show
          jbosch Jim Bosch added a comment - Merged after implementing Nate Lust 's suggestion.

            People

            Assignee:
            jbosch Jim Bosch
            Reporter:
            jbosch Jim Bosch
            Reviewers:
            Nate Lust
            Watchers:
            Jim Bosch, Nate Lust
            Votes:
            0 Vote for this issue
            Watchers:
            2 Start watching this issue

              Dates

              Created:
              Updated:
              Resolved:

                Jenkins

                No builds found.