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

Index directory disagreement SolrCore#initIndex

Attach filesAttach ScreenshotVotersWatch issueWatchersCreate sub-taskLinkCloneUpdate Comment AuthorReplace String in CommentUpdate Comment VisibilityDelete Comments
    XMLWordPrintableJSON

Details

    • Bug
    • Status: Closed
    • Minor
    • Resolution: Fixed
    • 1.4, 1.4.1
    • 3.1, 4.0-ALPHA
    • None
    • None

    Description

      getNewIndexDir is widely used in this method - but then when a new index is created, getIndexDir is used:

            // Create the index if it doesn't exist.
            if(!indexExists) {
              log.warn(logid+"Solr index directory '" + new File(getNewIndexDir()) + "' doesn't exist."
                      + " Creating new index...");
      
              SolrIndexWriter writer = new SolrIndexWriter("SolrCore.initIndex", getIndexDir(), getDirectoryFactory(), true, schema, solrConfig.mainIndexConfig, solrDelPolicy);
              writer.close();
            }
      

      also this piece uses getIndexDir():

            if (indexExists && firstTime && removeLocks) {
              // to remove locks, the directory must already exist... so we create it
              // if it didn't exist already...
              Directory dir = SolrIndexWriter.getDirectory(getIndexDir(), getDirectoryFactory(), solrConfig.mainIndexConfig);
              if (dir != null)  {
                if (IndexWriter.isLocked(dir)) {
                  log.warn(logid+"WARNING: Solr index directory '" + getIndexDir() + "' is locked.  Unlocking...");
                  IndexWriter.unlock(dir);
                }
                dir.close();
              }
            }
      

      Attachments

        Activity

          This comment will be Viewable by All Users Viewable by All Users
          Cancel

          People

            markrmiller@gmail.com Mark Miller
            markrmiller@gmail.com Mark Miller
            Votes:
            0 Vote for this issue
            Watchers:
            0 Start watching this issue

            Dates

              Created:
              Updated:
              Resolved:

              Slack

                Issue deployment