[GitHub] spark pull request #16395: [SPARK-17075][SQL] implemented filter estimation

2017-01-16 Thread wzhfy
Github user wzhfy commented on a diff in the pull request: https://github.com/apache/spark/pull/16395#discussion_r96182039 --- Diff: sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/plans/logical/statsEstimation/FilterEstimation.scala --- @@ -0,0 +1,620 @@ +/* +

[GitHub] spark pull request #16395: [SPARK-17075][SQL] implemented filter estimation

2017-01-16 Thread wzhfy
Github user wzhfy commented on a diff in the pull request: https://github.com/apache/spark/pull/16395#discussion_r96177721 --- Diff: sql/catalyst/src/test/scala/org/apache/spark/sql/catalyst/statsEstimation/FilterEstimationSuite.scala --- @@ -0,0 +1,309 @@ +/* + *

[GitHub] spark pull request #16395: [SPARK-17075][SQL] implemented filter estimation

2017-01-16 Thread wzhfy
Github user wzhfy commented on a diff in the pull request: https://github.com/apache/spark/pull/16395#discussion_r96185977 --- Diff: sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/plans/logical/statsEstimation/FilterEstimation.scala --- @@ -0,0 +1,620 @@ +/* +

[GitHub] spark pull request #16395: [SPARK-17075][SQL] implemented filter estimation

2017-01-16 Thread wzhfy
Github user wzhfy commented on a diff in the pull request: https://github.com/apache/spark/pull/16395#discussion_r96186262 --- Diff: sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/plans/logical/statsEstimation/FilterEstimation.scala --- @@ -0,0 +1,620 @@ +/* +

[GitHub] spark pull request #16395: [SPARK-17075][SQL] implemented filter estimation

2017-01-16 Thread wzhfy
Github user wzhfy commented on a diff in the pull request: https://github.com/apache/spark/pull/16395#discussion_r96178759 --- Diff: sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/plans/logical/statsEstimation/FilterEstimation.scala --- @@ -0,0 +1,620 @@ +/* +

[GitHub] spark pull request #16395: [SPARK-17075][SQL] implemented filter estimation

2017-01-16 Thread wzhfy
Github user wzhfy commented on a diff in the pull request: https://github.com/apache/spark/pull/16395#discussion_r96180586 --- Diff: sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/plans/logical/statsEstimation/FilterEstimation.scala --- @@ -0,0 +1,620 @@ +/* +

[GitHub] spark pull request #16395: [SPARK-17075][SQL] implemented filter estimation

2017-01-16 Thread wzhfy
Github user wzhfy commented on a diff in the pull request: https://github.com/apache/spark/pull/16395#discussion_r96178685 --- Diff: sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/plans/logical/statsEstimation/FilterEstimation.scala --- @@ -0,0 +1,620 @@ +/* +

[GitHub] spark pull request #16395: [SPARK-17075][SQL] implemented filter estimation

2017-01-16 Thread wzhfy
Github user wzhfy commented on a diff in the pull request: https://github.com/apache/spark/pull/16395#discussion_r96182452 --- Diff: sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/plans/logical/statsEstimation/FilterEstimation.scala --- @@ -0,0 +1,620 @@ +/* +

[GitHub] spark pull request #16395: [SPARK-17075][SQL] implemented filter estimation

2017-01-14 Thread ron8hu
Github user ron8hu commented on a diff in the pull request: https://github.com/apache/spark/pull/16395#discussion_r96128057 --- Diff: sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/plans/logical/statsEstimation/Range.scala --- @@ -0,0 +1,75 @@ +/* + * Licensed

[GitHub] spark pull request #16395: [SPARK-17075][SQL] implemented filter estimation

2017-01-14 Thread ron8hu
Github user ron8hu commented on a diff in the pull request: https://github.com/apache/spark/pull/16395#discussion_r96128021 --- Diff: sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/plans/logical/statsEstimation/Range.scala --- @@ -0,0 +1,75 @@ +/* + * Licensed

[GitHub] spark pull request #16395: [SPARK-17075][SQL] implemented filter estimation

2017-01-12 Thread ron8hu
Github user ron8hu commented on a diff in the pull request: https://github.com/apache/spark/pull/16395#discussion_r95935452 --- Diff: sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/plans/logical/basicLogicalOperators.scala --- @@ -116,6 +116,12 @@ case class

