Uploaded image for project: 'Magnolia'
  1. Magnolia
  2. MAGNOLIA-6734

Assess and clean up redundant system context executions

    Details

      Description

      With MAGNOLIA-6686, we now prevent re-wrapping current context within system context, if it is already a system context.

      While doing so, we added some warnings to show where we actually run into this. This has become pretty verbose already, as shown on the bundle builds for example.

      Now we need to look at such warnings and assess if we can remove redundant wrappings in sys-ctx, or if the warning is too harsh.

      Below is a trace example provided by Oanh Thai Hoang, just when assigning a task to herself in the pulse—in this particular case, it points to the GroupManager, but we should question if the extra wrapping comes from here or from the UI, i.e. rather keep the system-context encapsulation on the lower level components (which might be consumed from different ends), and get rid of the UI one.

      2016-08-01 18:39:05,839 WARN  info.magnolia.context.MgnlContext                 : Requested to run info.magnolia.jcr.wrapper.MgnlPropertySettingContentDecorator$ChangeLastUpdateDateOp:info.magnolia.jcr.wrapper.MgnlPropertySettingContentDecorator$ChangeLastUpdateDateOp@387a784d in system context while already in system context.
      (...) [16 more invocations via content-decorators ops]
      2016-08-01 18:39:05,848 WARN  info.magnolia.context.MgnlContext                 : Requested to run info.magnolia.jcr.wrapper.MgnlPropertySettingContentDecorator$ChangeLastUpdateDateOp:info.magnolia.jcr.wrapper.MgnlPropertySettingContentDecorator$ChangeLastUpdateDateOp@26a4d9d6 in system context while already in system context.
      2016-08-01 18:39:05,862 WARN  info.magnolia.context.MgnlContext                 : Requested to run info.magnolia.cms.security.MgnlGroupManager$4:get direct sub groups publishers in system context while already in system context.
      2016-08-01 18:39:05,863 WARN  info.magnolia.context.MgnlContext                 : Requested to run info.magnolia.cms.security.MgnlGroupManager$4:get direct sub groups demo-project-publishers in system context while already in system context.
      2016-08-01 18:39:05,865 WARN  info.magnolia.context.MgnlContext                 : Requested to run info.magnolia.cms.security.MgnlGroupManager$4:get direct sub groups publishers in system context while already in system context.
      2016-08-01 18:39:05,865 WARN  info.magnolia.context.MgnlContext                 : Requested to run info.magnolia.cms.security.MgnlGroupManager$4:get direct sub groups demo-project-publishers in system context while already in system context.
      2016-08-01 18:39:05,868 WARN  info.magnolia.context.MgnlContext                 : Requested to run info.magnolia.cms.security.MgnlGroupManager$4:get direct sub groups publishers in system context while already in system context.
      2016-08-01 18:39:05,868 WARN  info.magnolia.context.MgnlContext                 : Requested to run info.magnolia.cms.security.MgnlGroupManager$4:get direct sub groups demo-project-publishers in system context while already in system context.
      

        Attachments

          Issue Links

            Activity

              People

              • Assignee:
                Unassigned
                Reporter:
                mgeljic Mikaël Geljić
              • Votes:
                0 Vote for this issue
                Watchers:
                4 Start watching this issue

                Dates

                • Created:
                  Updated:
                  Resolved: