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

ZkSolrResourceLoader constructors accept a parameter called "collection" but it should be "configName"

    Details

    • Type: Bug
    • Status: Closed
    • Priority: Trivial
    • Resolution: Fixed
    • Affects Version/s: None
    • Fix Version/s: 5.0, 6.0
    • Component/s: None
    • Labels:
      None

      Description

      public ZkSolrResourceLoader(String instanceDir, String collection,
            ZkController zooKeeperController);
      public ZkSolrResourceLoader(String instanceDir, String collection, ClassLoader parent,
            Properties coreProperties, ZkController zooKeeperController);
      

      The CloudConfigSetService created ZkSolrResourceLoader using the configName (which is correct).

      We should renamed the param in ZkSolrResourceLoader to be configSetName and also rename the "collectionZkPath" member to be "configSetZkPath"

        Attachments

        1. SOLR-6715.patch
          14 kB
          Shalin Shekhar Mangar

          Activity

            People

            • Assignee:
              shalinmangar Shalin Shekhar Mangar
              Reporter:
              shalinmangar Shalin Shekhar Mangar
            • Votes:
              0 Vote for this issue
              Watchers:
              4 Start watching this issue

              Dates

              • Created:
                Updated:
                Resolved: