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

Remove redundant call to UpdateRepoStats during migration #18591

Merged
merged 7 commits into from
Feb 7, 2022

Commits on Feb 4, 2022

  1. remove redundant call to UpdateRepoStats during migration

    There is no need to call UpdateRepoStats in the InsertIssues and
    InsertPullRequests function. They are only called during migration by
    the CreateIssues and CreateReviews methods of the gitea uploader.
    
    The UpdateRepoStats function will be called by the Finish method of
    the gitea uploader after all reviews and issues are inserted. Calling
    it before is therefore redundant and the associated SQL requests are
    not cheap.
    
    The statistics tests done after inserting an issue or a pull request
    are also removed. They predate the implementation of UpdateRepoStats,
    back when the calculation of the statistics was an integral part of
    the migration function. The UpdateRepoStats is now tested
    independantly and these tests are no longer necessary.
    
    Signed-off-by: singuliere <[email protected]>
    singuliere committed Feb 4, 2022
    Configuration menu
    Copy the full SHA
    ef39703 View commit details
    Browse the repository at this point in the history

Commits on Feb 5, 2022

  1. Configuration menu
    Copy the full SHA
    4c19a1e View commit details
    Browse the repository at this point in the history

Commits on Feb 6, 2022

  1. Configuration menu
    Copy the full SHA
    874795e View commit details
    Browse the repository at this point in the history
  2. Configuration menu
    Copy the full SHA
    b22035d View commit details
    Browse the repository at this point in the history
  3. Configuration menu
    Copy the full SHA
    ffdb4ac View commit details
    Browse the repository at this point in the history
  4. Configuration menu
    Copy the full SHA
    e1a9fef View commit details
    Browse the repository at this point in the history

Commits on Feb 7, 2022

  1. Configuration menu
    Copy the full SHA
    ef074f2 View commit details
    Browse the repository at this point in the history