subject:"\[GitHub\] \[spark\] AngersZhuuuu commented on pull request #31485\: \[SPARK\-34137\]\[SQL\] Update suquery's stats when build LogicalPlan's stats"

[GitHub] [spark] AngersZhuuuu commented on pull request #31485: [SPARK-34137][SQL] Update suquery's stats when build LogicalPlan's stats

2021-02-07 Thread GitBox

AngersZh commented on pull request #31485: URL: https://github.com/apache/spark/pull/31485#issuecomment-774843305 > ```sql > create table t1 using parquet as select id as a, id as b, id as c from range(100); > create table t2 using parquet as select id as a, id as b, id as c

[GitHub] [spark] AngersZhuuuu commented on pull request #31485: [SPARK-34137][SQL] Update suquery's stats when build LogicalPlan's stats

2021-02-06 Thread GitBox

AngersZh commented on pull request #31485: URL: https://github.com/apache/spark/pull/31485#issuecomment-774577889 > Looks good. Is it possible to add a test in `StatisticsCollectionSuite`? How about current change?

[GitHub] [spark] AngersZhuuuu commented on pull request #31485: [SPARK-34137][SQL] Update suquery's stats when build LogicalPlan's stats

2021-02-06 Thread GitBox

AngersZh commented on pull request #31485: URL: https://github.com/apache/spark/pull/31485#issuecomment-774497964 > Looks good. Is it possible to add a test in `StatisticsCollectionSuite`? How about current UT?