Uploaded image for project: 'OpenMRS Core'
  1. OpenMRS Core
  2. TRUNK-3227

Get rid of the potentially-expensive call to getAllPatients()

    XMLWordPrintable

    Details

    • Type: Task
    • Status: Closed
    • Priority: Should
    • Resolution: Won't Fix
    • Affects Version/s: OpenMRS 1.9.0
    • Fix Version/s: None
    • Component/s: None
    • Labels:
    • Complexity:
      Medium

      Description

      TODO: get rid of the potentially-expensive call to getAllPatients()
      File: /openmrs/api/src/main/java/org/openmrs/api/db/hibernate/HibernatePatientSetDAO.java
      Line: 2127

      Since members of a relationship aren't necessarily Patients, but we're supposed to be returning
      just patients, all patients are fetched to compare against.
      We therefore need a smarter solution which does not require loading all patients!!!

        Attachments

          Issue Links

          relates to

          Bug - A problem which impairs or prevents the functions of the product. TRUNK-324 Go through the TODO items code and create tickets out of them

          • Should - Issue should be completed for its milestone; however, will not delay the milestone.
          • Code Review (Initial)

            Activity

              People

              Assignee:
              Unassigned
              Reporter:
              nullakilla Anton Alekseiev
              Votes:
              0 Vote for this issue
              Watchers:
              6 Start watching this issue

                Dates

                Created:
                Updated:
                Resolved:

                  Time Tracking

                  Estimated:
                  Original Estimate - 2 hours
                  2h
                  Remaining:
                  Remaining Estimate - 2 hours
                  2h
                  Logged:
                  Time Spent - Not Specified
                  Not Specified