Revise comments

fa61ea6e · Shinya Maeda · 92b2647d · fa61ea6e
Commit fa61ea6e authored May 28, 2018 by Shinya Maeda
Hide whitespace changes
Inline Side-by-side

Showing with 2 additions and 3 deletions

config/initializers/carrierwave_monkey_patch.rb config/initializers/carrierwave_monkey_patch.rb +2 -3

No files found.
--- a/config/initializers/carrierwave_monkey_patch.rb
+++ b/config/initializers/carrierwave_monkey_patch.rb
-# This is a monkey patch until https://github.com/carrierwaveuploader/carrierwave/pull/2314 has been merged
+# This fixes the problem https://gitlab.com/gitlab-org/gitlab-ce/issues/46182 that carrierwave eagerly loads upoloading files into memory
-# This fixes a problem https://gitlab.com/gitlab-org/gitlab-ce/issues/46182 that carrierwave loads large files into memory
+# There is an PR https://github.com/carrierwaveuploader/carrierwave/pull/2314 which has the identical change.
-# and triggers sidekiq shutdown by hitting RSS limit.
 module CarrierWave
  module Storage
    class Fog < Abstract