BigW Consortium Gitlab

  1. 19 Jan, 2017 1 commit
  2. 17 Nov, 2016 3 commits
  3. 21 Sep, 2016 1 commit
  4. 20 Sep, 2016 5 commits
    • Implement a second round of review comments from @DouweM. · 918e589c
      Timothy Andrew authored
      - Don't use `TableReferences` - using `.arel_table` is shorter!
      - Move some database-related code to `Gitlab::Database`
      - Remove the `MergeRequest#issues_closed` and
        `Issue#closed_by_merge_requests`  associations. They were either
        shadowing or were too similar to existing methods. They are not being
        used anywhere, so it's better to remove them to reduce confusion.
      - Use Rails 3-style validations
      - Index for `MergeRequest::Metrics#first_deployed_to_production_at`
      - Only include `CycleAnalyticsHelpers::TestGeneration` for specs that
        need it.
      - Other minor refactorings.
    • Fix rubocop spec. · 231a9f5b
      Timothy Andrew authored
      And `scss_lint`
    • Implement (some) comments from @DouweM's review. · 71d4bf72
      Timothy Andrew authored
      - Move things common to `Issue` and `MergeRequest` into `Issuable`
      - Move more database-specific functions into `Gitlab::Database`
      - Indentation changes and other minor refactorings.
    • Implement review comments from @yorickpeterse · 8957293d
      Timothy Andrew authored
      1. Change multiple updates to a single `update_all`
      
      2. Use cascading deletes
      
      3. Extract an average function for the database median.
      
      4. Move database median to `lib/gitlab/database`
      
      5. Use `delete_all` instead of `destroy_all`
      
      6. Minor refactoring
  5. 16 Sep, 2016 1 commit
  6. 15 Jul, 2016 2 commits
  7. 17 Jun, 2016 1 commit
    • Fix update_column_in_batches to update all rows · c1e756c2
      Yorick Peterse authored
      This changes update_column_in_batches to ensure it always updates all
      rows now. These changes also allow for an extra SELECT query to be
      removed, nor does it use the row count for determining offsets and the
      likes; instead it's only used to determine the batch size.
  8. 16 Jun, 2016 2 commits
  9. 15 Jun, 2016 2 commits
    • Don't update columns in batches in a transaction · 816c4535
      Yorick Peterse authored
      This ensures that whatever locks are acquired aren't held onto until the
      end of the transaction (= after _all_ rows have been updated). Timing
      wise there's also no difference between using a transaction and not
      using one.
    • Customizing of update_column_in_batches queries · 8966263e
      Yorick Peterse authored
      By passing a block to update_column_in_batches() one can now customize
      the queries executed. This in turn can be used to only update a specific
      set of rows instead of simply all the rows in the table.
  10. 13 Jun, 2016 3 commits
  11. 06 Jun, 2016 1 commit
  12. 03 Jun, 2016 3 commits
  13. 22 May, 2016 2 commits
  14. 19 May, 2016 2 commits
  15. 12 May, 2016 3 commits