[GitHub] spark pull request #16395: [SPARK-17075][SQL] implemented filter estimation

2017-01-12 Thread ron8hu
Github user ron8hu commented on a diff in the pull request: https://github.com/apache/spark/pull/16395#discussion_r95916520 --- Diff: sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/plans/logical/statsEstimation/FilterEstimation.scala --- @@ -0,0 +1,555 @@ +/* +

[GitHub] spark pull request #16395: [SPARK-17075][SQL] implemented filter estimation

2017-01-11 Thread rxin
Github user rxin commented on a diff in the pull request: https://github.com/apache/spark/pull/16395#discussion_r95726607 --- Diff: sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/plans/logical/statsEstimation/FilterEstimation.scala --- @@ -0,0 +1,555 @@ +/* + *

[GitHub] spark pull request #16395: [SPARK-17075][SQL] implemented filter estimation

2017-01-11 Thread ron8hu
Github user ron8hu commented on a diff in the pull request: https://github.com/apache/spark/pull/16395#discussion_r95720653 --- Diff: sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/plans/logical/statsEstimation/FilterEstimation.scala --- @@ -0,0 +1,555 @@ +/* +

[GitHub] spark pull request #16395: [SPARK-17075][SQL] implemented filter estimation

2017-01-11 Thread wzhfy
Github user wzhfy commented on a diff in the pull request: https://github.com/apache/spark/pull/16395#discussion_r95720309 --- Diff: sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/plans/logical/statsEstimation/Range.scala --- @@ -0,0 +1,75 @@ +/* + * Licensed

[GitHub] spark pull request #16395: [SPARK-17075][SQL] implemented filter estimation

2017-01-11 Thread ron8hu
Github user ron8hu commented on a diff in the pull request: https://github.com/apache/spark/pull/16395#discussion_r95719350 --- Diff: sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/plans/logical/statsEstimation/FilterEstimation.scala --- @@ -0,0 +1,555 @@ +/* +

[GitHub] spark pull request #16395: [SPARK-17075][SQL] implemented filter estimation

2017-01-11 Thread ron8hu
Github user ron8hu commented on a diff in the pull request: https://github.com/apache/spark/pull/16395#discussion_r95719141 --- Diff: sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/plans/logical/statsEstimation/FilterEstimation.scala --- @@ -0,0 +1,555 @@ +/* +

[GitHub] spark pull request #16395: [SPARK-17075][SQL] implemented filter estimation

2017-01-11 Thread ron8hu
Github user ron8hu commented on a diff in the pull request: https://github.com/apache/spark/pull/16395#discussion_r95718997 --- Diff: sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/plans/logical/statsEstimation/FilterEstimation.scala --- @@ -0,0 +1,555 @@ +/* +

[GitHub] spark pull request #16395: [SPARK-17075][SQL] implemented filter estimation

2017-01-11 Thread ron8hu
Github user ron8hu commented on a diff in the pull request: https://github.com/apache/spark/pull/16395#discussion_r95718952 --- Diff: sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/plans/logical/statsEstimation/FilterEstimation.scala --- @@ -0,0 +1,555 @@ +/* +

[GitHub] spark pull request #16395: [SPARK-17075][SQL] implemented filter estimation

2017-01-11 Thread ron8hu
Github user ron8hu commented on a diff in the pull request: https://github.com/apache/spark/pull/16395#discussion_r95718502 --- Diff: sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/plans/logical/statsEstimation/FilterEstimation.scala --- @@ -0,0 +1,555 @@ +/* +

[GitHub] spark pull request #16395: [SPARK-17075][SQL] implemented filter estimation

2017-01-11 Thread ron8hu
Github user ron8hu commented on a diff in the pull request: https://github.com/apache/spark/pull/16395#discussion_r95717900 --- Diff: sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/plans/logical/statsEstimation/FilterEstimation.scala --- @@ -0,0 +1,555 @@ +/* +

[GitHub] spark pull request #16395: [SPARK-17075][SQL] implemented filter estimation

2017-01-11 Thread ron8hu
Github user ron8hu commented on a diff in the pull request: https://github.com/apache/spark/pull/16395#discussion_r95717298 --- Diff: sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/plans/logical/statsEstimation/EstimationUtils.scala --- @@ -52,3 +56,12 @@ object

