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

Make MRExecutor pollInterval configurable

    XMLWordPrintableJSON

Details

    • Improvement
    • Status: Resolved
    • Major
    • Resolution: Fixed
    • None
    • 1.0.0
    • Core
    • 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. 0001-CRUNCH-665-Add-crunch.max.poll.interval-property.patch
          3 kB
          Clément MATHIEU
        2. user-guide.mdtext.patch
          0.9 kB
          Clément MATHIEU

        Activity

          People

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

            Dates

              Created:
              Updated:
              Resolved: