- 25 5月, 2018 2 次提交
-
-
由 Stan Hu 提交于
When deleting associated records, Rails loads all associations into memory (https://github.com/rails/rails/issues/22510) before destroying them. This can cause a surge in memory and cause destruction of objects to fail due to idle in transaction database timeouts. This fix is inspired from https://github.com/thisismydesign to destroy `has_many` relationships in batches. Closes #44610
-
由 Oswaldo Ferreira 提交于
We request Gitaly in a N+1 manner to build discussion diffs. Once the diffs are from different revisions, it's hard to make a single request to the service in order to build the whole response. With this change we solve this problem and simplify a lot fetching this piece of info.
-
- 23 5月, 2018 2 次提交
-
-
由 Jan Provaznik 提交于
This can occur when a note is added to a commit and then this commit is deleted.
-
由 Rémy Coutable 提交于
Signed-off-by: NRémy Coutable <remy@rymai.me>
-
- 18 5月, 2018 4 次提交
-
-
由 Matija Čupić 提交于
This reverts commit 2ebafdfb
-
由 Kamil Trzciński 提交于
-
由 Jacopo 提交于
-
由 Harish Ved 提交于
-
- 17 5月, 2018 6 次提交
-
-
由 Matija Čupić 提交于
-
由 Matija Čupić 提交于
-
由 Matija Čupić 提交于
This reverts commit 4e1bb1d1.
-
由 Matija Čupić 提交于
-
由 Matija Čupić 提交于
-
由 Matija Čupić 提交于
-
- 16 5月, 2018 4 次提交
-
-
由 Jan Provaznik 提交于
-
由 Jan Provaznik 提交于
destroy_all loads all records at once
-
由 Jan Provaznik 提交于
ObjectStore uploader requires presence of associated `uploads` record when deleting the upload file (through the carrierwave's after_commit hook) because we keep info whether file is LOCAL or REMOTE in `upload` object. For this reason we can not destroy uploads as "dependent: :destroy" hook because these would be deleted too soon. Instead we rely on carrierwave's hook to destroy `uploads` in after_commit hook. But in before_destroy hook we still have to delete not-mounted uploads (which don't use carrierwave's destroy hook). This has to be done in before_Destroy instead of after_commit because `FileUpload` requires existence of model's object on destroy action. This is not ideal state of things, in a next step we should investigate how to unify model dependencies so we can use same workflow for all uploads. Related to #45425
-
由 Matija Čupić 提交于
-
- 12 5月, 2018 1 次提交
-
-
由 Matija Čupić 提交于
-
- 11 5月, 2018 1 次提交
-
-
由 blackst0ne 提交于
In Rails 5.0 automatic inverse does not work for polymorphic relathionships. It was fixed in Rails 5.2: https://github.com/rails/rails/pull/28808 Until that the `inverse_of: :source` argument should be set explicitly.
-
- 10 5月, 2018 3 次提交
-
-
由 Brett Walker 提交于
-
由 Matija Čupić 提交于
-
由 Jacopo 提交于
-
- 08 5月, 2018 3 次提交
-
-
由 Matija Čupić 提交于
-
由 Nick Thomas 提交于
-
由 Brett Walker 提交于
-
- 05 5月, 2018 1 次提交
-
-
由 Mario de la Ossa 提交于
-
- 04 5月, 2018 1 次提交
-
-
由 Shinya Maeda 提交于
-
- 03 5月, 2018 4 次提交
-
-
由 Shinya Maeda 提交于
-
由 Shinya Maeda 提交于
-
由 Shinya Maeda 提交于
-
由 Shinya Maeda 提交于
-
- 02 5月, 2018 2 次提交
-
-
由 Shinya Maeda 提交于
-
由 Shinya Maeda 提交于
Introduce `use_fast_destroy` helper for parent associations. Rename method names in build_trace_chunks. Forbid `destroy` method when `FastDestroyAll` included.
-
- 01 5月, 2018 1 次提交
-
-
由 Shinya Maeda 提交于
-
- 25 4月, 2018 1 次提交
-
-
由 Zeger-Jan van de Weg 提交于
Direct disk access is done through Gitaly now, so the legacy path was deprecated. This path was used in Gitlab::Shell however. This required the refactoring in this commit. Added is the removal of direct path access on the project model, as that lookup wasn't needed anymore is most cases. Closes https://gitlab.com/gitlab-org/gitaly/issues/1111
-
- 24 4月, 2018 2 次提交
-
-
由 Bob Van Landuyt 提交于
-
由 Micaël Bergeron 提交于
-
- 23 4月, 2018 1 次提交
-
-
由 Lin Jen-Shin 提交于
-
- 20 4月, 2018 1 次提交
-
-
由 Andreas Brandl 提交于
-