[SPARK-11390] [SQL] Query plan with/without filterPushdown indistinguishable

[zeocio]

…ishable

Propagate pushed filters to PhyicalRDD in DataSourceStrategy.apply

[marmbrus]

Lets call it extraInformation in both places.

[marmbrus]

Can you fix the title / description so it doesn't truncate. Maybe add a test too so this doesn't get lost in refactoring.

[zeocio]

@marmbrus i was thinking about test cases, which sql/xxxSuite should this test case be added to?

[marmbrus]

maybe sql/core/src/test/scala/org/apache/spark/sql/execution/PlannerSuite.scala ?

[zeocio]

Changes to string formatting of Filter expr can break the test case. Did this recently change from "(key#27 = 15)" to "EqualTo(key,15)" ?

[zeocio]

@marmbrus is the PR good to go?

[yhuai]

ok to test

[SparkQA]

Test build #45891 has finished for PR 9679 at commit e8a40cc.

• This patch fails Scala style tests.
• This patch merges cleanly.
• This patch adds no public classes.

[SparkQA]

Test build #45900 has finished for PR 9679 at commit 96684bc.

• This patch fails Scala style tests.
• This patch merges cleanly.
• This patch adds no public classes.

[SparkQA]

Test build #45903 has finished for PR 9679 at commit c697837.

• This patch fails PySpark unit tests.
• This patch merges cleanly.
• This patch adds no public classes.

[SparkQA]

Test build #45960 has finished for PR 9679 at commit 9597b39.

• This patch passes all tests.
• This patch does not merge cleanly.
• This patch adds no public classes.

[SparkQA]

Test build #46011 has finished for PR 9679 at commit 7e47f43.

• This patch passes all tests.
• This patch merges cleanly.
• This patch adds the following public classes (experimental):\n * case class JSONOptions(\n

[zeocio]

Please merge.

[marmbrus]

Thanks, merging to master and 1.6.