SNOW-1552377: Add support for flatmap using udtf#2812
Conversation
|
Seems like your changes contain some Local Testing changes, please request review from @snowflakedb/local-testing |
|
Seems like your changes contain some Local Testing changes, please request review from @snowflakedb/local-testing |
sfc-gh-jrose
left a comment
There was a problem hiding this comment.
Based on docstrings it seems unclear what the difference between map and flatmap is.
let me update the doc strings. The main difference is df.map is 1-1 mapping for each row and flat map can have 1-many mapping. |
|
Seems like your changes contain some Local Testing changes, please request review from @snowflakedb/local-testing |
…add-support-for-flatmap-using-udtf
|
Seems like your changes contain some Local Testing changes, please request review from @snowflakedb/local-testing |
|
Seems like your changes contain some Local Testing changes, please request review from @snowflakedb/local-testing |
15bc20c
into
SNOW-1830033-dataframe-api-bcrs
Which Jira issue is this PR addressing? Make sure that there is an accompanying issue to your PR.
Fixes SNOW-1552377
Fill out the following pre-review checklist:
Please describe how your code solves the related issue.
This PR adds implementation for
Dataframe.flatmapand refactors implementation ofDataframe.mapto de-duplicate common code used in flatmap.