Re: Staged PIP package mysteriously ungzipped, non-installable inside the worker

2020-08-17 Thread Valentyn Tymofieiev
Will defer to the release manager; one reason to cherry-pick is that 2.24.0 will be the last release with Python 2 support, so Py2 users of Portable Python Local Runner might appreciate the fix, since they won't be able to use the next release. On Thu, Aug 13, 2020 at 6:28 PM Eugene Kirpichov wro

ANNOUNCE: Beam 2.24.0 will be the last release with Python 2.7 and Python 3.5 support.

2020-08-17 Thread Valentyn Tymofieiev
After the discussion[1] on dev@ (bcc'ed), we decided to extend our prior[2] proposal to sunset Python 2 by one more release, therefore we will be removing Python 2.7 support in Beam 2.25.0 [3]. Moving forward, to have a predictable cadence for adding and removing support for future Python versions

Re: Staged PIP package mysteriously ungzipped, non-installable inside the worker

2020-08-17 Thread Daniel Oliveira
Normally I'd say not to cherry-pick this since the issue is only affecting one runner and isn't really a regression, but given that it's the last Py2 release and there won't be a follow-up release that will be able to include this fix, I think it's worth making an exception this time. There should

Re: Staged PIP package mysteriously ungzipped, non-installable inside the worker

2020-08-17 Thread Robert Bradshaw
+1 Thanks, Eugene, for finding and fixing this! FWIW, most use of Python from the Python Portable Runner used the embedded environment (this is the default direct runner), so dependencies are already present. On Mon, Aug 17, 2020 at 3:19 PM Daniel Oliveira wrote: > > Normally I'd say not to che

Re: Staged PIP package mysteriously ungzipped, non-installable inside the worker

2020-08-17 Thread Robert Bradshaw
I checked Java, it looks like the way things are structured we do not have that bug there. On Mon, Aug 17, 2020 at 3:31 PM Robert Bradshaw wrote: > > +1 > > Thanks, Eugene, for finding and fixing this! > > FWIW, most use of Python from the Python Portable Runner used the > embedded environment (t