Uploaded image for project: 'Spark'
  1. Spark
  2. SPARK-28403

Executor Allocation Manager can add an extra executor when speculative tasks

Rank to TopRank to BottomAttach filesAttach ScreenshotBulk Copy AttachmentsBulk Move AttachmentsVotersWatch issueWatchersCreate sub-taskConvert to sub-taskLinkCloneLabelsUpdate Comment AuthorReplace String in CommentUpdate Comment VisibilityDelete Comments
    XMLWordPrintableJSON

Details

    • Bug
    • Status: Resolved
    • Major
    • Resolution: Incomplete
    • 2.3.0
    • None
    • Spark Core

    Description

      It looks like SPARK-19326 added a bug in the execuctor allocation maanger where it adds an extra executor when it shouldn't when we have pending speculative tasks but the target number didn't change. https://github.com/apache/spark/blob/master/core/src/main/scala/org/apache/spark/ExecutorAllocationManager.scala#L377

      It doesn't look like this is necessary since it already added in the pendingSpeculative tasks.

      See the questioning of this on the PR at:

      https://github.com/apache/spark/pull/18492/files#diff-b096353602813e47074ace09a3890d56R379

      Attachments

        Issue Links

        Blocked

        Bug - A problem which impairs or prevents the functions of the product. SPARK-30511 Spark marks intentionally killed speculative tasks as pending leads to holding idle executors

        • Major - Major loss of function.
        • Resolved
        is caused by

        Bug - A problem which impairs or prevents the functions of the product. SPARK-19326 Speculated task attempts do not get launched in few scenarios

        • Major - Major loss of function.
        • Resolved

        Activity
          This comment will be Viewable by All Users Viewable by All Users
          Cancel

          People

            Unassigned Unassigned
            tgraves Thomas Graves
            Holden Karau Holden Karau
            Votes:
            0 Vote for this issue
            Watchers:
            3 Start watching this issue

            Dates

              Created:
              Updated:
              Resolved:

              Slack

                Issue deployment