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.

Sign up for GitHub

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

Jump to bottom

Deprecating Facade Affiliations Processing #2173

Merged
merged 4 commits into from
Feb 14, 2023
Merged

Deprecating Facade Affiliations Processing #2173

merged 4 commits into from
Feb 14, 2023

Conversation

sgoggins
Copy link
Member

Description

  • The Facade task, which is used for commit counting in Augur, previously had two functions for "nuking" and "rebuilding" affiliation data inside the commit table. This is a feature no longer required, as it can be replaced with simpler logic, queries, and materialized views if needed. These were also computationally expensive tasks that added limited value.
  • Facade task version bumped to 1.3.0

ABrain7710 and others added 3 commits February 13, 2023 23:22
@ABrain7710
Fix machine learnig db connections and fix session passing
13c7eb1 
Signed-off-by: Andrew Brain <andrewbrain2019@gmail.com>
@sgoggins
Deprecated Facade nuke_affiliations and fix_empty_affiliations be…
eb3eadf 
…cause they no longer are necessary, and they cause performance issues.
@sgoggins
Update Facade Version.
f895b03
@sgoggins
Merge pull request #2174 from chaoss/db-improvements
71698e4 
Fix machine learnig db connections and fix session passing
@sgoggins
Copy link
Member Author

@ABrain7710 / @IsaacMilarky : I also just merged @ABrain7710 's database connection patch. I am testing with 8knot big data.

@sgoggins sgoggins added metric Related to metrics, new or otherwise CHAOSS Issues that relate directly to our goal of being a good reference implementation of CHAOSS metrics server Related to the Augur server release Related to releasing a new version of Augur labels Feb 14, 2023
@sgoggins sgoggins merged commit e20e2c1 into main Feb 14, 2023
@sgoggins sgoggins deleted the spg-patch-bba branch March 13, 2023 23:06
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@IsaacMilarky IsaacMilarky IsaacMilarky approved these changes

@ABrain7710 ABrain7710 Awaiting requested review from ABrain7710

Assignees
No one assigned
Labels
CHAOSS Issues that relate directly to our goal of being a good reference implementation of CHAOSS metrics metric Related to metrics, new or otherwise release Related to releasing a new version of Augur server Related to the Augur server
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
3 participants
@sgoggins @IsaacMilarky @ABrain7710