An error occurred fetching the project authors.
- 16 Nov, 2015 1 commit
-
-
Kamil Trzcinski authored
-
- 13 Nov, 2015 1 commit
-
-
Dmitriy Zaporozhets authored
Signed-off-by: Dmitriy Zaporozhets <dmitriy.zaporozhets@gmail.com>
-
- 30 Oct, 2015 4 commits
-
-
Yorick Peterse authored
This further improves performance of User.find_by_any_email and is roughly twice as fast as the previous UNION setup. Thanks again to @dlemstra for suggesting this.
-
Yorick Peterse authored
MySQL doesn't support the previous syntax.
-
Yorick Peterse authored
This is significantly faster than using a sub-query, at least when run on the GitLab.com production database. The benchmarks are a lot slower now with these changes, most likely due to PostgreSQL choosing a different (and less efficient) plan based on the amount of data present in the test database. Thanks to @dlemstra for suggesting the use of a UNION.
-
Yorick Peterse authored
This query used to rely on a JOIN, effectively producing the following SQL: SELECT users.* FROM users LEFT OUTER JOIN emails ON emails.user_id = users.id WHERE (users.email = X OR emails.email = X) LIMIT 1; The use of a JOIN means having to scan over all Emails and users, join them together and then filter out the rows that don't match the criteria (though this step may be taken into account already when joining). In the new setup this query instead uses a sub-query, producing the following SQL: SELECT * FROM users WHERE id IN (select user_id FROM emails WHERE email = X) OR email = X LIMIT 1; This query has the benefit that it: 1. Doesn't have to JOIN any rows 2. Only has to operate on a relatively small set of rows from the "emails" table. Since most users will only have a handful of Emails associated (certainly not hundreds or even thousands) the size of the set returned by the sub-query is small enough that it should not become problematic. Performance of the old versus new version can be measured using the following benchmark: # Save this in ./bench.rb require 'benchmark/ips' email = 'yorick@gitlab.com' def User.find_by_any_email_old(email) user_table = arel_table email_table = Email.arel_table query = user_table. project(user_table[Arel.star]). join(email_table, Arel::Nodes::OuterJoin). on(user_table[:id].eq(email_table[:user_id])). where(user_table[:email].eq(email).or(email_table[:email].eq(email))) find_by_sql(query.to_sql).first end Benchmark.ips do |bench| bench.report 'original' do User.find_by_any_email_old(email) end bench.report 'optimized' do User.find_by_any_email(email) end bench.compare! end Running this locally using "bundle exec rails r bench.rb" produces the following output: Calculating ------------------------------------- original 1.000 i/100ms optimized 93.000 i/100ms ------------------------------------------------- original 11.103 (± 0.0%) i/s - 56.000 optimized 948.713 (± 5.3%) i/s - 4.743k Comparison: optimized: 948.7 i/s original: 11.1 i/s - 85.45x slower In other words, the new setup is 85x faster compared to the old setup, at least when running this benchmark locally. For GitLab.com these improvements result in User.find_by_any_email taking only ~170 ms to run, instead of around 800 ms. While this is "only" an improvement of about 4.5 times (instead of 85x) it's still significantly better than before. Fixes #3242
-
- 26 Oct, 2015 1 commit
-
-
Kamil Trzcinski authored
-
- 17 Oct, 2015 1 commit
-
-
Stan Hu authored
The counter_cache decrement function is called when a project star is deleted, but there was no guarantee multiple workers would not attempt to delete the same item simultaneously. Use an atomic update to prevent the count from going negative. Closes #3067
-
- 15 Oct, 2015 1 commit
-
-
Yorick Peterse authored
Performance is improved in two steps: 1. On PostgreSQL an expression index is used for checking lower(email) and lower(username). 2. The check to determine if we're searching for a username or Email is moved to Ruby. Thanks to @haynes for suggesting and writing the initial implementation of this. Moving the check to Ruby makes this method an additional 1.5 times faster compared to doing the check in the SQL query. With performance being improved I've now also tweaked the amount of iterations required by the User.by_login benchmark. This method now runs between 900 and 1000 iterations per second.
-
- 05 Oct, 2015 3 commits
-
-
Peter Göbel authored
-
Peter Göbel authored
-
Kamil Trzcinski authored
-
- 03 Oct, 2015 1 commit
-
-
Guilherme Garnier authored
-
- 02 Oct, 2015 1 commit
-
-
Robert Speicher authored
-
- 01 Oct, 2015 1 commit
-
-
Cristian Bica authored
-
- 29 Sep, 2015 2 commits
-
-
Rémy Coutable authored
This simplifies the "Report button" to not use open a dropdown and adds a tooltip on this button. This also removes an extra spec and adds missing specs.
-
Rémy Coutable authored
-
- 26 Sep, 2015 1 commit
-
-
Stan Hu authored
Closes #2662
-
- 15 Sep, 2015 1 commit
-
-
Kamil Trzcinski authored
-
- 06 Sep, 2015 1 commit
-
-
Stan Hu authored
-
- 27 Aug, 2015 1 commit
-
-
Valery Sizov authored
-
- 21 Aug, 2015 1 commit
-
-
Joel Koglin authored
-
- 20 Aug, 2015 1 commit
-
-
Stan Hu authored
Only show recent push event if the branch still exists or a recent merge request has not been created Closes #2277
-
- 04 Aug, 2015 1 commit
-
-
Zeger-Jan van de Weg authored
-
- 02 Aug, 2015 1 commit
-
-
Atsushi Ishida authored
-
- 30 Jul, 2015 1 commit
-
-
Douwe Maan authored
-
- 29 Jul, 2015 1 commit
-
-
Douwe Maan authored
-
- 28 Jul, 2015 1 commit
-
-
Douwe Maan authored
-
- 23 Jul, 2015 1 commit
-
-
Robert Speicher authored
-
- 13 Jul, 2015 1 commit
-
-
Dmitriy Zaporozhets authored
Signed-off-by: Dmitriy Zaporozhets <dmitriy.zaporozhets@gmail.com>
-
- 10 Jul, 2015 1 commit
-
-
Robert Speicher authored
This method encapsulates all the logic for disabling 2FA on a specific User model.
-
- 24 Jun, 2015 1 commit
-
-
Robert Speicher authored
-
- 23 Jun, 2015 2 commits
-
-
Robert Speicher authored
- Removes looking up authors/committers by name - Renames `User.find_for_commit` to `User.find_by_any_email`
-
Robert Speicher authored
-
- 20 Jun, 2015 1 commit
-
-
Robert Speicher authored
-
- 19 Jun, 2015 3 commits
-
-
Robert Speicher authored
-
Robert Speicher authored
-
Robert Speicher authored
-
- 17 Jun, 2015 1 commit
-
-
Robert Speicher authored
Now it executes a single query instead of a possible three at the cost of some scary-looking ARel calls.
-
- 16 Jun, 2015 1 commit
-
-
swellard authored
-