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

Release 2022.4.2 #240

Closed
jrbourbeau opened this issue Apr 28, 2022 · 10 comments
Closed

Release 2022.4.2 #240

jrbourbeau opened this issue Apr 28, 2022 · 10 comments

Comments

@jrbourbeau
Copy link
Member

If there are no blockers I'd like to release dask and distributed 2022.4.2 this Friday as part of our usual release cadence. In particular, it would be good to get some pandas compatibility updates (for the next pandas release) out (xref dask/dask#8961) as well as a fix for a reported deadlock (xref dask/distributed#6110).

I know @jcrist @rjzamora @ian-r-rose @bryanwweber are working on changing the default values for some parquet-related keyword arguments. @jcrist mentioned offline it would be good to include all those changes in the same release. @jcrist what's in flight that you'd like to see included?

@mrocklin @fjetter @crusaderky there's been a lot of activity in distributed since the last release. How are things looking for releasing this Friday? Are there specific PRs you explicitly want to see included or not included in this release?

cc @jsignell @quasiben @jakirkham

@rjzamora
Copy link
Member

rjzamora commented Apr 28, 2022

On the parquet side it would be ideal to merge dask/dask#8988 and dask/dask#8992 as close together as possible (as well as the related doc changes).

It would also be good to get dask/dask#8989 in since it's currently blocking dask-sql CI (another from_delayed surprise).

Status:

  • dask#8988 (merged)
  • dask#8992 (merged)
  • dask#8989 (merged)

@crusaderky
Copy link

crusaderky commented Apr 28, 2022

@rjzamora
Copy link
Member

rjzamora commented Apr 29, 2022

Given this comment from @ian-r-rose in the from_delayed fix, I wonder if anyone is interested in trying to include the from_map PR (dask/dask#8911) in this release (with a clear warning that the API is experimental in the docstring)? This way we could encourage users to "kick the tires" and provide feedback sooner rather than later?

No pressure from me at all - Just interested in other perspectives.

[EDIT: On second thought, I don't think there is any great reason to rush on from_map. So, feel free to ignore the comment above :) ]

@fjetter
Copy link
Member

fjetter commented Apr 29, 2022

I am trying to find reviewers for the two PRs. If we can get them in before the release, that would be big

Edit: We just had a conversation about the two PRs and we prefer not to rush them in right now but rather take our time for proper review.

@jrbourbeau
Copy link
Member Author

Alright, it looks like all the parquet-related PRs are in and I see dask/distributed#6223 is closed (good work everybody 🎉). @fjetter I'm interpreting #240 (comment) as those PRs shouldn't block the release, is that correct?

@mrocklin
Copy link
Member

Correct. I think that you're good to go @jrbourbeau

@mrocklin
Copy link
Member

It's a good point to note that we're clearing out a few non-trivial failures these days on distributed. We might want to ask for an accelerated release in the future.

@jrbourbeau
Copy link
Member Author

Sounds good -- going to start pushing the release out now...

@jrbourbeau
Copy link
Member Author

2022.4.2 is out. Thanks all!

@mrocklin
Copy link
Member

mrocklin commented Apr 30, 2022 via email

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

No branches or pull requests

5 participants