[GitHub] spark pull request #16395: [SPARK-17075][SQL] implemented filter estimation

2017-01-11 Thread ron8hu
Github user ron8hu commented on a diff in the pull request: https://github.com/apache/spark/pull/16395#discussion_r95716353 --- Diff: sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/plans/logical/statsEstimation/FilterEstimation.scala --- @@ -0,0 +1,555 @@ +/* +

[GitHub] spark pull request #16395: [SPARK-17075][SQL] implemented filter estimation

2017-01-11 Thread ron8hu
Github user ron8hu commented on a diff in the pull request: https://github.com/apache/spark/pull/16395#discussion_r95710436 --- Diff: sql/catalyst/src/test/scala/org/apache/spark/sql/catalyst/statsEstimation/FilterEstimationSuite.scala --- @@ -0,0 +1,173 @@ +/* + *

[GitHub] spark pull request #16395: [SPARK-17075][SQL] implemented filter estimation

2017-01-11 Thread ron8hu
Github user ron8hu commented on a diff in the pull request: https://github.com/apache/spark/pull/16395#discussion_r95709024 --- Diff: sql/catalyst/src/test/scala/org/apache/spark/sql/catalyst/statsEstimation/FilterEstimationSuite.scala --- @@ -0,0 +1,173 @@ +/* + *

[GitHub] spark pull request #16395: [SPARK-17075][SQL] implemented filter estimation

2017-01-11 Thread ron8hu
Github user ron8hu commented on a diff in the pull request: https://github.com/apache/spark/pull/16395#discussion_r95707722 --- Diff: sql/catalyst/src/test/scala/org/apache/spark/sql/catalyst/statsEstimation/FilterEstimationSuite.scala --- @@ -0,0 +1,173 @@ +/* + *

[GitHub] spark pull request #16395: [SPARK-17075][SQL] implemented filter estimation

2017-01-11 Thread ron8hu
Github user ron8hu commented on a diff in the pull request: https://github.com/apache/spark/pull/16395#discussion_r95707748 --- Diff: sql/catalyst/src/test/scala/org/apache/spark/sql/catalyst/statsEstimation/FilterEstimationSuite.scala --- @@ -0,0 +1,173 @@ +/* + *

[GitHub] spark pull request #16395: [SPARK-17075][SQL] implemented filter estimation

2017-01-11 Thread ron8hu
Github user ron8hu commented on a diff in the pull request: https://github.com/apache/spark/pull/16395#discussion_r95707644 --- Diff: sql/catalyst/src/test/scala/org/apache/spark/sql/catalyst/statsEstimation/FilterEstimationSuite.scala --- @@ -0,0 +1,173 @@ +/* + *

[GitHub] spark pull request #16395: [SPARK-17075][SQL] implemented filter estimation

2017-01-11 Thread ron8hu
Github user ron8hu commented on a diff in the pull request: https://github.com/apache/spark/pull/16395#discussion_r95695560 --- Diff: sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/plans/logical/statsEstimation/FilterEstimation.scala --- @@ -0,0 +1,555 @@ +/* +

[GitHub] spark pull request #16395: [SPARK-17075][SQL] implemented filter estimation

2017-01-11 Thread ron8hu
Github user ron8hu commented on a diff in the pull request: https://github.com/apache/spark/pull/16395#discussion_r95691485 --- Diff: sql/catalyst/src/test/scala/org/apache/spark/sql/catalyst/statsEstimation/FilterEstimationSuite.scala --- @@ -0,0 +1,173 @@ +/* + *

[GitHub] spark pull request #16395: [SPARK-17075][SQL] implemented filter estimation

2017-01-11 Thread ron8hu
Github user ron8hu commented on a diff in the pull request: https://github.com/apache/spark/pull/16395#discussion_r95690239 --- Diff: sql/catalyst/src/test/scala/org/apache/spark/sql/catalyst/statsEstimation/FilterEstimationSuite.scala --- @@ -0,0 +1,173 @@ +/* + *

[GitHub] spark pull request #16395: [SPARK-17075][SQL] implemented filter estimation

2017-01-11 Thread rxin
Github user rxin commented on a diff in the pull request: https://github.com/apache/spark/pull/16395#discussion_r95527981 --- Diff: sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/plans/logical/basicLogicalOperators.scala --- @@ -116,6 +116,12 @@ case class

