Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

SNOW-989398: Refactor resample.fillna to use ASOF Join #2196

Merged
merged 11 commits into from
Sep 4, 2024

Conversation

sfc-gh-nkrishna
Copy link
Contributor

@sfc-gh-nkrishna sfc-gh-nkrishna commented Aug 29, 2024

SNOW-989398

This PR refactors resample.fillna() to use the ASOF Join.

Signed-off-by: Naren Krishna <[email protected]>
@sfc-gh-nkrishna sfc-gh-nkrishna added the NO-PANDAS-CHANGEDOC-UPDATES This PR does not update Snowpark pandas docs label Aug 29, 2024
@sfc-gh-nkrishna sfc-gh-nkrishna changed the title SNOW-1630279: Refactor resample fillna to use join_utils SNOW-1630279: Refactor resample.fillna to use OrderedDataFrame join Aug 29, 2024
@sfc-gh-nkrishna sfc-gh-nkrishna marked this pull request as ready for review August 30, 2024 04:31
@sfc-gh-nkrishna sfc-gh-nkrishna requested a review from a team as a code owner August 30, 2024 04:31
Signed-off-by: Naren Krishna <[email protected]>
Copy link
Contributor

@sfc-gh-vbudati sfc-gh-vbudati left a comment

Choose a reason for hiding this comment

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

Looks great! Thanks Naren!

Copy link
Collaborator

@sfc-gh-evandenberg sfc-gh-evandenberg left a comment

Choose a reason for hiding this comment

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

Small comment, looks good to me.

Signed-off-by: Naren Krishna <[email protected]>
@sfc-gh-nkrishna sfc-gh-nkrishna changed the title SNOW-1630279: Refactor resample.fillna to use OrderedDataFrame join SNOW-1630279, SNOW-989398: Refactor resample.fillna to use OrderedDataFrame join Sep 3, 2024
@sfc-gh-nkrishna sfc-gh-nkrishna changed the title SNOW-1630279, SNOW-989398: Refactor resample.fillna to use OrderedDataFrame join SNOW-1630279, SNOW-989398: Refactor resample.fillna to use ASOF Join Sep 3, 2024
Signed-off-by: Naren Krishna <[email protected]>
@sfc-gh-nkrishna sfc-gh-nkrishna changed the title SNOW-1630279, SNOW-989398: Refactor resample.fillna to use ASOF Join SNOW-989398: Refactor resample.fillna to use ASOF Join Sep 3, 2024
Copy link
Collaborator

@sfc-gh-yzou sfc-gh-yzou left a comment

Choose a reason for hiding this comment

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

left some small comments

CHANGELOG.md Outdated Show resolved Hide resolved
Signed-off-by: Naren Krishna <[email protected]>
@sfc-gh-nkrishna sfc-gh-nkrishna added the NO-CHANGELOG-UPDATES This pull request does not need to update CHANGELOG.md label Sep 4, 2024
@sfc-gh-nkrishna sfc-gh-nkrishna merged commit b80d0e6 into main Sep 4, 2024
37 of 39 checks passed
@sfc-gh-nkrishna sfc-gh-nkrishna deleted the nkrishna-SNOW-1630279-resample-join branch September 4, 2024 03:17
@github-actions github-actions bot locked and limited conversation to collaborators Sep 4, 2024
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
NO-CHANGELOG-UPDATES This pull request does not need to update CHANGELOG.md NO-PANDAS-CHANGEDOC-UPDATES This PR does not update Snowpark pandas docs snowpark-pandas
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants