Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Research ofec_all_report_mv #3230

Closed
2 tasks done
jwchumley opened this issue Jun 22, 2018 · 3 comments
Closed
2 tasks done

Research ofec_all_report_mv #3230

jwchumley opened this issue Jun 22, 2018 · 3 comments
Assignees
Milestone

Comments

@jwchumley
Copy link
Contributor

jwchumley commented Jun 22, 2018

Problem: There are multiple endpoints that assume a committee will file the correct form type. i.e. a candidate committee will file form 3. When a committee files on the wrong form type these endpoints miss that report.

Solution:
Rather than fix each endpoint we want to create a single view based on the v_sum table that has all form types so it can be used in all the relevant end points.

Things to consider:

Only the columns currently in v_sum handle paper amendments that may be missing values in some or all summary columns (not required by the regs to give us anything that has not changed). So adding more columns from the report tables is risky.

The endpoint used by the candidate and committee profile pages returns different columns depending on committee type. We need to make sure the new view supports all of the committee profile presentations.

For more detail: #3154

  • Have a meeting - Jeff, Helen, Paul, Laura, DBAs
  • Figure out columns
@lbeaufort lbeaufort modified the milestones: Sprint 6.4, Sprint 6.3 Jun 26, 2018
@lbeaufort lbeaufort changed the title Create ofec_all_reports_mv Research, Meeting, Plan - Create ofec_all_report_mv Jun 26, 2018
@PaulClark2 PaulClark2 modified the milestones: Sprint 6.3, Sprint 6.4 Jul 3, 2018
@hcaofec hcaofec modified the milestones: Sprint 6.4, Sprint 6.5 Jul 20, 2018
@hcaofec
Copy link
Contributor

hcaofec commented Aug 2, 2018

we decide to create a new materialized view using current fec_v_sum...vw to include all reports. At phase one, this mv will include F3, F3P and F3X. But we are aware of the possibility of need for adding more forms to this mv.

@hcaofec
Copy link
Contributor

hcaofec commented Aug 31, 2018

After consulting with DBA team, we will create view instead of materialized view if performance is good. View does not need to be refreshed as materialized view

@hcaofec hcaofec modified the milestones: bugs August 2018, Sprint 7.1 Aug 31, 2018
@patphongs patphongs changed the title Research, Meeting, Plan - Create ofec_all_report_mv Create ofec_all_report_mv Sep 5, 2018
@hcaofec hcaofec changed the title Create ofec_all_report_mv Research / Create ofec_all_report_mv Sep 12, 2018
@hcaofec hcaofec mentioned this issue Sep 12, 2018
8 tasks
@hcaofec
Copy link
Contributor

hcaofec commented Sep 12, 2018

A view was created initially but the time to load the view is getting longer with more columns added to it. So a materialized view with same structure will be created.

New issue is here: #3373

@hcaofec hcaofec closed this as completed Sep 12, 2018
@hcaofec hcaofec changed the title Research / Create ofec_all_report_mv Research ofec_all_report_mv Sep 12, 2018
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

4 participants