Repository: beam Updated Branches: refs/heads/master dd0f8d984 -> eb347bf0d
Fix two checkstyle violations One snuck in via https://github.com/apache/beam/pull/2264, the other one (in MutationDetector) I'm not sure - it's been around for long. I'm also pretty sure it's a checkstyle bug, but the throws declaration doesn't hurt and makes checkstyle stop complaining. Project: http://git-wip-us.apache.org/repos/asf/beam/repo Commit: http://git-wip-us.apache.org/repos/asf/beam/commit/362e7e87 Tree: http://git-wip-us.apache.org/repos/asf/beam/tree/362e7e87 Diff: http://git-wip-us.apache.org/repos/asf/beam/diff/362e7e87 Branch: refs/heads/master Commit: 362e7e87f24e7633a871a9de151f67ddc239ccfc Parents: dd0f8d9 Author: Eugene Kirpichov <[email protected]> Authored: Wed Mar 22 16:06:39 2017 -0700 Committer: Thomas Groh <[email protected]> Committed: Wed Mar 22 16:11:50 2017 -0700 ---------------------------------------------------------------------- .../java/org/apache/beam/runners/core/SimpleDoFnRunnerTest.java | 1 - .../src/main/java/org/apache/beam/sdk/util/MutationDetector.java | 2 +- 2 files changed, 1 insertion(+), 2 deletions(-) ---------------------------------------------------------------------- http://git-wip-us.apache.org/repos/asf/beam/blob/362e7e87/runners/core-java/src/test/java/org/apache/beam/runners/core/SimpleDoFnRunnerTest.java ---------------------------------------------------------------------- diff --git a/runners/core-java/src/test/java/org/apache/beam/runners/core/SimpleDoFnRunnerTest.java b/runners/core-java/src/test/java/org/apache/beam/runners/core/SimpleDoFnRunnerTest.java index d8c5149..4ae5332 100644 --- a/runners/core-java/src/test/java/org/apache/beam/runners/core/SimpleDoFnRunnerTest.java +++ b/runners/core-java/src/test/java/org/apache/beam/runners/core/SimpleDoFnRunnerTest.java @@ -18,7 +18,6 @@ package org.apache.beam.runners.core; import static org.hamcrest.Matchers.contains; -import static org.hamcrest.Matchers.equalTo; import static org.hamcrest.Matchers.is; import static org.hamcrest.Matchers.isA; import static org.junit.Assert.assertThat; http://git-wip-us.apache.org/repos/asf/beam/blob/362e7e87/sdks/java/core/src/main/java/org/apache/beam/sdk/util/MutationDetector.java ---------------------------------------------------------------------- diff --git a/sdks/java/core/src/main/java/org/apache/beam/sdk/util/MutationDetector.java b/sdks/java/core/src/main/java/org/apache/beam/sdk/util/MutationDetector.java index a626153..d3f2db2 100644 --- a/sdks/java/core/src/main/java/org/apache/beam/sdk/util/MutationDetector.java +++ b/sdks/java/core/src/main/java/org/apache/beam/sdk/util/MutationDetector.java @@ -28,5 +28,5 @@ public interface MutationDetector extends AutoCloseable { /** * @throws IllegalMutationException if illegal mutations are detected. */ - void verifyUnmodified(); + void verifyUnmodified() throws IllegalMutationException; }
