Uploaded image for project: 'Nuxeo Platform'
  1. Nuxeo Platform
  2. NXP-28156

Remove WorkManager#listWorkIds usage from PictureViewsGenerationWork

    XMLWordPrintable

    Details

      Description

      Check and remove
      https://github.com/nuxeo/nuxeo/blob/master/nuxeo-features/nuxeo-platform-imaging/nuxeo-platform-imaging-core/src/main/java/org/nuxeo/ecm/platform/picture/PictureViewsGenerationWork.java#L145

      Here the idea is to not trigger pictureViewsGenerationDone event while a concurrent work with the same id is running,
      ensuring the event is sent only for the last work (NXP-12949)

      The id is: repositoryName + ':' + docId + ':' + xpath + ":pictureView",
      so it is per blob which means that we are on the coalescing Work here, not a group of Work + trigger case.

      This Work is not used anymore in master since NXP-26420, should it be removed or annotate as deprecated?

        Attachments

          Issue Links

          depends on

          Improvement - An improvement or enhancement to an existing feature or task. NXP-28182 Be able to trigger an action after completion of a group of Works

          • Minor - Minor loss of function, or other problem where easy workaround is present.
          • Resolved
          is related to

          Improvement - An improvement or enhancement to an existing feature or task. NXP-12949 Send events when all the video conversions or picture views generation are done

          • Minor - Minor loss of function, or other problem where easy workaround is present.
          • Resolved

          Improvement - An improvement or enhancement to an existing feature or task. NXP-26420 Re-implement Picture.RecomputeViews operation using BAF

          • Minor - Minor loss of function, or other problem where easy workaround is present.
          • Resolved

            Activity

              People

              • Votes:
                0 Vote for this issue
                Watchers:
                3 Start watching this issue

                Dates

                • Created:
                  Updated:
                  Resolved:

                  Time Tracking

                  Estimated:
                  Original Estimate - Not Specified
                  Not Specified
                  Remaining:
                  Remaining Estimate - 0 minutes
                  0m
                  Logged:
                  Time Spent - 4 hours
                  4h