[GitHub] spark pull request #16395: [SPARK-17075][SQL] implemented filter estimation

2017-01-10 Thread rxin
Github user rxin commented on a diff in the pull request: https://github.com/apache/spark/pull/16395#discussion_r95524651 --- Diff: sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/plans/logical/statsEstimation/FilterEstimation.scala --- @@ -0,0 +1,555 @@ +/* + *

[GitHub] spark pull request #16395: [SPARK-17075][SQL] implemented filter estimation

2017-01-10 Thread rxin
Github user rxin commented on a diff in the pull request: https://github.com/apache/spark/pull/16395#discussion_r95524585 --- Diff: sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/plans/logical/statsEstimation/FilterEstimation.scala --- @@ -0,0 +1,555 @@ +/* + *

[GitHub] spark pull request #16395: [SPARK-17075][SQL] implemented filter estimation

2017-01-10 Thread rxin
Github user rxin commented on a diff in the pull request: https://github.com/apache/spark/pull/16395#discussion_r95524418 --- Diff: sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/plans/logical/statsEstimation/FilterEstimation.scala --- @@ -0,0 +1,555 @@ +/* + *

[GitHub] spark pull request #16395: [SPARK-17075][SQL] implemented filter estimation

2017-01-10 Thread rxin
Github user rxin commented on a diff in the pull request: https://github.com/apache/spark/pull/16395#discussion_r95524337 --- Diff: sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/plans/logical/statsEstimation/FilterEstimation.scala --- @@ -0,0 +1,555 @@ +/* + *

[GitHub] spark pull request #16395: [SPARK-17075][SQL] implemented filter estimation

2017-01-10 Thread rxin
Github user rxin commented on a diff in the pull request: https://github.com/apache/spark/pull/16395#discussion_r95524302 --- Diff: sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/plans/logical/statsEstimation/FilterEstimation.scala --- @@ -0,0 +1,555 @@ +/* + *

[GitHub] spark pull request #16395: [SPARK-17075][SQL] implemented filter estimation

2017-01-10 Thread rxin
Github user rxin commented on a diff in the pull request: https://github.com/apache/spark/pull/16395#discussion_r95524281 --- Diff: sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/plans/logical/statsEstimation/FilterEstimation.scala --- @@ -0,0 +1,555 @@ +/* + *

[GitHub] spark pull request #16395: [SPARK-17075][SQL] implemented filter estimation

2017-01-10 Thread rxin
Github user rxin commented on a diff in the pull request: https://github.com/apache/spark/pull/16395#discussion_r95524128 --- Diff: sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/plans/logical/statsEstimation/FilterEstimation.scala --- @@ -0,0 +1,555 @@ +/* + *

[GitHub] spark pull request #16395: [SPARK-17075][SQL] implemented filter estimation

2017-01-10 Thread rxin
Github user rxin commented on a diff in the pull request: https://github.com/apache/spark/pull/16395#discussion_r95524086 --- Diff: sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/plans/logical/statsEstimation/FilterEstimation.scala --- @@ -0,0 +1,555 @@ +/* + *

[GitHub] spark pull request #16395: [SPARK-17075][SQL] implemented filter estimation

2017-01-10 Thread rxin
Github user rxin commented on a diff in the pull request: https://github.com/apache/spark/pull/16395#discussion_r95523860 --- Diff: sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/plans/logical/statsEstimation/FilterEstimation.scala --- @@ -0,0 +1,555 @@ +/* + *

[GitHub] spark pull request #16395: [SPARK-17075][SQL] implemented filter estimation

2017-01-10 Thread rxin
Github user rxin commented on a diff in the pull request: https://github.com/apache/spark/pull/16395#discussion_r95523914 --- Diff: sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/plans/logical/statsEstimation/FilterEstimation.scala --- @@ -0,0 +1,555 @@ +/* + *

[GitHub] spark pull request #16395: [SPARK-17075][SQL] implemented filter estimation

2017-01-10 Thread rxin
Github user rxin commented on a diff in the pull request: https://github.com/apache/spark/pull/16395#discussion_r95523806 --- Diff: sql/catalyst/src/test/scala/org/apache/spark/sql/catalyst/statsEstimation/FilterEstimationSuite.scala --- @@ -0,0 +1,173 @@ +/* + * Licensed

