You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@beam.apache.org by tg...@apache.org on 2017/05/10 17:50:04 UTC
[2/2] beam git commit: Remove verifyDeterministic from StructuredCoder
Remove verifyDeterministic from StructuredCoder
This is replaced within all uses in the Core SDK by
Coder.verifyDeterministic(Coder, String, List<Coder>), which is a
general utility.
Project: http://git-wip-us.apache.org/repos/asf/beam/repo
Commit: http://git-wip-us.apache.org/repos/asf/beam/commit/da5b3660
Tree: http://git-wip-us.apache.org/repos/asf/beam/tree/da5b3660
Diff: http://git-wip-us.apache.org/repos/asf/beam/diff/da5b3660
Branch: refs/heads/release-2.0.0
Commit: da5b3660c9e92991cb27c6f54d30eeffbe6122f6
Parents: 60fc4dc
Author: Thomas Groh <tg...@google.com>
Authored: Tue May 9 18:09:37 2017 -0700
Committer: Thomas Groh <tg...@google.com>
Committed: Wed May 10 10:45:40 2017 -0700
----------------------------------------------------------------------
.../apache/beam/sdk/coders/StructuredCoder.java | 17 -----------------
1 file changed, 17 deletions(-)
----------------------------------------------------------------------
http://git-wip-us.apache.org/repos/asf/beam/blob/da5b3660/sdks/java/core/src/main/java/org/apache/beam/sdk/coders/StructuredCoder.java
----------------------------------------------------------------------
diff --git a/sdks/java/core/src/main/java/org/apache/beam/sdk/coders/StructuredCoder.java b/sdks/java/core/src/main/java/org/apache/beam/sdk/coders/StructuredCoder.java
index 42c0598..db5900b 100644
--- a/sdks/java/core/src/main/java/org/apache/beam/sdk/coders/StructuredCoder.java
+++ b/sdks/java/core/src/main/java/org/apache/beam/sdk/coders/StructuredCoder.java
@@ -18,7 +18,6 @@
package org.apache.beam.sdk.coders;
import java.io.ByteArrayOutputStream;
-import java.util.Arrays;
import java.util.Collections;
import java.util.List;
import org.apache.beam.sdk.values.TypeDescriptor;
@@ -98,22 +97,6 @@ public abstract class StructuredCoder<T> extends Coder<T> {
return builder.toString();
}
- protected void verifyDeterministic(String message, Iterable<Coder<?>> coders)
- throws NonDeterministicException {
- for (Coder<?> coder : coders) {
- try {
- coder.verifyDeterministic();
- } catch (NonDeterministicException e) {
- throw new NonDeterministicException(this, message, e);
- }
- }
- }
-
- protected void verifyDeterministic(String message, Coder<?>... coders)
- throws NonDeterministicException {
- verifyDeterministic(message, Arrays.asList(coders));
- }
-
/**
* {@inheritDoc}
*