Uploaded image for project: 'Crunch'
  1. Crunch
  2. CRUNCH-665

Make MRExecutor pollInterval configurable

    XMLWordPrintableJSON

    Details

    • Type: Improvement
    • Status: Resolved
    • Priority: Major
    • Resolution: Fixed
    • Affects Version/s: None
    • Fix Version/s: None
    • Component/s: Core
    • Labels:
      None

      Description

      MRExecutor#pollInterval is currently hard coded to 50..1_000 for local mode and 500..10_000 otherwise. While being a sensible trade-off, one might want to poll more aggressively. For example, 1 seconds is usually too high for small integration tests.

      A few experiments on internal test suites showed that lowering poll interval could halve execution time.

      What do you think about adding a new configuration parameter that would override default config? I can craft a patch if needed.

        Attachments

        1. user-guide.mdtext.patch
          0.9 kB
          Clément MATHIEU
        2. 0001-CRUNCH-665-Add-crunch.max.poll.interval-property.patch
          3 kB
          Clément MATHIEU

          Activity

            People

            • Assignee:
              jwills Josh Wills
              Reporter:
              clement@unportant.info Clément MATHIEU
            • Votes:
              0 Vote for this issue
              Watchers:
              3 Start watching this issue

              Dates

              • Created:
                Updated:
                Resolved: