beam-commits mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From tg...@apache.org
Subject [2/2] beam git commit: Remove verifyDeterministic from StructuredCoder
Date Wed, 10 May 2017 17:50:04 GMT
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 <tgroh@google.com>
Authored: Tue May 9 18:09:37 2017 -0700
Committer: Thomas Groh <tgroh@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}
    *


Mime
View raw message