Uploaded image for project: 'Kuali Rice Development'
  1. Kuali Rice Development
  2. KULRICE-6704

reuse ObjectMapper instance in DocumentSearchInternalUtils for better performance

    Details

    • Type: Improvement
    • Status: Closed
    • Priority: Major
    • Resolution: Fixed
    • Affects Version/s: None
    • Fix Version/s: 2.0.0-rc3, 2.0
    • Component/s: Development, Performance
    • Security Level: Public (Public: Anyone can view)
    • Labels:
      None
    • Rice Module:
      KEW
    • KAI Review Status:
      Not Required
    • KTI Review Status:
      Not Required

      Description

      ObjectMapper caches stuff internally, so reusing it makes performance much better for.

      Profiling doc search, we noticed that ObjectMapper.readValue (which is called inside DocumentSearchServiceImpl.getCriteriaFromSavedSearch inside DocumentSearchInternalUtils) seemed to be very slow. Some experimentation shows that reusing it was much faster.

        Attachments

          Activity

          gilesp Peter Giles (Inactive) created issue -
          gilesp Peter Giles (Inactive) made changes -
          Field Original Value New Value
          Status Open [ 1 ] Resolved [ 5 ]
          Resolution Fixed [ 1 ]
          Hide
          jcoltrin Jessica Coltrin (Inactive) added a comment -

          Closing since this item is now in the 2.0 release notes.

          Show
          jcoltrin Jessica Coltrin (Inactive) added a comment - Closing since this item is now in the 2.0 release notes.
          jcoltrin Jessica Coltrin (Inactive) made changes -
          Status Resolved [ 5 ] Closed [ 6 ]
          spatterson Shem Patterson (Inactive) made changes -
          Workflow custom [ 133339 ] Copy of custom for rice [ 213839 ]
          spatterson Shem Patterson (Inactive) made changes -
          Workflow Copy of custom for rice [ 213839 ] custom [ 223587 ]
          spatterson Shem Patterson (Inactive) made changes -
          Workflow custom [ 223587 ] Rice Workflow [ 233335 ]

            People

            • Assignee:
              gilesp Peter Giles (Inactive)
              Reporter:
              gilesp Peter Giles (Inactive)
            • Votes:
              0 Vote for this issue
              Watchers:
              0 Start watching this issue

              Dates

              • Created:
                Updated:
                Resolved: