Re: Committed without review. Sorry.
Thanks for being transparent about this, Pablo. :) 👍 Best, Łukasz pt., 13 gru 2019 o 06:32 Kenneth Knowles napisał(a): > Thanks for alerting dev@. > > IMO this was handled perfectly by all parties. > > Kenn > > On Thu, Dec 12, 2019 at 4:09 PM Valentyn Tymofieiev > wrote: > >> The change LGTM, so you can consider it reviewed. In general it would be >> nice to set up alerts to catch these situations, to make sure they don't go >> unnoticed. >> >> Also as a reminder - please don't commit or merge PRs into release >> branches without a review from a release manager. >> >> On Thu, Dec 12, 2019 at 3:44 PM Pablo Estrada wrote: >> >>> Seed job runs okay: >>> https://builds.apache.org/job/beam_SeedJob_Standalone/3865/console >>> >>> >>> On Thu, Dec 12, 2019 at 3:28 PM Pablo Estrada >>> wrote: >>> I accidentally committed a small change to master: https://github.com/apache/beam/commit/6018326ffe74aac7d8c44ded296b92f8b5c0b556 I am verifying that this works as intended for now. What should we do about this? Revert? Leave as is if it works fine? Best -P >>>
Re: Committed without review. Sorry.
Thanks for alerting dev@. IMO this was handled perfectly by all parties. Kenn On Thu, Dec 12, 2019 at 4:09 PM Valentyn Tymofieiev wrote: > The change LGTM, so you can consider it reviewed. In general it would be > nice to set up alerts to catch these situations, to make sure they don't go > unnoticed. > > Also as a reminder - please don't commit or merge PRs into release > branches without a review from a release manager. > > On Thu, Dec 12, 2019 at 3:44 PM Pablo Estrada wrote: > >> Seed job runs okay: >> https://builds.apache.org/job/beam_SeedJob_Standalone/3865/console >> >> >> On Thu, Dec 12, 2019 at 3:28 PM Pablo Estrada wrote: >> >>> I accidentally committed a small change to master: >>> https://github.com/apache/beam/commit/6018326ffe74aac7d8c44ded296b92f8b5c0b556 >>> >>> I am verifying that this works as intended for now. >>> >>> What should we do about this? Revert? Leave as is if it works fine? >>> Best >>> -P >>> >>
Re: Committed without review. Sorry.
The change LGTM, so you can consider it reviewed. In general it would be nice to set up alerts to catch these situations, to make sure they don't go unnoticed. Also as a reminder - please don't commit or merge PRs into release branches without a review from a release manager. On Thu, Dec 12, 2019 at 3:44 PM Pablo Estrada wrote: > Seed job runs okay: > https://builds.apache.org/job/beam_SeedJob_Standalone/3865/console > > > On Thu, Dec 12, 2019 at 3:28 PM Pablo Estrada wrote: > >> I accidentally committed a small change to master: >> https://github.com/apache/beam/commit/6018326ffe74aac7d8c44ded296b92f8b5c0b556 >> >> I am verifying that this works as intended for now. >> >> What should we do about this? Revert? Leave as is if it works fine? >> Best >> -P >> >
Re: Committed without review. Sorry.
Seed job runs okay: https://builds.apache.org/job/beam_SeedJob_Standalone/3865/console On Thu, Dec 12, 2019 at 3:28 PM Pablo Estrada wrote: > I accidentally committed a small change to master: > https://github.com/apache/beam/commit/6018326ffe74aac7d8c44ded296b92f8b5c0b556 > > I am verifying that this works as intended for now. > > What should we do about this? Revert? Leave as is if it works fine? > Best > -P >
Committed without review. Sorry.
I accidentally committed a small change to master: https://github.com/apache/beam/commit/6018326ffe74aac7d8c44ded296b92f8b5c0b556 I am verifying that this works as intended for now. What should we do about this? Revert? Leave as is if it works fine? Best -P