Solr
  1. Solr
  2. SOLR-389

RequestHandlerBase javadocs improvement

    Details

    • Type: Improvement Improvement
    • Status: Resolved
    • Priority: Trivial Trivial
    • Resolution: Fixed
    • Affects Version/s: None
    • Fix Version/s: None
    • Component/s: documentation
    • Labels:
      None

      Description

      Provide more javadocs on RequestHandlerBase#.init(NamedList) method to explain about the defaults, appends and invariants

      1. SOLR-389.patch
        3 kB
        Grant Ingersoll
      2. SOLR-389.patch
        3 kB
        Grant Ingersoll

        Activity

        Hide
        Grant Ingersoll added a comment -

        First draft of javadocs for RequestHandlerBase#init method explaining the use of defaults, invariants and appends parameters.

        Show
        Grant Ingersoll added a comment - First draft of javadocs for RequestHandlerBase#init method explaining the use of defaults, invariants and appends parameters.
        Hide
        Grant Ingersoll added a comment -

        Changed @see method to point to SolrPluginUtils instead of U#setDefaults

        Show
        Grant Ingersoll added a comment - Changed @see method to point to SolrPluginUtils instead of U#setDefaults
        Hide
        Ryan McKinley added a comment -

        Thanks Grant

        Do you want me to commit this now? Is it still "In Progress" (I have not seen people use that status before, does it mean you are still working on it?)

        Show
        Ryan McKinley added a comment - Thanks Grant Do you want me to commit this now? Is it still "In Progress" (I have not seen people use that status before, does it mean you are still working on it?)
        Hide
        Ryan McKinley added a comment -

        I will delete:
        http://wiki.apache.org/solr/RequestHandlerBase

        to avoid the duplicate docs

        Show
        Ryan McKinley added a comment - I will delete: http://wiki.apache.org/solr/RequestHandlerBase to avoid the duplicate docs
        Hide
        Grant Ingersoll added a comment -

        It should be done, pending your review.

        Show
        Grant Ingersoll added a comment - It should be done, pending your review.
        Hide
        Ryan McKinley added a comment -

        commited a while back. Thanks Grant

        Show
        Ryan McKinley added a comment - commited a while back. Thanks Grant

          People

          • Assignee:
            Ryan McKinley
            Reporter:
            Grant Ingersoll
          • Votes:
            0 Vote for this issue
            Watchers:
            0 Start watching this issue

            Dates

            • Created:
              Updated:
              Resolved:

              Development