Lucene - Core
  1. Lucene - Core
  2. LUCENE-4064

Move ContentSource to PerfRunData out of DocMaker

    Details

    • Type: Improvement Improvement
    • Status: Closed
    • Priority: Minor Minor
    • Resolution: Fixed
    • Affects Version/s: None
    • Fix Version/s: 4.0-ALPHA
    • Component/s: modules/benchmark
    • Labels:
      None
    • Lucene Fields:
      New, Patch Available

      Description

      ContentSource is currently private to DocMaker, however if someone wants to write an AddDocTask that does not use DocMaker, he needs to initialize ContentSource by himself.

      It would be better if ContentSource is a shared resource on PerfRunData that is used by DocMaker, rather than owned by it.

      I'll post a patch shortly.

        Activity

        Transition Time In Source Status Execution Times Last Executer Last Execution Date
        Open Open Resolved Resolved
        5h 12m 1 Shai Erera 17/May/12 12:22
        Resolved Resolved Closed Closed
        357d 23h 20m 1 Uwe Schindler 10/May/13 11:42
        Uwe Schindler made changes -
        Status Resolved [ 5 ] Closed [ 6 ]
        Shai Erera made changes -
        Status Open [ 1 ] Resolved [ 5 ]
        Resolution Fixed [ 1 ]
        Hide
        Shai Erera added a comment -

        Committed revision 1339555.

        Show
        Shai Erera added a comment - Committed revision 1339555.
        Shai Erera made changes -
        Field Original Value New Value
        Attachment LUCENE-4064.patch [ 12527811 ]
        Hide
        Shai Erera added a comment -

        Patch:

        • Moves ContentSource outside of DocMaker to PerfRunData
        • Fixes ConsumeContentSourceTask to behave well in a multi-threaded benchmark

        I think this is ready to commit.

        Show
        Shai Erera added a comment - Patch: Moves ContentSource outside of DocMaker to PerfRunData Fixes ConsumeContentSourceTask to behave well in a multi-threaded benchmark I think this is ready to commit.
        Shai Erera created issue -

          People

          • Assignee:
            Shai Erera
            Reporter:
            Shai Erera
          • Votes:
            0 Vote for this issue
            Watchers:
            1 Start watching this issue

            Dates

            • Created:
              Updated:
              Resolved:

              Development