[SPARK-13023] Check for presence of 'root' module after computing test_modules, not changed_modules

• Type: Bug
• Status: Resolved
• Priority: Major
• Resolution: Fixed
• Fix Version/s: 1.6.2, 2.0.0
• Component/s: Project Infra
• Assignee: Josh Rosen
• Reporter: Josh Rosen
• Created: 26/Jan/16 23:55
• Updated: 22/Jun/16 13:26
• Resolved: 27/Jan/16 16:32

Description

There's a minor bug in how we handle the `root` module in the `modules_to_test()` function in `dev/run-tests.py`: since `root` now depends on `build` (since every test needs to run on any build test), we now need to check for the presence of root in `modules_to_test` instead of `changed_modules`.

Issue Links

• links to: [Github] Pull Request #10933 (JoshRosen)
• links to: [Github] Pull Request #12743 (yhuai)
• links to: [Github] Pull Request #13845 (HyukjinKwon)