[GitHub] spark pull request #16395: [SPARK-17075][SQL] implemented filter estimation

2017-01-10 Thread rxin
Github user rxin commented on a diff in the pull request: https://github.com/apache/spark/pull/16395#discussion_r95523768 --- Diff: sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/plans/logical/statsEstimation/Range.scala --- @@ -0,0 +1,75 @@ +/* + * Licensed to

[GitHub] spark pull request #16395: [SPARK-17075][SQL] implemented filter estimation

2017-01-10 Thread rxin
Github user rxin commented on a diff in the pull request: https://github.com/apache/spark/pull/16395#discussion_r95523666 --- Diff: sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/plans/logical/statsEstimation/Range.scala --- @@ -0,0 +1,75 @@ +/* + * Licensed to

[GitHub] spark pull request #16395: [SPARK-17075][SQL] implemented filter estimation

2017-01-10 Thread rxin
Github user rxin commented on a diff in the pull request: https://github.com/apache/spark/pull/16395#discussion_r95522835 --- Diff: sql/catalyst/src/test/scala/org/apache/spark/sql/catalyst/statsEstimation/FilterEstimationSuite.scala --- @@ -0,0 +1,173 @@ +/* + * Licensed

[GitHub] spark pull request #16395: [SPARK-17075][SQL] implemented filter estimation

2017-01-10 Thread rxin
Github user rxin commented on a diff in the pull request: https://github.com/apache/spark/pull/16395#discussion_r95522818 --- Diff: sql/catalyst/src/test/scala/org/apache/spark/sql/catalyst/statsEstimation/FilterEstimationSuite.scala --- @@ -0,0 +1,173 @@ +/* + * Licensed

[GitHub] spark pull request #16395: [SPARK-17075][SQL] implemented filter estimation

2017-01-10 Thread rxin
Github user rxin commented on a diff in the pull request: https://github.com/apache/spark/pull/16395#discussion_r95522719 --- Diff: sql/catalyst/src/test/scala/org/apache/spark/sql/catalyst/statsEstimation/FilterEstimationSuite.scala --- @@ -0,0 +1,173 @@ +/* + * Licensed

[GitHub] spark pull request #16395: [SPARK-17075][SQL] implemented filter estimation

2017-01-10 Thread rxin
Github user rxin commented on a diff in the pull request: https://github.com/apache/spark/pull/16395#discussion_r95522619 --- Diff: sql/catalyst/src/test/scala/org/apache/spark/sql/catalyst/statsEstimation/FilterEstimationSuite.scala --- @@ -0,0 +1,173 @@ +/* + * Licensed

[GitHub] spark pull request #16395: [SPARK-17075][SQL] implemented filter estimation

2017-01-10 Thread rxin
Github user rxin commented on a diff in the pull request: https://github.com/apache/spark/pull/16395#discussion_r95522395 --- Diff: sql/catalyst/src/test/scala/org/apache/spark/sql/catalyst/statsEstimation/FilterEstimationSuite.scala --- @@ -0,0 +1,173 @@ +/* + * Licensed

[GitHub] spark pull request #16395: [SPARK-17075][SQL] implemented filter estimation

2017-01-10 Thread rxin
Github user rxin commented on a diff in the pull request: https://github.com/apache/spark/pull/16395#discussion_r95522373 --- Diff: sql/catalyst/src/test/scala/org/apache/spark/sql/catalyst/statsEstimation/FilterEstimationSuite.scala --- @@ -0,0 +1,173 @@ +/* + * Licensed

[GitHub] spark pull request #16395: [SPARK-17075][SQL] implemented filter estimation

2017-01-10 Thread rxin
Github user rxin commented on a diff in the pull request: https://github.com/apache/spark/pull/16395#discussion_r95522411 --- Diff: sql/catalyst/src/test/scala/org/apache/spark/sql/catalyst/statsEstimation/FilterEstimationSuite.scala --- @@ -0,0 +1,173 @@ +/* + * Licensed

[GitHub] spark pull request #16395: [SPARK-17075][SQL] implemented filter estimation

2017-01-10 Thread rxin
Github user rxin commented on a diff in the pull request: https://github.com/apache/spark/pull/16395#discussion_r95522307 --- Diff: sql/catalyst/src/test/scala/org/apache/spark/sql/catalyst/statsEstimation/FilterEstimationSuite.scala --- @@ -0,0 +1,173 @@ +/* + * Licensed

