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

Add some level of synchronization to the root of the API #283

Merged
merged 3 commits into from
Sep 9, 2017

Commits on Jun 8, 2016

  1. Add some level of synchronization to the root of the API

    This adds some synchronization to the maps at the root of the API to avoid duplicated calls to the actual GH REST API.  Specifically this is targeted around the two maps, orgs and users.  This fix makes the GHPRB jenkins plugin behave much better when there are lots of projects that could build for a specific repo (even if only a few are actually triggered)
    
    There are also a few fixes around GHUser and GHPullRequest
    * GHPullRequest was checking a field that may be null (merged_by) when determining whether to fetch details.  An unmerged PR would make a bunch of Github API calls for each property accessed.
    * Where GHUser was returned in various objects, we weren't going through the caching mechanism at the root, so calls to APIs on GHUSer often resulted in new REST calls.  Instead, return from the cache wherever possible.
    mmitche committed Jun 8, 2016
    Configuration menu
    Copy the full SHA
    9f3f644 View commit details
    Browse the repository at this point in the history

Commits on Aug 29, 2017

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