Uploaded image for project: 'Spring Integration'
  1. Spring Integration
  2. INT-3049

DelayedMessageWrapper should be public in order to access the wrapped message from infrastructure code

        Details

        • Type: Improvement
        • Status: Closed
        • Priority: Minor
        • Resolution: Complete
        • Affects Version/s: 2.2.1
        • Fix Version/s: 2.2.5, 3.0 M3
        • Component/s: Core
        • Labels:

          Description

          When using a delay the DelayMessageHandler wraps the message info a static private DelayedMessageWrapper instance, which contains the orignal message.

          There is no ability to access the original message in a regular way, except to use refelction tricks which is ugly.

          So DelayedMessageWrapper should be public.

            Issue Links

            depends on

            Improvement - An improvement or enhancement to an existing feature or task. INT-2243 Add expression support to <delayer>

            • Minor - Minor loss of function, or other problem where easy workaround is present.
            • Closed
            is depended on by

            Sub-task - The sub-task of the issue INT-3078 Ensure that Delayer works correctly with RedisMessageStore

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

            Sub-task - The sub-task of the issue INT-3079 Ensure that Delayer works correctly with GemfireMessageStore

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

            Bug - A problem which impairs or prevents the functions of the product. INT-3070 MongoDbMessageStore cannot read delayed messages

            • Critical - Crashes, loss of data, severe memory leak.
            • Closed

              Activity

              Ascending order - Click to sort in descending order
              Hide
              Permalink
              abilan Artem Bilan added a comment -

              Thanks, Stefan!

              But it is 'Improvement': Framework doesn't provide any inconsistent behavior for current state.

              Show
              abilan Artem Bilan added a comment - Thanks, Stefan! But it is 'Improvement': Framework doesn't provide any inconsistent behavior for current state.
              Hide
              Permalink
              sker65@gmail.com Stefan Rinke added a comment -

              thx, of cause. it's not a bug, i will be an improvment.

              Show
              sker65@gmail.com Stefan Rinke added a comment - thx, of cause. it's not a bug, i will be an improvment.
              Hide
              Permalink
              abilan Artem Bilan added a comment -

              This issue is very simple: will be addressed within INT-2243.

              Show
              abilan Artem Bilan added a comment - This issue is very simple: will be addressed within INT-2243 .
              Hide
              Permalink
              abilan Artem Bilan added a comment -

              PR: https://github.com/SpringSource/spring-integration/pull/826

              Show
              abilan Artem Bilan added a comment - PR: https://github.com/SpringSource/spring-integration/pull/826
              Hide
              Permalink
              abilan Artem Bilan added a comment -

              2.2.x backport PR: https://github.com/SpringSource/spring-integration/pull/852

              Show
              abilan Artem Bilan added a comment - 2.2.x backport PR: https://github.com/SpringSource/spring-integration/pull/852
              Hide
              Permalink
              grussell Gary Russell added a comment -

              Merged

              Show
              grussell Gary Russell added a comment - Merged

                People

                • Assignee:
                  abilan Artem Bilan
                  Reporter:
                  sker65@gmail.com Stefan Rinke
                • Votes:
                  0 Vote for this issue
                  Watchers:
                  4 Start watching this issue

                  Dates

                  • Created:
                    Updated:
                    Resolved: