apache / incubator-wayang

Apache Wayang(incubating) is the first cross-platform data processing system.
https://wayang.incubator.apache.org/
Apache License 2.0
189 stars 82 forks source link

cleaned the code for issue-259 #460

Closed kamir closed 3 months ago

kamir commented 3 months ago

The previous PR is replaced by this one.

https://github.com/apache/incubator-wayang/pull/355

2pk03 commented 3 months ago

@kamir, there are conflicts in wayang-commons/wayang-core/src/main/java/org/apache/wayang/core/optimizer/cardinality/DefaultCardinalityEstimator.java - can you please check?

kamir commented 3 months ago

Resolved conflicts in wayang-commons/wayang-core/src/main/java/org/apache/wayang/core/optimizer/cardinality/DefaultCardinalityEstimator.java

kamir commented 3 months ago

We can close this PR. Merged my changes into "kamir-patch-1"

Ready for someone to check.

We must probably improve the test routine, so that it can run also in the cloud with Github Action.