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

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

    XMLWordPrintable

Details

    • Task
    • Status: Closed
    • Should
    • Resolution: Won't Fix
    • OpenMRS 1.9.0
    • None
    • None
    • 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!!!

      Gliffy Diagrams

        Attachments

          Issue Links

            relates to

            Epic - Created by Jira Software - do not edit or delete. Issue type for a big user story that needs to be broken down. 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)

            Epic - Created by Jira Software - do not edit or delete. Issue type for a big user story that needs to be broken down. TRUNK-6087 COPY: 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.
            • Needs Assessment

            Epic - Created by Jira Software - do not edit or delete. Issue type for a big user story that needs to be broken down. TRUNK-6108 COPY: 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.
            • Needs Assessment

            Activity

              People

                Unassigned Unassigned
                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