Skip to content

Conversation

MartijnVisser
Copy link
Contributor

No description provided.

Copy link
Contributor

@knaufk knaufk left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks @MartijnVisser. UNNEST is a good topic. IMO the example is unnecessarily complex to highlight the build-in function, at least I hard a time getting it.

@MartijnVisser
Copy link
Contributor Author

@knaufk I've simplified the recipe by removing the MAP and ROW data types and now focussing only on ARRAY. LMKWYT

@MartijnVisser MartijnVisser requested a review from knaufk December 2, 2021 11:58
Copy link
Contributor

@knaufk knaufk left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks. Looks good voerall. A few more minor suggestions.

Copy link

@twalthr twalthr left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

One minor comment. Otherwise LGTM.

@MartijnVisser MartijnVisser merged commit 1a5eb60 into ververica:main Dec 6, 2021
@MartijnVisser MartijnVisser deleted the recipe/arrays branch December 6, 2021 09:59
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants