I would opt for (2), but I'm not sure who has permissions to do that. It should be easy to re-merge the couple of things that have gone in since then.
On Tue, Mar 6, 2018 at 2:43 PM Kenneth Knowles <k...@google.com> wrote: > Hi all, > > You may have noticed that our tests are red. A pull request that was meant > for the Go SDK branch accidentally got merged onto the master branch. > Things have been merged to master since then. > > I've opened a revert at https://github.com/apache/beam/pull/4808 > > The next time there is a master to go-sdk merge it will need to be > re-reverted. > > Two other options are (1) leave it there and disable it in whatever way > and (2) rebase dropping the commit and force push master (breaks all > checkouts that are past it). > > Kenn >
