Uploaded image for project: 'Terrier Core'
  1. Terrier Core
  2. TR-176

Allow abitrary context objects in SearchRequest

    Details

    • Type: Improvement
    • Status: Resolved
    • Priority: Major
    • Resolution: Fixed
    • Affects Version/s: 3.5
    • Fix Version/s: 3.6
    • Component/s: .querying
    • Labels:
      None

      Description

      Hi All,

      I propose the following changes to the SearchRequest interface:

      * * *

      void setContext(String Name, Object Value);
      Object getContext(String Name);

      * * *

      As I have faced with the growing demand for more and more complex Control Parameters during my development I figured that rather than trying to squeeze everything into a simple Parameter String with setControl(String Name,String Value) method it would be much more simple to extend the interface with the abovementioned pair of methods

      Not only would they provide greater flexibility and speed increase when passing information between components but later changes could be carried out much easier.

      Should you have any queries on this please feel free to mail me.

      Cheers,

      Greg.

        Attachments

          Activity

          gergiusz Greg Maksa created issue -
          Hide
          dyaaa Dyaa Albakour added a comment -

          This patch contains a modification of the SearchRequest interface and the Request class to allow attaching context object as well as control strings

          Show
          dyaaa Dyaa Albakour added a comment - This patch contains a modification of the SearchRequest interface and the Request class to allow attaching context object as well as control strings
          dyaaa Dyaa Albakour made changes -
          Field Original Value New Value
          Attachment TR-176.v1.patch [ 10342 ]
          craigm Craig Macdonald made changes -
          Summary Interface change ==> SearchRequest Allow abitrary context objects in SearchRequest
          Hide
          craigm Craig Macdonald added a comment -

          Committed to trunk r3616.

          Show
          craigm Craig Macdonald added a comment - Committed to trunk r3616.
          craigm Craig Macdonald made changes -
          Status Open [ 1 ] Resolved [ 5 ]
          Fix Version/s 3.6 [ 10060 ]
          Resolution Fixed [ 1 ]
          craigm Craig Macdonald made changes -
          Assignee Craig Macdonald [ craigm ] Dyaa Albakour [ dyaaa ]

            People

            • Assignee:
              dyaaa Dyaa Albakour
              Reporter:
              gergiusz Greg Maksa
            • Watchers:
              0 Start watching this issue

              Dates

              • Created:
                Updated:
                Resolved: