Uploaded image for project: 'Kuali Rice Development'
  1. Kuali Rice Development
  2. KULRICE-4451

Re-examine how RiceApplicationConfigurationService is being used to resolve doc handler urls (and other variables) from across the service bus

    Details

    • Rice Module:
      KNS, KEW
    • KAI Review Status:
      Not Required
    • KTI Review Status:
      Not Required

      Description

      Currently, if KEW encounters a doc handler url of the following form:

      $

      {application.url}

      /.....

      If the document type for that document has a service namespace along the lines of something like "KFS" then it will go to the KFS application first and try and resolve application.url from the KFS application across the service bus.

      This is a good idea in theory, however for applications that have doc handler urls that don't need to call back into the application, this extra step adds extra overhead since those variables will never be resolved successfully from the client. It may also be possible in certain cases that you don't want it to resolve from the client. At the very least we need to add some caching in here to reduce the number of times we hit the service bus. Additionally, it may be worthwhile to have the RiceApplicationService have a way to provide the standalone server with information on whether or not those kinds of callbacks into the client application should be used.

        Attachments

          Issue Links

            Activity

            ewestfal Eric Westfall created issue -
            ewestfal Eric Westfall made changes -
            Field Original Value New Value
            Fix Version/s 1.1? [ 15811 ]
            Fix Version/s 1.1 [ 14190 ]
            ewestfal Eric Westfall made changes -
            Fix Version/s 1.1? [ 15811 ]
            ewestfal Eric Westfall made changes -
            Fix Version/s 1.1 [ 14190 ]
            ewestfal Eric Westfall made changes -
            Component/s Version Compatibility [ 13202 ]
            Hide
            ewestfal Eric Westfall added a comment -

            Jeremy, just wanted to put this on your radar as per our discussion this week on this service.

            Show
            ewestfal Eric Westfall added a comment - Jeremy, just wanted to put this on your radar as per our discussion this week on this service.
            ewestfal Eric Westfall made changes -
            Assignee Jeremy Hanson [ jjhanso ]
            jjhanso Jeremy Hanson made changes -
            Original Estimate 1 week [ 144000 ]
            Remaining Estimate 1 week [ 144000 ]
            ewestfal Eric Westfall made changes -
            Fix Version/s 2.0.0-m7 [ 16314 ]
            ewestfal Eric Westfall made changes -
            Start Date
            Fix Date 2011-08-08 [ set to sprint end date ]
            jcoltrin Jessica Coltrin (Inactive) made changes -
            Fix Version/s 2.0.0-m8 [ 16326 ]
            Fix Version/s 2.0.0-m7 [ 16314 ]
            jcoltrin Jessica Coltrin (Inactive) made changes -
            Start Date
            Fix Date 2011-08-08 2011-09-05 [ set to sprint end date ]
            jcoltrin Jessica Coltrin (Inactive) made changes -
            Fix Version/s 2.0.0-m9 [ 16327 ]
            Fix Version/s 2.0.0-m8 [ 16326 ]
            jcoltrin Jessica Coltrin (Inactive) made changes -
            Start Date
            Fix Date 2011-09-05 2011-10-10 [ set to sprint end date ]
            jcoltrin Jessica Coltrin (Inactive) made changes -
            Rice Lead jjhanso
            ewestfal Eric Westfall made changes -
            Fix Version/s 2.0.0-m9 [ 16327 ]
            jjhanso Jeremy Hanson made changes -
            Fix Version/s 2.0.0-b1 [ 16315 ]
            jjhanso Jeremy Hanson made changes -
            Start Date
            Fix Date 2011-10-10 2011-11-07 [ set to sprint end date ]
            jcoltrin Jessica Coltrin (Inactive) made changes -
            Priority Major [ 3 ] Critical [ 2 ]
            Hide
            jjhanso Jeremy Hanson added a comment -

            Eric, assigning this to you because you mentioned you might get rid of the RiceApplicationConfigurationService. If that falls through, feel free to assign back to me.

            Show
            jjhanso Jeremy Hanson added a comment - Eric, assigning this to you because you mentioned you might get rid of the RiceApplicationConfigurationService. If that falls through, feel free to assign back to me.
            jjhanso Jeremy Hanson made changes -
            Assignee Jeremy Hanson [ jjhanso ] Eric Westfall [ ewestfal ]
            Hide
            ewestfal Eric Westfall added a comment -

            Work on this issue has been completed as part of KULRICE-5762

            Show
            ewestfal Eric Westfall added a comment - Work on this issue has been completed as part of KULRICE-5762
            ewestfal Eric Westfall made changes -
            Status Open [ 1 ] Resolved [ 5 ]
            Resolution Duplicate [ 3 ]
            ewestfal Eric Westfall made changes -
            Link This issue duplicates KULRICE-5762 [ KULRICE-5762 ]
            Hide
            jcoltrin Jessica Coltrin (Inactive) added a comment -

            Closing since these items are now in the release notes.

            Show
            jcoltrin Jessica Coltrin (Inactive) added a comment - Closing since these items are now in the release notes.
            jcoltrin Jessica Coltrin (Inactive) made changes -
            Status Resolved [ 5 ] Closed [ 6 ]
            spatterson Shem Patterson (Inactive) made changes -
            Workflow custom [ 85750 ] Copy of custom for rice [ 212129 ]
            spatterson Shem Patterson (Inactive) made changes -
            Workflow Copy of custom for rice [ 212129 ] custom [ 221877 ]
            spatterson Shem Patterson (Inactive) made changes -
            Workflow custom [ 221877 ] Rice Workflow [ 231625 ]

              People

              • Assignee:
                ewestfal Eric Westfall
                Reporter:
                ewestfal Eric Westfall
              • Votes:
                0 Vote for this issue
                Watchers:
                0 Start watching this issue

                Dates

                • Created:
                  Updated:
                  Resolved:

                  Time Tracking

                  Estimated:
                  Original Estimate - 1 week
                  1w
                  Remaining:
                  Remaining Estimate - 1 week
                  1w
                  Logged:
                  Time Spent - Not Specified
                  Not Specified