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

Avoid running MLlib and Streaming tests when testing SQL PRs

Attach filesAttach ScreenshotVotersWatch issueWatchersCreate sub-taskLinkCloneUpdate Comment AuthorReplace String in CommentUpdate Comment VisibilityDelete Comments
    XMLWordPrintableJSON

Details

    • Bug
    • Status: Resolved
    • Blocker
    • Resolution: Fixed
    • None
    • 1.2.0
    • Project Infra, SQL
    • None

    Description

      We are bumping up against the 120 minute time limit for tests pretty regularly now. Since we have decreased the number of shuffle partitions and up-ed the parallelism I don't think there is much low hanging fruit to speed up the SQL tests. (The tests that are listed as taking 2-3 minutes are actually 100s of tests that I think are valuable). Instead I propose we avoid running tests that we don't need to.

      This will have the added benefit of eliminating failures in SQL due to flaky streaming tests.

      Note that this won't fix the full builds that are run for every commit. There I think we just just up the test timeout.

      cc: Josh Rosen Patrick Wendell

      Attachments

        Activity

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

          People

            nchammas Nicholas Chammas
            marmbrus Michael Armbrust
            Votes:
            0 Vote for this issue
            Watchers:
            4 Start watching this issue

            Dates

              Created:
              Updated:
              Resolved:

              Slack

                Issue deployment