[GitHub] flink pull request #5619: [FLINK-8838] [table] Add Support for UNNEST a Mult...

2018-05-22 Thread asfgit
Github user asfgit closed the pull request at: https://github.com/apache/flink/pull/5619 ---

[GitHub] flink pull request #5619: [FLINK-8838] [table] Add Support for UNNEST a Mult...

2018-05-22 Thread twalthr
Github user twalthr commented on a diff in the pull request: https://github.com/apache/flink/pull/5619#discussion_r189914232 --- Diff: flink-libraries/flink-table/src/main/scala/org/apache/flink/table/plan/util/ExplodeFunctionUtil.scala --- @@ -18,74 +18,143 @@ package

[GitHub] flink pull request #5619: [FLINK-8838] [table] Add Support for UNNEST a Mult...

2018-05-17 Thread suez1224
Github user suez1224 commented on a diff in the pull request: https://github.com/apache/flink/pull/5619#discussion_r189167881 --- Diff: flink-libraries/flink-table/src/main/scala/org/apache/flink/table/plan/util/ExplodeFunctionUtil.scala --- @@ -18,74 +18,244 @@ package

[GitHub] flink pull request #5619: [FLINK-8838] [table] Add Support for UNNEST a Mult...

2018-05-17 Thread lincoln-lil
Github user lincoln-lil commented on a diff in the pull request: https://github.com/apache/flink/pull/5619#discussion_r189159027 --- Diff: flink-libraries/flink-table/src/main/scala/org/apache/flink/table/plan/util/ExplodeFunctionUtil.scala --- @@ -18,74 +18,244 @@ pack

[GitHub] flink pull request #5619: [FLINK-8838] [table] Add Support for UNNEST a Mult...

2018-05-17 Thread lincoln-lil
Github user lincoln-lil commented on a diff in the pull request: https://github.com/apache/flink/pull/5619#discussion_r189156884 --- Diff: flink-libraries/flink-table/src/main/scala/org/apache/flink/table/plan/util/ExplodeFunctionUtil.scala --- @@ -18,74 +18,244 @@ pack

[GitHub] flink pull request #5619: [FLINK-8838] [table] Add Support for UNNEST a Mult...

2018-05-17 Thread suez1224
Github user suez1224 commented on a diff in the pull request: https://github.com/apache/flink/pull/5619#discussion_r189144224 --- Diff: flink-libraries/flink-table/src/main/scala/org/apache/flink/table/plan/util/ExplodeFunctionUtil.scala --- @@ -18,74 +18,244 @@ package

[GitHub] flink pull request #5619: [FLINK-8838] [table] Add Support for UNNEST a Mult...

2018-05-17 Thread suez1224
Github user suez1224 commented on a diff in the pull request: https://github.com/apache/flink/pull/5619#discussion_r189144701 --- Diff: flink-libraries/flink-table/src/main/scala/org/apache/flink/table/plan/util/ExplodeFunctionUtil.scala --- @@ -18,74 +18,244 @@ package

[GitHub] flink pull request #5619: [FLINK-8838] [table] Add Support for UNNEST a Mult...

2018-05-17 Thread suez1224
Github user suez1224 commented on a diff in the pull request: https://github.com/apache/flink/pull/5619#discussion_r189144263 --- Diff: flink-libraries/flink-table/src/main/scala/org/apache/flink/table/plan/util/ExplodeFunctionUtil.scala --- @@ -18,74 +18,244 @@ package

[GitHub] flink pull request #5619: [FLINK-8838] [table] Add Support for UNNEST a Mult...

2018-05-17 Thread suez1224
Github user suez1224 commented on a diff in the pull request: https://github.com/apache/flink/pull/5619#discussion_r189145367 --- Diff: flink-libraries/flink-table/src/test/scala/org/apache/flink/table/runtime/batch/sql/AggregateITCase.scala --- @@ -367,6 +367,36 @@ class Aggregat

[GitHub] flink pull request #5619: [FLINK-8838] [table] Add Support for UNNEST a Mult...

2018-05-17 Thread lincoln-lil
Github user lincoln-lil commented on a diff in the pull request: https://github.com/apache/flink/pull/5619#discussion_r188990005 --- Diff: flink-libraries/flink-table/src/main/scala/org/apache/flink/table/plan/util/ExplodeFunctionUtil.scala --- @@ -18,74 +18,129 @@ pack

[GitHub] flink pull request #5619: [FLINK-8838] [table] Add Support for UNNEST a Mult...

2018-05-17 Thread lincoln-lil
Github user lincoln-lil commented on a diff in the pull request: https://github.com/apache/flink/pull/5619#discussion_r188988362 --- Diff: flink-libraries/flink-table/src/main/scala/org/apache/flink/table/plan/rules/logical/LogicalUnnestRule.scala --- @@ -104,7 +109,7 @@ class Log

[GitHub] flink pull request #5619: [FLINK-8838] [table] Add Support for UNNEST a Mult...

2018-05-17 Thread twalthr
Github user twalthr commented on a diff in the pull request: https://github.com/apache/flink/pull/5619#discussion_r188927749 --- Diff: flink-libraries/flink-table/src/main/scala/org/apache/flink/table/plan/rules/logical/LogicalUnnestRule.scala --- @@ -104,7 +109,7 @@ class Logical

[GitHub] flink pull request #5619: [FLINK-8838] [table] Add Support for UNNEST a Mult...

2018-05-17 Thread twalthr
Github user twalthr commented on a diff in the pull request: https://github.com/apache/flink/pull/5619#discussion_r188908643 --- Diff: flink-libraries/flink-table/src/main/scala/org/apache/flink/table/plan/util/ExplodeFunctionUtil.scala --- @@ -18,74 +18,129 @@ package

[GitHub] flink pull request #5619: [FLINK-8838] [table] Add Support for UNNEST a Mult...

2018-05-17 Thread twalthr
Github user twalthr commented on a diff in the pull request: https://github.com/apache/flink/pull/5619#discussion_r188910728 --- Diff: flink-libraries/flink-table/src/test/scala/org/apache/flink/table/runtime/batch/sql/AggregateITCase.scala --- @@ -367,6 +367,36 @@ class Aggregate

[GitHub] flink pull request #5619: [FLINK-8838] [table] Add Support for UNNEST a Mult...

2018-05-17 Thread twalthr
Github user twalthr commented on a diff in the pull request: https://github.com/apache/flink/pull/5619#discussion_r188906074 --- Diff: flink-libraries/flink-table/src/main/scala/org/apache/flink/table/plan/util/ExplodeFunctionUtil.scala --- @@ -18,74 +18,129 @@ package

[GitHub] flink pull request #5619: [FLINK-8838] [table] Add Support for UNNEST a Mult...

2018-05-17 Thread twalthr
Github user twalthr commented on a diff in the pull request: https://github.com/apache/flink/pull/5619#discussion_r188931486 --- Diff: flink-libraries/flink-table/src/test/scala/org/apache/flink/table/runtime/batch/sql/AggregateITCase.scala --- @@ -367,6 +367,36 @@ class Aggregate

[GitHub] flink pull request #5619: [FLINK-8838] [table] Add Support for UNNEST a Mult...

2018-03-02 Thread lincoln-lil
GitHub user lincoln-lil opened a pull request: https://github.com/apache/flink/pull/5619 [FLINK-8838] [table] Add Support for UNNEST a MultiSet type field. ## What is the purpose of the change *This PR add support for UNNEST a MultiSet type field according to SQL standard UNNEST