Skip to content

G
gitlab-foss

李少辉-开发者 / gitlab-foss

通知 15
Star 0
Fork 0
G
gitlab-foss

前往新版Gitcode，体验更适合开发者的 AI 搜索 >>

切换分支/标签
  1. 29 6月, 2016 28 次提交
  3. 28 6月, 2016 12 次提交
    • Y
      Use clock_gettime for all performance timestamps · d7b4f36a
      由 Yorick Peterse 提交于 
      Process.clock_gettime allows getting the real time in nanoseconds as
well as allowing one to get a monotonic timestamp. This offers greater
accuracy without the overhead of having to allocate a Time instance. In
general using Time.now/Time.new is about 2x slower than using
Process.clock_gettime(). For example:

    require 'benchmark/ips'

    Benchmark.ips do |bench|
      bench.report 'Time.now' do
        Time.now.to_f
      end

      bench.report 'clock_gettime' do
        Process.clock_gettime(Process::CLOCK_MONOTONIC, :millisecond)
      end

      bench.compare!
    end

Running this benchmark gives:

    Calculating -------------------------------------
                Time.now   108.052k i/100ms
           clock_gettime   125.984k i/100ms
    -------------------------------------------------
                Time.now      2.343M (± 7.1%) i/s -     11.670M
           clock_gettime      4.979M (± 0.8%) i/s -     24.945M

    Comparison:
           clock_gettime:  4979393.8 i/s
                Time.now:  2342986.8 i/s - 2.13x slower

Another benefit of using Process.clock_gettime() is that we can simplify
the code a bit since it can give timestamps in nanoseconds out of the
box.
      d7b4f36a
    • F
      Merge branch 'remove-gon-variable' into 'master' · 7a94432e
      由 Fatih Acet 提交于 
      Unused gon variable with very bad performance

## What does this MR do?

Remove an unused variable with a lot of performance penalty. It was opening a transaction just to return a constant string.

## Does this MR meet the acceptance criteria?

- [x] [CHANGELOG](https://gitlab.com/gitlab-org/gitlab-ce/blob/master/CHANGELOG) entry added

See merge request !4969
      7a94432e
    • G
      Remove redundant check in commit builds API endpoint · 2b8f04a6
      由 Grzegorz Bizon 提交于
      2b8f04a6
    • P
      Unused got variable with very bad performance · b5fa56eb
      由 Paco Guzman 提交于 
      In any case if just want the value which is always ‘gitlab’

require 'benchmark/ips'

Project.first # To load database things
GitlabIssueTrackerService.first # To load database things

Benchmark.ips do |x|
  x.config(:time => 5, :warmup => 2)

  x.report("current") do
    Project.new.default_issue_tracker.to_param
  end

  x.report("") do
    'gitlab'
  end

  x.compare!
end

Calculating -------------------------------------
             current     4.000  i/100ms
                        30.938k i/100ms
-------------------------------------------------
             current     47.298  (±10.6%) i/s -    232.000 
                          4.366M (±20.9%) i/s -     17.202M

Comparison:
                    :  4366456.0 i/s
             current:       47.3 i/s - 92318.26x slower
      b5fa56eb
    • A
      Merge branch 'iankelling-api-doc' into 'master' · 1ade080e
      由 Achilleas Pipinellis 提交于 
      docs: fix default_branch_protection default



See merge request !4479
      1ade080e
    • D
      Merge branch '19170-mobile-buttons' into 'master' · abc6004f
      由 Dmitriy Zaporozhets 提交于 
      Switched mobile button icons to ellipsis and angle

## What does this MR do?
Switches the mobile button icons

## What are the relevant issue numbers?
Closes #19170   
Part of #19200 

## Screenshots (if relevant)
![Screen_Shot_2016-06-27_at_9.08.28_AM](/uploads/7784489402e342e671d02b24d2ea0d64/Screen_Shot_2016-06-27_at_9.08.28_AM.png)


See merge request !4944
      abc6004f
    • A
      Update CHANGELOG · ca92ad90
      由 Alfredo Sumaran 提交于
      ca92ad90
    • A
      Switched mobile button icons to ellipsis and angle · 706c5a13
      由 Annabel Dunstone 提交于
      706c5a13
    • D
      Merge branch '19171-pinned-width' into 'master' · bbbd0e6c
      由 Dmitriy Zaporozhets 提交于 
      Resolve "Pin should show up at 1280px min"

## What does this MR do?
Decreased window min width for pinned sidebar

## What are the relevant issue numbers?
Closes  #19171    
Part of #19200 

## Screenshots (if relevant)
![Screen_Shot_2016-06-27_at_9.36.13_AM](/uploads/d0a87bca5af1bee808c5b1046c0ecf72/Screen_Shot_2016-06-27_at_9.36.13_AM.png)

See merge request !4947
      bbbd0e6c
    • K
      Merge branch 'use-git-depth' into 'master' · ec102992
      由 Kamil Trzciński 提交于 
      Use GIT_DEPTH for builds

## What does this MR do?
Enables experimental feature to use shallow cloning. 

## Why was this MR needed?
To speed up the builds and reduce the pressure on NFS servers. This should save us between 30s to 1m of the time of each build.

## More information
 `GIT_DEPTH`: https://gitlab.com/gitlab-org/gitlab-ci-multi-runner/merge_requests/188

## Problems
- Too small value for `GIT_DEPTH` can make it impossible to retry old changes.
You will see `unresolved reference` in build log.
We should then reconsider changing `GIT_DEPTH` to higher value
- Mechanism that rely on `git describe` may not work correctly when `GIT_DEPTH` is set. This will happen, because only part of the git history is present on the build machine

## Requirements
GitLab Runner 1.3.0. Currently all our internal runners use beta release or 1.3.0 with support for `GIT_DEPTH`.

@pcarranza Please decide when to merge that. We should start monitoring to see an impact on our NFS servers.

cc @jacobvosmaer-gitlab @pcarranza 


See merge request !4730
      ec102992
    • A
      Merge branch 'add-smtp-upgrade-info' into 'master' · 9315597a
      由 Achilleas Pipinellis 提交于 
      Add SMTP upgrade information

8.8 to 8.9 requires an update to the SMTP settings for source installs.

See merge request !4966
      9315597a
    • A
      Update CHANGELOG · b262e240
      由 Annabel Dunstone 提交于
      b262e240