Uploaded image for project: 'Coin'
  1. Coin
  2. COIN-528

hide resolve[d]Repository() and ensureRepoExists() from public API

    XMLWordPrintable

Details

    • Suggestion
    • Resolution: Won't Do
    • P4: Low
    • None
    • None
    • Scheduler
    • None

    Description

      Git calls happen all over the place in Coin, and they involve heavy tooling around them. Three functions that are spread all over the place and account for thousands of the git calls, are

      • resolveRepository()
      • resolvedRepository()
      • ensureRepoExists()

      They all belong to class RepositoryManager. What would it take to have this class acting as a separate service, and invoking resolveRepository() and ensureRepoExists() only in __init__() while hiding it from externals?

      This ticket is for coming up with a feasible and worthwhile refactoring for this mess, since it affects performance and readability of the code.

      Attachments

        Issue Links

          No reviews matched the request. Check your Options in the drop-down menu of this sections header.

          Activity

            People

              tosaario Toni Saario
              jimis Dimitrios Apostolou
              Votes:
              0 Vote for this issue
              Watchers:
              1 Start watching this issue

              Dates

                Created:
                Updated:
                Resolved:

                Gerrit Reviews

                  There are no open Gerrit changes