[GitHub] spark issue #19537: [SQL] Mark strategies with override for clarity.

2017-10-19 Thread ericjperry
Github user ericjperry commented on the issue: https://github.com/apache/spark/pull/19537 @srowen not that I noticed during my reading at least. I can't speak much about classes outside of the SQL portion of the project, t

[GitHub] spark pull request #19537: [SQL] Mark strategies with override for clarity.

2017-10-19 Thread ericjperry
GitHub user ericjperry opened a pull request: https://github.com/apache/spark/pull/19537 [SQL] Mark strategies with override for clarity. ## What changes were proposed in this pull request? This is a very trivial PR, simply marking `strategies` in `SparkPlanner` with the