[SPARK-22847] Remove the duplicate code in AppStatusListener while assigning schedulingPool for stage

• Type: Improvement
• Status: Resolved
• Priority: Trivial
• Resolution: Fixed
• Affects Version/s: 2.2.1
• Fix Version/s: 2.3.0
• Component/s: Spark Core
• Labels: newbie
• Flags: Patch
• Assignee: wuyi
• Reporter: wuyi
• Created: 20/Dec/17 12:56
• Updated: 21/Dec/17 03:15
• Resolved: 20/Dec/17 22:28

Description

In AppStatusListener's onStageSubmitted(event: SparkListenerStageSubmitted) method, there are duplicate code:

// schedulingPool was assigned twice with the same code
stage.schedulingPool = Option(event.properties).flatMap { p =>
      Option(p.getProperty("spark.scheduler.pool"))
    }.getOrElse(SparkUI.DEFAULT_POOL_NAME)

But, it does not make any sense to do this and there are no comment to explain for this.

Issue Links

• links to: [Github] Pull Request #20033 (Ngone51)