[GitHub] flink pull request #3062: [FLINK-5144] Fix error while applying rule Aggrega...

2017-01-17 Thread asfgit
Github user asfgit closed the pull request at: https://github.com/apache/flink/pull/3062 --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub as well. If your project does not have this feature enabled and wishes so, or if the feature is

[GitHub] flink pull request #3062: [FLINK-5144] Fix error while applying rule Aggrega...

2017-01-17 Thread twalthr
Github user twalthr commented on a diff in the pull request: https://github.com/apache/flink/pull/3062#discussion_r96410334 --- Diff: tools/maven/suppressions.xml --- @@ -23,6 +23,8 @@ under the License. "http://www.puppycrawl.com/dtds/suppressions_1_1.dtd;>

[GitHub] flink pull request #3062: [FLINK-5144] Fix error while applying rule Aggrega...

2017-01-10 Thread KurtYoung
Github user KurtYoung commented on a diff in the pull request: https://github.com/apache/flink/pull/3062#discussion_r95427815 --- Diff: flink-libraries/flink-table/src/test/scala/org/apache/flink/table/api/scala/batch/sql/CorrelateITCase.scala --- @@ -0,0 +1,68 @@ +/* + *

[GitHub] flink pull request #3062: [FLINK-5144] Fix error while applying rule Aggrega...

2017-01-10 Thread KurtYoung
Github user KurtYoung commented on a diff in the pull request: https://github.com/apache/flink/pull/3062#discussion_r95427718 --- Diff: tools/maven/suppressions.xml --- @@ -23,6 +23,8 @@ under the License. "http://www.puppycrawl.com/dtds/suppressions_1_1.dtd;>

[GitHub] flink pull request #3062: [FLINK-5144] Fix error while applying rule Aggrega...

2017-01-10 Thread twalthr
Github user twalthr commented on a diff in the pull request: https://github.com/apache/flink/pull/3062#discussion_r95381802 --- Diff: tools/maven/suppressions.xml --- @@ -23,6 +23,8 @@ under the License. "http://www.puppycrawl.com/dtds/suppressions_1_1.dtd;>

[GitHub] flink pull request #3062: [FLINK-5144] Fix error while applying rule Aggrega...

2017-01-10 Thread twalthr
Github user twalthr commented on a diff in the pull request: https://github.com/apache/flink/pull/3062#discussion_r95381307 --- Diff: flink-libraries/flink-table/src/test/scala/org/apache/flink/table/api/scala/batch/sql/CorrelateITCase.scala --- @@ -0,0 +1,68 @@ +/* + *

[GitHub] flink pull request #3062: [FLINK-5144] Fix error while applying rule Aggrega...

2017-01-10 Thread KurtYoung
Github user KurtYoung commented on a diff in the pull request: https://github.com/apache/flink/pull/3062#discussion_r95374019 --- Diff: tools/maven/suppressions.xml --- @@ -23,6 +23,8 @@ under the License. "http://www.puppycrawl.com/dtds/suppressions_1_1.dtd;>

[GitHub] flink pull request #3062: [FLINK-5144] Fix error while applying rule Aggrega...

2017-01-10 Thread KurtYoung
Github user KurtYoung commented on a diff in the pull request: https://github.com/apache/flink/pull/3062#discussion_r95373645 --- Diff: flink-libraries/flink-table/src/test/scala/org/apache/flink/table/api/scala/batch/sql/CorrelateITCase.scala --- @@ -0,0 +1,68 @@ +/* + *

[GitHub] flink pull request #3062: [FLINK-5144] Fix error while applying rule Aggrega...

2017-01-10 Thread twalthr
Github user twalthr commented on a diff in the pull request: https://github.com/apache/flink/pull/3062#discussion_r95361767 --- Diff: flink-libraries/flink-table/src/test/scala/org/apache/flink/table/api/scala/batch/sql/CorrelateITCase.scala --- @@ -0,0 +1,68 @@ +/* + *

[GitHub] flink pull request #3062: [FLINK-5144] Fix error while applying rule Aggrega...

2017-01-10 Thread twalthr
Github user twalthr commented on a diff in the pull request: https://github.com/apache/flink/pull/3062#discussion_r95362143 --- Diff: tools/maven/suppressions.xml --- @@ -23,6 +23,8 @@ under the License. "http://www.puppycrawl.com/dtds/suppressions_1_1.dtd;>

[GitHub] flink pull request #3062: [FLINK-5144] Fix error while applying rule Aggrega...

2017-01-04 Thread KurtYoung
GitHub user KurtYoung opened a pull request: https://github.com/apache/flink/pull/3062 [FLINK-5144] Fix error while applying rule AggregateJoinTransposeRule There are two calcite's issues related. One is during calcite's decorrelation, there will be an assertion error: