FailedChanges

Summary

  1. [SPARK-25716][SQL][MINOR] remove unnecessary collection operation in (details)
Commit 0820484ba1457f0511f02c20a04d6c8fb9099d02 by gatorsmile
[SPARK-25716][SQL][MINOR] remove unnecessary collection operation in
valid constraints generation
## What changes were proposed in this pull request?
Project logical operator generates valid constraints using two opposite
operations. It substracts child constraints from all constraints, than
union child constraints again. I think it may be not necessary.
Aggregate operator has the same problem with Project.
This PR try to remove these two opposite collection operations.
## How was this patch tested?
Related unit tests: ProjectEstimationSuite CollapseProjectSuite
PushProjectThroughUnionSuite UnsafeProjectionBenchmark
GeneratedProjectionSuite CodeGeneratorWithInterpretedFallbackSuite
TakeOrderedAndProjectSuite GenerateUnsafeProjectionSuite
BucketedRandomProjectionLSHSuite RemoveRedundantAliasAndProjectSuite
AggregateBenchmark AggregateOptimizeSuite AggregateEstimationSuite
DecimalAggregatesSuite DateFrameAggregateSuite ObjectHashAggregateSuite
TwoLevelAggregateHashMapSuite ObjectHashAggregateExecBenchmark
SingleLevelAggregateHaspMapSuite TypedImperativeAggregateSuite
RewriteDistinctAggregatesSuite HashAggregationQuerySuite
HashAggregationQueryWithControlledFallbackSuite
TypedImperativeAggregateSuite
TwoLevelAggregateHashMapWithVectorizedMapSuite
Closes #22706 from SongYadong/generate_constraints.
Authored-by: SongYadong <song.yadong1@zte.com.cn> Signed-off-by:
gatorsmile <gatorsmile@gmail.com>
The file was modifiedsql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/plans/logical/LogicalPlan.scala (diff)
The file was modifiedsql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/plans/logical/basicLogicalOperators.scala (diff)