Details

    • Type: Improvement Improvement
    • Status: Closed Closed
    • Priority: Major Major
    • Resolution: Fixed
    • Affects Version/s: None
    • Fix Version/s: SCM-ACTIVITY-1.4
    • Component/s: SCM Activity
    • Labels:
      None
    • Number of attachments :
      0

      Description

      If Maven SCM is thread-safe (to be verified) then executing SCM commands in parallel should dramatically increase performances.

        Activity

        Hide
        Evgeny Mandrikov added a comment -

        Cool ! Thanks for your time on tests.

        Show
        Evgeny Mandrikov added a comment - Cool ! Thanks for your time on tests.
        Hide
        Freddy Mallet added a comment -

        Evgeny, if you want to challenge something please provide your own measures ... but you might be surprised.

        Show
        Freddy Mallet added a comment - Evgeny, if you want to challenge something please provide your own measures ... but you might be surprised.
        Hide
        Evgeny Mandrikov added a comment -

        I surprised already And here is timings for Git on my machine :

        • 1 thread = 50s
        • 2 threads = 31s
        • 4 threads = 23s
        • 8 threads = 22s
        Show
        Evgeny Mandrikov added a comment - I surprised already And here is timings for Git on my machine : 1 thread = 50s 2 threads = 31s 4 threads = 23s 8 threads = 22s
        Hide
        David Gageot added a comment -

        Added a threadCount property that enables the plugin to retrieve the blame information with multiple threads.

        Currently this property has a default value of 1 which could be too much conservative. What do you think?

        Show
        David Gageot added a comment - Added a threadCount property that enables the plugin to retrieve the blame information with multiple threads. Currently this property has a default value of 1 which could be too much conservative. What do you think?
        Hide
        Freddy Mallet added a comment -

        Manually tested

        Show
        Freddy Mallet added a comment - Manually tested

          People

          • Assignee:
            Simon Brandhof
            Reporter:
            Simon Brandhof
          • Votes:
            0 Vote for this issue
            Watchers:
            5 Start watching this issue

            Dates

            • Created:
              Updated:
              Resolved: