Skip to content

Commit

Permalink
[SPARK-23140][SQL] Add DataSourceV2Strategy to Hive Session state's p…
Browse files Browse the repository at this point in the history
…lanner

## What changes were proposed in this pull request?

`DataSourceV2Strategy` is missing in `HiveSessionStateBuilder`'s planner, which will throw exception as described in [SPARK-23140](https://issues.apache.org/jira/browse/SPARK-23140).

## How was this patch tested?

Manual test.

Author: jerryshao <[email protected]>

Closes #20305 from jerryshao/SPARK-23140.

(cherry picked from commit 7a22483)
Signed-off-by: Wenchen Fan <[email protected]>
  • Loading branch information
jerryshao authored and cloud-fan committed Jan 18, 2018
1 parent 2a87c3a commit f801ac4
Showing 1 changed file with 1 addition and 16 deletions.
Original file line number Diff line number Diff line change
Expand Up @@ -96,22 +96,7 @@ class HiveSessionStateBuilder(session: SparkSession, parentState: Option[Session
override val sparkSession: SparkSession = session

override def extraPlanningStrategies: Seq[Strategy] =
super.extraPlanningStrategies ++ customPlanningStrategies

override def strategies: Seq[Strategy] = {
experimentalMethods.extraStrategies ++
extraPlanningStrategies ++ Seq(
FileSourceStrategy,
DataSourceStrategy(conf),
SpecialLimits,
InMemoryScans,
HiveTableScans,
Scripts,
Aggregation,
JoinSelection,
BasicOperators
)
}
super.extraPlanningStrategies ++ customPlanningStrategies ++ Seq(HiveTableScans, Scripts)
}
}

Expand Down

0 comments on commit f801ac4

Please sign in to comment.