Uploaded image for project: 'Solr'
  1. Solr
  2. SOLR-7710

replace various "async" with CommonAdminParams.ASYNC

    Details

    • Type: Wish
    • Status: Closed
    • Priority: Minor
    • Resolution: Fixed
    • Affects Version/s: None
    • Fix Version/s: 5.3
    • Component/s: None
    • Labels:
      None

      Description

      • github pull request with proposed changes to follow shortly.
      • Motivation is addition of a test case for an unrelated change (not yet uploaded) and not wishing to add literally "async" in that test case. Using CommonAdminParams.ASYNC instead of "async" can also make code base 'navigation' a little easier e.g. locating of existing test cases.
      • The proposed change also adds a little CommonAdminParamsTest to test the backwards compatibility of CommonAdminParams parameter constants i.e. if someone accidentally changes those constants then the test will flag that up.

        Attachments

        1. SOLR-7710.patch
          18 kB
          Ramkumar Aiyengar

          Activity

            People

            • Assignee:
              andyetitmoves Ramkumar Aiyengar
              Reporter:
              cpoerschke Christine Poerschke
            • Votes:
              0 Vote for this issue
              Watchers:
              5 Start watching this issue

              Dates

              • Created:
                Updated:
                Resolved: