[GitHub] [flink] xuyangzhong commented on pull request #17118: [FLINK-22603][table-planner]The digest can be produced by SourceAbilitySpec.

2021-09-08 Thread GitBox
xuyangzhong commented on pull request #17118: URL: https://github.com/apache/flink/pull/17118#issuecomment-915709849 @flinkbot run azure -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the

[GitHub] [flink] xuyangzhong commented on pull request #17118: [FLINK-22603][table-planner]The digest can be produced by SourceAbilitySpec.

2021-09-07 Thread GitBox
xuyangzhong commented on pull request #17118: URL: https://github.com/apache/flink/pull/17118#issuecomment-914148618 @flinkbot run azure -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the

[GitHub] [flink] xuyangzhong commented on pull request #17118: [FLINK-22603][table-planner]The digest can be produced by SourceAbilitySpec.

2021-09-02 Thread GitBox
xuyangzhong commented on pull request #17118: URL: https://github.com/apache/flink/pull/17118#issuecomment-912210846 @Airblader Hiļ¼Œthanks to review my immature code, and i want to explain why the implement function _getExtraDigests_ needs an _Optional_ arg. -- In the