Never mind. Updated pr based on the consensus. Best, tison.
Enrico Olivelli <eolive...@gmail.com> 于2020年3月31日周二 上午2:10写道： > Tison, > can you please update your PR and drop the PY2 script ? > > I apologize, I asked you to keep it, we could have done it simpler :-) > > Enrico > > Il giorno lun 30 mar 2020 alle ore 18:43 Patrick Hunt > <ph...@apache.org> ha scritto: > > > > p3 is fine with me. Trying to maintain multiple can be a hassle and p2 is > > EOL already. > > > > Patrick > > > > On Mon, Mar 30, 2020 at 5:19 AM Norbert Kalmar > <nkal...@cloudera.com.invalid> > > wrote: > > > > > +1 for v3 only (I have both v2/v3 interpreters, but as you mentioned, > we > > > should only need to maintain 1 script). > > > > > > - Norbert > > > > > > On Mon, Mar 30, 2020 at 1:35 PM Enrico Olivelli <eolive...@gmail.com> > > > wrote: > > > > > > > Il giorno lun 30 mar 2020 alle ore 13:33 Andor Molnar > > > > <an...@apache.org> ha scritto: > > > > > > > > > > Hi folks, > > > > > > > > > > We have a nice PR about upgrading the commit script to Python3: > > > > > > > > > > https://github.com/apache/zookeeper/pull/1295 > > > > > > > > > > I think we could just drop the previous v2 version with no > problem, but > > > > folks on the PR would like to keep it. > > > > > Source control keeps the old version anyway and we wouldn’t need to > > > > maintain 2 scripts in the long run. Is there anybody (committer) who > > > > doesn’t have python3 interpreter? > > > > > > > > > > Additionally Py2 is already EoL since this January. > > > > > > > > I am fine with Python3 only. > > > > > > > > Enrico > > > > > > > > > > > > > > > > > > Andor > > > > > > > > > > > > > > > > > >
