commit | 4b9c5cb1a7fb272db2314380fa2b31282b4c1f91 | [log] [tgz] |
---|---|---|
author | Hugo Arès <hugo.ares@ericsson.com> | Fri Mar 24 17:48:33 2017 -0400 |
committer | David Pursehouse <dpursehouse@collab.net> | Wed Mar 29 03:09:55 2017 +0000 |
tree | c73f643a1e161ae22f476096dac777c990a1739b | |
parent | f2d4fac3f35febc56fb6b64b482bfb35a4de9451 [diff] |
Precompute h2 persisted cache space used per entry The problem is h2 cannot execute concurrent queries on the same database. The query to get the disk stats(count and space) is a long lasting query which blocks any other queries to the same cache. The disk stats are needed when pruning the cache (every day at 01:00) and when caches are listed, either using the REST API or the show-caches ssh command. Here is an example of the problem: diff_intraline of 1GB limit with 1.6 million entries can take up to 16 seconds to compute disk stats. During that time, any request that needs to get/insert data into the diff intraline will be blocked until disk stats are done. Every time an admin executes the show-caches command, a lot of requests can be blocked for a long time; (number of big persisted caches) x (time to compute stats) The problem is even worse during pruning because right after computing the space, another long lasting query is executed; select every row and associated space used. Precomputing the space used improves the situation. In the previous example, it reduces the query time from 16 to 4 seconds. The real fix would be to also precompute the sum of space used by all entries in a separate table but this is more complex since it would involve triggers to keep the sum up to date when a row is inserted, updated or deleted. Even with this, it would still not be perfect, in the previous example, it reduces the query time from 4 to 2.5 seconds. The number of rows also needs to be precomputed, but precomputing this would be wrong since counting number of rows should be quick in a database; it should not be implemented in the schema. This change improves the situation by cutting the query time by 4 with minimal change. A real fix will be done in another change. Bug: Issue 5876 Change-Id: I38805688289a474aa719c52ab03079cbf11d1c5c
Gerrit is a code review and project management tool for Git based projects.
Gerrit makes reviews easier by showing changes in a side-by-side display, and allowing inline comments to be added by any reviewer.
Gerrit simplifies Git based project maintainership by permitting any authorized user to submit changes to the master Git repository, rather than requiring all approved changes to be merged in by hand by the project maintainer.
For information about how to install and use Gerrit, refer to the documentation.
Our canonical Git repository is located on googlesource.com. There is a mirror of the repository on Github.
Please report bugs on the issue tracker.
Gerrit is the work of hundreds of contributors. We appreciate your help!
Please read the contribution guidelines.
Note that we do not accept Pull Requests via the Github mirror.
The IRC channel on freenode is #gerrit. An archive is available at: echelog.com.
The Developer Mailing list is repo-discuss on Google Groups.
Gerrit is provided under the Apache License 2.0.
Install Bazel and run the following:
git clone --recursive https://gerrit.googlesource.com/gerrit cd gerrit && bazel build release
The instruction how to configure GerritForge/BinTray repositories is here
On Debian/Ubuntu run:
apt-get update & apt-get install gerrit=<version>-<release>
NOTE: release is a counter that starts with 1 and indicates the number of packages that have been released with the same version of the software.
On CentOS/RedHat run:
yum clean all && yum install gerrit-<version>[-<release>]
On Fedora run:
dnf clean all && dnf install gerrit-<version>[-<release>]
Docker images of Gerrit are available on DockerHub
To run a CentOS 7 based Gerrit image:
docker run -p 8080:8080 gerritforge/gerrit-centos7[:version]
To run a Ubuntu 15.04 based Gerrit image:
docker run -p 8080:8080 gerritforge/gerrit-ubuntu15.04[:version]
NOTE: release is optional. Last released package of the version is installed if the release number is omitted.