Details

      Description

      I'm working on a couple of SolrEntityProcessor improvements:

      1) ability to provide a specific HttpClient instance to make authentication, https connections and so on.
      2) introduce consistency between solr parameters and parameters used from SolrEntityProcessor to query Solr: better rename fields to fl and format to wt while we can (the processor has not been released yet). Better use CommonParams constants instead of new ones with same content.

      1. SOLR-3190.patch
        7 kB
        Luca Cavanna
      2. SOLR-3190.patch
        6 kB
        Luca Cavanna

        Activity

        Hide
        Luca Cavanna added a comment -

        Attached patch. Added overridable getHttpClient method which allows subclasses to provide their own HttpClient instance easily.
        Renamed fields to fl and format to wt, used CommonParams.
        If you guys agree with these changes I'll update the wiki once this will be committed.
        By the way, if someone has some SolrEntityProcessor improvement to propose, just leave a comment and I'll see what I can do.

        Show
        Luca Cavanna added a comment - Attached patch. Added overridable getHttpClient method which allows subclasses to provide their own HttpClient instance easily. Renamed fields to fl and format to wt, used CommonParams. If you guys agree with these changes I'll update the wiki once this will be committed. By the way, if someone has some SolrEntityProcessor improvement to propose, just leave a comment and I'll see what I can do.
        Hide
        Luca Cavanna added a comment -

        New patch, I added also the support for the qt parameter.

        Show
        Luca Cavanna added a comment - New patch, I added also the support for the qt parameter.
        Hide
        Martijn van Groningen added a comment -

        Looks good Luca. I'll commit this soon.

        Show
        Martijn van Groningen added a comment - Looks good Luca. I'll commit this soon.
        Hide
        Martijn van Groningen added a comment -

        Committed to trunk and 3x branch.

        Show
        Martijn van Groningen added a comment - Committed to trunk and 3x branch.
        Hide
        Luca Cavanna added a comment -

        Thanks Martijn!

        Show
        Luca Cavanna added a comment - Thanks Martijn!

          People

          • Assignee:
            Martijn van Groningen
            Reporter:
            Luca Cavanna
          • Votes:
            0 Vote for this issue
            Watchers:
            0 Start watching this issue

            Dates

            • Created:
              Updated:
              Resolved:

              Development