[GitHub] [flink] KurtYoung commented on a change in pull request #10160: [FLINK-14723][table-planner-blink] Improve some code of computed column in planner

2019-11-12 Thread GitBox
KurtYoung commented on a change in pull request #10160: [FLINK-14723][table-planner-blink] Improve some code of computed column in planner URL: https://github.com/apache/flink/pull/10160#discussion_r345590382 ## File path: flink-table/flink-table-api-scala-bridge/src/main/java/org

[GitHub] [flink] KurtYoung commented on a change in pull request #10160: [FLINK-14723][table-planner-blink] Improve some code of computed column in planner

2019-11-12 Thread GitBox
KurtYoung commented on a change in pull request #10160: [FLINK-14723][table-planner-blink] Improve some code of computed column in planner URL: https://github.com/apache/flink/pull/10160#discussion_r345575523 ## File path: flink-table/flink-table-api-scala-bridge/src/main/java/org

[GitHub] [flink] KurtYoung commented on a change in pull request #10160: [FLINK-14723][table-planner-blink] Improve some code of computed column in planner

2019-11-12 Thread GitBox
KurtYoung commented on a change in pull request #10160: [FLINK-14723][table-planner-blink] Improve some code of computed column in planner URL: https://github.com/apache/flink/pull/10160#discussion_r345561333 ## File path: flink-table/flink-table-api-java-bridge/src/main/java/org/

[GitHub] [flink] KurtYoung commented on a change in pull request #10160: [FLINK-14723][table-planner-blink] Improve some code of computed column in planner

2019-11-12 Thread GitBox
KurtYoung commented on a change in pull request #10160: [FLINK-14723][table-planner-blink] Improve some code of computed column in planner URL: https://github.com/apache/flink/pull/10160#discussion_r345557237 ## File path: flink-table/flink-table-api-java-bridge/src/main/java/org/

[GitHub] [flink] KurtYoung commented on a change in pull request #10160: [FLINK-14723][table-planner-blink] Improve some code of computed column in planner

2019-11-12 Thread GitBox
KurtYoung commented on a change in pull request #10160: [FLINK-14723][table-planner-blink] Improve some code of computed column in planner URL: https://github.com/apache/flink/pull/10160#discussion_r345201519 ## File path: flink-table/flink-table-api-java-bridge/src/main/java/org/

[GitHub] [flink] KurtYoung commented on a change in pull request #10160: [FLINK-14723][table-planner-blink] Improve some code of computed column in planner

2019-11-12 Thread GitBox
KurtYoung commented on a change in pull request #10160: [FLINK-14723][table-planner-blink] Improve some code of computed column in planner URL: https://github.com/apache/flink/pull/10160#discussion_r345200618 ## File path: flink-table/flink-table-api-java-bridge/src/main/java/org/

[GitHub] [flink] KurtYoung commented on a change in pull request #10160: [FLINK-14723][table-planner-blink] Improve some code of computed column in planner

2019-11-12 Thread GitBox
KurtYoung commented on a change in pull request #10160: [FLINK-14723][table-planner-blink] Improve some code of computed column in planner URL: https://github.com/apache/flink/pull/10160#discussion_r345183411 ## File path: flink-table/flink-table-planner-blink/src/test/java/org/ap

[GitHub] [flink] KurtYoung commented on a change in pull request #10160: [FLINK-14723][table-planner-blink] Improve some code of computed column in planner

2019-11-12 Thread GitBox
KurtYoung commented on a change in pull request #10160: [FLINK-14723][table-planner-blink] Improve some code of computed column in planner URL: https://github.com/apache/flink/pull/10160#discussion_r345182616 ## File path: flink-table/flink-table-api-java-bridge/src/main/java/org/