Maniphest T208900

LYBUNT report load issue
Closed, ResolvedPublic4 Estimated Story Points
Actions

Description

The LYBUNT: MG CY2018 report in Civi doesn't seem to load. We use this report to review for outreach at year-end.

report/instance/125?reset=1&output=criteria

CREATE TEMPORARY TABLE civicrm_tmp_e_rptlybunt_20181107_5be2016904034 DEFAULT CHARACTER SET utf8 COLLATE utf8_unicode_ci

SELECT SQL_CALC_FOUND_ROWS contact_civireport.id as cid  FROM civicrm_tmp_e_rptgrp_20181107_5be20168ee838 group_temp_table INNER JOIN civicrm_contact contact_civireport
  ON group_temp_table.id = contact_civireport.id    INNER JOIN civicrm_contribution contribution_civireport  ON contribution_civireport.contact_id = contact_civireport.id
   AND contribution_civireport.is_test = 0
   AND contribution_civireport.receive_date BETWEEN '20170101000000' AND '20171231235959'
 

 LEFT JOIN civicrm_contribution cont_exclude ON cont_exclude.contact_id = contact_civireport.id
   AND cont_exclude.receive_date BETWEEN '2018-01-01' AND '20181231235959' WHERE cont_exclude.id IS NULL AND 1 AND 1
GROUP BY contact_civireport.id

Details

Subject	Repo	Branch	Lines +/-
Lybunt Clarify Row count reporting on calculated rows	wikimedia/fundraising/crm/civicrm	master	+42 -5
dev/core#518 Fix lybunt performance	wikimedia/fundraising/crm/civicrm	master	+6 -4
Lybunt report - improve developer support for debugging this report	wikimedia/fundraising/crm/civicrm	master	+9 -12

Customize query in gerrit

Event Timeline

• LeanneS created this task.Nov 6 2018, 9:13 PM

Restricted Application added a subscriber: Aklapper. · View Herald TranscriptNov 6 2018, 9:13 PM

• LeanneS updated the task description. (Show Details)Nov 6 2018, 9:16 PM

• LeanneS updated the task description. (Show Details)

CaitVirtue subscribed.Nov 6 2018, 9:32 PM

Eileenmcnaughton moved this task from Backlog to UserRequests (misc) on the Wikimedia-Fundraising-CiviCRM board.Nov 13 2018, 7:48 PM

• DStrine moved this task from Triage to Current Sprint on the Fundraising-Backlog board.Nov 13 2018, 9:05 PM

• DStrine added a project: Fundraising Sprint Window dressing is mostly olive oil.Nov 13 2018, 9:15 PM

I'm still not able to make this report work. @LeanneS does it work for you?

@CaitVirtue it was pulled into the sprint today so I don't expect it to work yet - but will start on it now (since I was just deciding what to work on next)

Hmm - so it seems to work for other years not this year. Bug is probably that we have accepted too many donations - we should stop that :-)

Well more seriously - this isn't a regression per se - we just outgrew what used to work. I've done some analysis here https://lab.civicrm.org/dev/core/issues/518 and I think I can make the query much more efficient. I wrote a bunch of unit tests & did some code cleanup on this report a couple of years back so that will help now

Change 473322 had a related patch set uploaded (by Eileen; owner: Eileen):
[wikimedia/fundraising/crm/civicrm@master] Lybunt report - improve developer support for debugging this report

https://gerrit.wikimedia.org/r/473322

gerritbot added a project: Patch-For-Review.Nov 14 2018, 3:31 AM

Eileenmcnaughton claimed this task.Nov 14 2018, 4:50 AM

Eileenmcnaughton moved this task from Backlog to Doing on the Fundraising Sprint Window dressing is mostly olive oil board.

Change 473623 had a related patch set uploaded (by Eileen; owner: Eileen):
[wikimedia/fundraising/crm/civicrm@master] dev/core#518 Fix lybunt performance

https://gerrit.wikimedia.org/r/473623

Eileenmcnaughton moved this task from Doing to Review on the Fundraising Sprint Window dressing is mostly olive oil board.Nov 15 2018, 2:24 AM

Change 473654 had a related patch set uploaded (by Eileen; owner: Eileen):
[wikimedia/fundraising/crm/civicrm@master] Lybunt Clarify Row count reporting on calculated rows

https://gerrit.wikimedia.org/r/473654

Change 473322 merged by jenkins-bot:
[wikimedia/fundraising/crm/civicrm@master] Lybunt report - improve developer support for debugging this report

https://gerrit.wikimedia.org/r/473322

Change 473623 merged by jenkins-bot:
[wikimedia/fundraising/crm/civicrm@master] dev/core#518 Fix lybunt performance

https://gerrit.wikimedia.org/r/473623

@CaitVirtue @LeanneS I just pushed a fix - give the Lybunt another go