Fix two checkstyle violations

One snuck in via apache#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.
mransley · Mar 22, 2017 · 362e7e8 · 362e7e8
1 parent dd0f8d9
commit 362e7e8
Show file tree

Hide file tree

Showing 2 changed files with 1 addition and 2 deletions.
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
@@ -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;

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
@@ -28,5 +28,5 @@ public interface MutationDetector extends AutoCloseable {
  /**
  * @throws IllegalMutationException if illegal mutations are detected.
  */
- void verifyUnmodified();
+ void verifyUnmodified() throws IllegalMutationException;
 }