Skip to:
Content

Changes between Initial Version and Version 1 of Ticket #7442, comment 6


Ignore:
Timestamp:
04/12/17 19:50:30 (9 months ago)
Author:
r-a-y
Comment:

Legend:

Unmodified
Added
Removed
Modified
  • Ticket #7442, comment 6

    initial v1  
    33`bp-search-optimize.patch` passes existing unit tests.  The rationale behind the "query for IDs first" approach makes sense to me. 
    44 
    5 Looks good to me, but needs sign off from a lead dev.  We might need some new unit tests to cover this change though. 
     5Looks good, but needs sign off from a lead dev.  We might need some new unit tests to cover this change though.