FYI, it looks like what happened is that the final commit in that PR didn't get pushed ti github before merging. Verifying locally and sending you a PR to fix this.
On Fri, Feb 22, 2019 at 11:56 AM Reuven Lax <[email protected]> wrote: > Sending a fix right now. > > On Fri, Feb 22, 2019 at 11:49 AM Ryan Williams <[email protected]> > wrote: > >> I sent a revert at #7928 <https://github.com/apache/beam/pull/7928> in >> case anyone wants to take a look >> >> On Fri, Feb 22, 2019 at 2:39 PM Ryan Williams <[email protected]> >> wrote: >> >>> AFAICT #7635 was merged with compileTestJava failures >>> <https://builds.apache.org/job/beam_PreCommit_Java_Phrase/754/console>, >>> and compileTestJava is broken on master now. >>> >>> My guess is it was thought that the Java PreCommit failure was due to >>> the flaky Kudu test (which also failed in the build in question), so the >>> other true failures were missed. >>> >>
