Uploaded image for project: 'eZ Publish / Platform'
  1. eZ Publish / Platform
  2. EZP-26071

[IO][S3] Don't load asset binary unless we need to create a variation

    Details

    • Type: Improvement Improvement
    • Status: Closed
    • Priority: High High
    • Resolution: Fixed
    • Affects Version/s: 1.4.0
    • Fix Version/s: 1.5.0, 1.4.1
    • Component/s: Platform > IO
    • Labels:

      Description

      When on S3 loading files is quite a bit slower than NFS/EFS, which is magnitudes slower than local files system.

      So we should not load the asset in unless we absolutely need to, current code is loading it to eagerly causing it to be triggered on every call to ezfield_image.

      PR: https://github.com/ezsystems/ezpublish-kernel/pull/1717

        Issue Links

          Activity

          Hide
          André Rømcke added a comment -

          Merged: https://github.com/ezsystems/ezpublish-kernel/commit/96f15b1cb893da9dc2b84fac13389d2c8195732f

          QA: No need to test anything specific here, it will be part of kernel v6.4.1-rc2 for overall testing there.

          Show
          André Rømcke added a comment - Merged: https://github.com/ezsystems/ezpublish-kernel/commit/96f15b1cb893da9dc2b84fac13389d2c8195732f QA: No need to test anything specific here, it will be part of kernel v6.4.1-rc2 for overall testing there.
          Hide
          Rui Silva (Inactive) added a comment -

          No QA needed.
          Closing as fixed.
          Thanks for the heads up, André.

          Show
          Rui Silva (Inactive) added a comment - No QA needed. Closing as fixed. Thanks for the heads up, André.

            People

            • Assignee:
              Unassigned
              Reporter:
              Samuel Roze
            • Votes:
              0 Vote for this issue
              Watchers:
              1 Start watching this issue

              Dates

              • Created:
                Updated:
                Resolved: