Solr
  1. Solr
  2. SOLR-2702

Add support for NRTCachingDirectory

    Details

    • Type: Improvement Improvement
    • Status: Closed
    • Priority: Major Major
    • Resolution: Fixed
    • Affects Version/s: None
    • Fix Version/s: 4.0-BETA, 6.0
    • Component/s: None
    • Labels:
      None

      Description

      would be nice to have this option for the new NRT support

      1. SOLR-2702.patch
        13 kB
        Yonik Seeley
      2. SOLR-2702.patch
        3 kB
        Yonik Seeley

        Issue Links

          Activity

          Hide
          Mark Miller added a comment -

          Getting this issue right counts on SOLR-2654

          Show
          Mark Miller added a comment - Getting this issue right counts on SOLR-2654
          Hide
          Jason Rutherglen added a comment -

          Can we mark this for Lucene 3.x as well?

          Show
          Jason Rutherglen added a comment - Can we mark this for Lucene 3.x as well?
          Hide
          Yonik Seeley added a comment -

          We should perhaps even configure this to be used by the example server by default given that the performance impact should be negligible if NRT isn't being used.

          Show
          Yonik Seeley added a comment - We should perhaps even configure this to be used by the example server by default given that the performance impact should be negligible if NRT isn't being used.
          Hide
          Jason Rutherglen added a comment -

          This issue [only] needs to add configuration options given NRTCachingDirectoryFactory is in trunk.

          Show
          Jason Rutherglen added a comment - This issue [only] needs to add configuration options given NRTCachingDirectoryFactory is in trunk.
          Hide
          Mark Miller added a comment -

          Another issue currently is that this will not work with SolrCloud correctly because it is not detected as an FSDir it appears, and so we don't get the index dir, and replication does not work.

          Show
          Mark Miller added a comment - Another issue currently is that this will not work with SolrCloud correctly because it is not detected as an FSDir it appears, and so we don't get the index dir, and replication does not work.
          Hide
          Yonik Seeley added a comment -

          Easiest approach for now would be to add a NRTCachingDirectory.getDelegate() call and then also check for that.

          Show
          Yonik Seeley added a comment - Easiest approach for now would be to add a NRTCachingDirectory.getDelegate() call and then also check for that.
          Hide
          Yonik Seeley added a comment -

          Quick patch for the issue of getting the index directory. No new tests, but all existing tests pass.

          Show
          Yonik Seeley added a comment - Quick patch for the issue of getting the index directory. No new tests, but all existing tests pass.
          Hide
          Yonik Seeley added a comment -

          Here's the final patch that changes the default to NRT and also changes the tests that require an FS dir to use NRT most of the time. Looping adds and soft commits on OS-X, I saw about a 3x performance increase in turnaround time.

          All tests pass and I plan on committing shortly.

          Show
          Yonik Seeley added a comment - Here's the final patch that changes the default to NRT and also changes the tests that require an FS dir to use NRT most of the time. Looping adds and soft commits on OS-X, I saw about a 3x performance increase in turnaround time. All tests pass and I plan on committing shortly.
          Hide
          Hoss Man added a comment -

          bulk fixing the version info for 4.0-ALPHA and 4.0 all affected issues have "hoss20120711-bulk-40-change" in comment

          Show
          Hoss Man added a comment - bulk fixing the version info for 4.0-ALPHA and 4.0 all affected issues have "hoss20120711-bulk-40-change" in comment
          Hide
          Robert Muir added a comment -

          rmuir20120906-bulk-40-change

          Show
          Robert Muir added a comment - rmuir20120906-bulk-40-change
          Hide
          Hoss Man added a comment -

          this appears to have been committed and included in 4.0-BETA

          Show
          Hoss Man added a comment - this appears to have been committed and included in 4.0-BETA

            People

            • Assignee:
              Yonik Seeley
              Reporter:
              Mark Miller
            • Votes:
              0 Vote for this issue
              Watchers:
              3 Start watching this issue

              Dates

              • Created:
                Updated:
                Resolved:

                Development