[SPARK-23573] Create linter rule to prevent misuse of SparkContext.hadoopConfiguration in SQL modules

• Type: Improvement
• Status: Resolved
• Priority: Major
• Resolution: Duplicate
• Affects Version/s: 2.4.0
• Fix Version/s: 2.4.0
• Component/s: SQL
• Assignee: Gengliang Wang
• Reporter: Xiao Li
• Created: 03/Mar/18 01:15
• Updated: 20/Aug/18 17:12
• Resolved: 20/Aug/18 17:12

Description

SparkContext.hadoopConfiguration should not be used directly in SQL module.
Instead, one should always use sessionState.newHadoopConfiguration(), which blends in configs set per session.

The idea is to add the linter rule to ban it.

• Restrict the linter rule to the components that use SQL. use the parameter `scalastyleSources`?
• Exclude genuinely valid uses, like e.g. in SessionState (ok, that can be done per case with scalastyle:off in the code.

Issue Links

• is duplicated by: SPARK-24919 Scala linter rule for sparkContext.hadoopConfiguration