Timo Walther comments

Results 18 comments of


                                            Timo Walther

[DRAFT] FLIP-376 DISTRIBUTED BY

Replaced by #24155.

[FLINK-34366][table] Add support to group rows by column ordinals

@jeyhunkarimov First, it would be great to identify the motivation behind this ticket. Is it just because we want to be compatible with BigQuery? How many vendors support it this...

Questions about Example_06_DataStream_Join

1. This is indeed a bug. I will fix it in my next iteration. Thanks for pointing it out! 2. Yes, there is more code required to make this production...

[FLINK-31664] Implement ARRAY_INTERSECT function

This is indeed a tricky one. I also spent a significant amount of time here. Every vendor does it differently. I looked also at programming languages such as PHP, C#....

[FLINK-37923][sql] Introduce VARIANT type and PARSE_JSON to Flink SQL

Thanks @Sxnan. I added it for my list for tomorrow. I'm sure it can still make it before the feature freeze.

[FLINK-35016] catalog changes for model resource

@lihaosky can we close this PR then? It is very confusing if comments are spread across PRs. And I basically have to re-review everything because my "marked as reviewed" is...

[FLINK-37245] Prevent masking null values in BinaryRowData

@mxm could you rebase this PR? We also observed similar issues when implementing PTFs. The field getter in RowData got fixed already. Not sure about the other changes in this...

[FLINK-35717][Table SQL / API] Allow DISTRIBUTED BY in CREATE TABLE AS (CTAS)

@flinkbot run azure