[GitHub] spark pull request #16395: [SPARK-17075][SQL] implemented filter estimation

2017-01-10 Thread rxin
Github user rxin commented on a diff in the pull request: https://github.com/apache/spark/pull/16395#discussion_r95522247 --- Diff: sql/catalyst/src/test/scala/org/apache/spark/sql/catalyst/statsEstimation/FilterEstimationSuite.scala --- @@ -0,0 +1,173 @@ +/* + * Licensed

[GitHub] spark pull request #16395: [SPARK-17075][SQL] implemented filter estimation

2017-01-10 Thread rxin
Github user rxin commented on a diff in the pull request: https://github.com/apache/spark/pull/16395#discussion_r95522065 --- Diff: sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/plans/logical/statsEstimation/FilterEstimation.scala --- @@ -0,0 +1,555 @@ +/* + *

[GitHub] spark pull request #16395: [SPARK-17075][SQL] implemented filter estimation

2017-01-10 Thread rxin
Github user rxin commented on a diff in the pull request: https://github.com/apache/spark/pull/16395#discussion_r95521090 --- Diff: sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/plans/logical/statsEstimation/EstimationUtils.scala --- @@ -52,3 +56,12 @@ object

[GitHub] spark pull request #16395: [SPARK-17075][SQL] implemented filter estimation

2017-01-10 Thread ron8hu
Github user ron8hu commented on a diff in the pull request: https://github.com/apache/spark/pull/16395#discussion_r95511085 --- Diff: sql/core/src/test/scala/org/apache/spark/sql/estimation/FilterEstimationSuite.scala --- @@ -0,0 +1,226 @@ +/* + * Licensed to the Apache

[GitHub] spark pull request #16395: [SPARK-17075][SQL] implemented filter estimation

2017-01-08 Thread ron8hu
Github user ron8hu commented on a diff in the pull request: https://github.com/apache/spark/pull/16395#discussion_r95096602 --- Diff: sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/plans/logical/estimation/FilterEstimation.scala --- @@ -0,0 +1,479 @@ +/* + *

[GitHub] spark pull request #16395: [SPARK-17075][SQL] implemented filter estimation

2017-01-08 Thread ron8hu
Github user ron8hu commented on a diff in the pull request: https://github.com/apache/spark/pull/16395#discussion_r95096610 --- Diff: sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/plans/logical/estimation/FilterEstimation.scala --- @@ -0,0 +1,479 @@ +/* + *

[GitHub] spark pull request #16395: [SPARK-17075][SQL] implemented filter estimation

2017-01-08 Thread ron8hu
Github user ron8hu commented on a diff in the pull request: https://github.com/apache/spark/pull/16395#discussion_r95096571 --- Diff: sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/plans/logical/estimation/FilterEstimation.scala --- @@ -0,0 +1,479 @@ +/* + *

[GitHub] spark pull request #16395: [SPARK-17075][SQL] implemented filter estimation

2017-01-08 Thread ron8hu
Github user ron8hu commented on a diff in the pull request: https://github.com/apache/spark/pull/16395#discussion_r95096457 --- Diff: sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/plans/logical/estimation/FilterEstimation.scala --- @@ -0,0 +1,479 @@ +/* + *

[GitHub] spark pull request #16395: [SPARK-17075][SQL] implemented filter estimation

2017-01-08 Thread ron8hu
Github user ron8hu commented on a diff in the pull request: https://github.com/apache/spark/pull/16395#discussion_r95096432 --- Diff: sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/plans/logical/estimation/FilterEstimation.scala --- @@ -0,0 +1,479 @@ +/* + *

[GitHub] spark pull request #16395: [SPARK-17075][SQL] implemented filter estimation

2017-01-08 Thread ron8hu
Github user ron8hu commented on a diff in the pull request: https://github.com/apache/spark/pull/16395#discussion_r95096451 --- Diff: sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/plans/logical/estimation/FilterEstimation.scala --- @@ -0,0 +1,479 @@ +/* + *

[GitHub] spark pull request #16395: [SPARK-17075][SQL] implemented filter estimation

2017-01-08 Thread ron8hu
Github user ron8hu commented on a diff in the pull request: https://github.com/apache/spark/pull/16395#discussion_r95096418 --- Diff: sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/plans/logical/estimation/FilterEstimation.scala --- @@ -0,0 +1,479 @@ +/* + *

