[GitHub] [flink] matriv commented on pull request #15744: [FLINK-22113][table-planner-blink] Implement the column uniqueness checking for TableSourceTable

2021-11-22 Thread GitBox
matriv commented on pull request #15744: URL: https://github.com/apache/flink/pull/15744#issuecomment-975527105 @xuguangheng Again, apologies for the delay, but since we'd like to get forward and fix this soon, I'd like to take over, I hope that's fine for you. -- This is an automated

[GitHub] [flink] matriv commented on pull request #15744: [FLINK-22113][table-planner-blink] Implement the column uniqueness checking for TableSourceTable

2021-10-28 Thread GitBox
matriv commented on pull request #15744: URL: https://github.com/apache/flink/pull/15744#issuecomment-953738271 Hi @xuguangheng, apologies for the delay in looking at your PR. We would like to fix this bug soon, as it's a quite important one, would you have the time to continue this PR

[GitHub] [flink] matriv commented on pull request #15744: [FLINK-22113][table-planner-blink] Implement the column uniqueness checking for TableSourceTable

2021-10-06 Thread GitBox
matriv commented on pull request #15744: URL: https://github.com/apache/flink/pull/15744#issuecomment-935904382 @xuguangheng Thank you for your effort! - Overall it looks good, please rebase with master to get the new layout of the modules - Please replace the deprecated