[GitHub] spark pull request #16395: [SPARK-17075][SQL] implemented filter estimation

2017-01-05 Thread rxin
Github user rxin commented on a diff in the pull request: https://github.com/apache/spark/pull/16395#discussion_r94906528 --- Diff: sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/plans/logical/estimation/FilterEstimation.scala --- @@ -0,0 +1,479 @@ +/* + *

[GitHub] spark pull request #16395: [SPARK-17075][SQL] implemented filter estimation

2017-01-05 Thread rxin
Github user rxin commented on a diff in the pull request: https://github.com/apache/spark/pull/16395#discussion_r94906278 --- Diff: sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/plans/logical/estimation/FilterEstimation.scala --- @@ -0,0 +1,479 @@ +/* + *

[GitHub] spark pull request #16395: [SPARK-17075][SQL] implemented filter estimation

2017-01-05 Thread rxin
Github user rxin commented on a diff in the pull request: https://github.com/apache/spark/pull/16395#discussion_r94906270 --- Diff: sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/plans/logical/estimation/FilterEstimation.scala --- @@ -0,0 +1,479 @@ +/* + *

[GitHub] spark pull request #16395: [SPARK-17075][SQL] implemented filter estimation

2017-01-05 Thread rxin
Github user rxin commented on a diff in the pull request: https://github.com/apache/spark/pull/16395#discussion_r94906266 --- Diff: sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/plans/logical/estimation/FilterEstimation.scala --- @@ -0,0 +1,479 @@ +/* + *

[GitHub] spark pull request #16395: [SPARK-17075][SQL] implemented filter estimation

2017-01-05 Thread rxin
Github user rxin commented on a diff in the pull request: https://github.com/apache/spark/pull/16395#discussion_r94906221 --- Diff: sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/plans/logical/estimation/FilterEstimation.scala --- @@ -0,0 +1,479 @@ +/* + *

[GitHub] spark pull request #16395: [SPARK-17075][SQL] implemented filter estimation

2017-01-05 Thread rxin
Github user rxin commented on a diff in the pull request: https://github.com/apache/spark/pull/16395#discussion_r94906180 --- Diff: sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/plans/logical/estimation/Range.scala --- @@ -0,0 +1,75 @@ +/* + * Licensed to the

[GitHub] spark pull request #16395: [SPARK-17075][SQL] implemented filter estimation

2017-01-05 Thread rxin
Github user rxin commented on a diff in the pull request: https://github.com/apache/spark/pull/16395#discussion_r94906142 --- Diff: sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/plans/logical/estimation/Range.scala --- @@ -0,0 +1,75 @@ +/* + * Licensed to the

[GitHub] spark pull request #16395: [SPARK-17075][SQL] implemented filter estimation

2017-01-05 Thread rxin
Github user rxin commented on a diff in the pull request: https://github.com/apache/spark/pull/16395#discussion_r94905939 --- Diff: sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/plans/logical/estimation/FilterEstimation.scala --- @@ -0,0 +1,479 @@ +/* + *

[GitHub] spark pull request #16395: [SPARK-17075][SQL] implemented filter estimation

2017-01-05 Thread rxin
Github user rxin commented on a diff in the pull request: https://github.com/apache/spark/pull/16395#discussion_r94905864 --- Diff: sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/plans/logical/estimation/FilterEstimation.scala --- @@ -0,0 +1,479 @@ +/* + *

[GitHub] spark pull request #16395: [SPARK-17075][SQL] implemented filter estimation

2017-01-05 Thread rxin
Github user rxin commented on a diff in the pull request: https://github.com/apache/spark/pull/16395#discussion_r94905897 --- Diff: sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/plans/logical/estimation/FilterEstimation.scala --- @@ -0,0 +1,479 @@ +/* + *

[GitHub] spark pull request #16395: [SPARK-17075][SQL] implemented filter estimation

2017-01-05 Thread rxin
Github user rxin commented on a diff in the pull request: https://github.com/apache/spark/pull/16395#discussion_r94905771 --- Diff: sql/core/src/test/scala/org/apache/spark/sql/estimation/FilterEstimationSuite.scala --- @@ -0,0 +1,226 @@ +/* + * Licensed to the Apache

<    1   2