You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@beam.apache.org by lc...@apache.org on 2017/04/30 04:58:44 UTC

[1/2] beam git commit: Remove useless continue statements

Repository: beam
Updated Branches:
  refs/heads/master 33358eb23 -> fc55d2f81


Remove useless continue statements


Project: http://git-wip-us.apache.org/repos/asf/beam/repo
Commit: http://git-wip-us.apache.org/repos/asf/beam/commit/cc7cfcce
Tree: http://git-wip-us.apache.org/repos/asf/beam/tree/cc7cfcce
Diff: http://git-wip-us.apache.org/repos/asf/beam/diff/cc7cfcce

Branch: refs/heads/master
Commit: cc7cfcce6315dd5227d55f791e47db6f8853e6f2
Parents: 33358eb
Author: wtanaka.com <wt...@yahoo.com>
Authored: Sat Apr 29 18:44:06 2017 -1000
Committer: Lukasz Cwik <lc...@google.com>
Committed: Sat Apr 29 21:57:57 2017 -0700

----------------------------------------------------------------------
 .../java/org/apache/beam/sdk/options/PipelineOptionsFactory.java    | 1 -
 .../apache/beam/sdk/io/range/ByteKeyRangeInterpolateKeyTest.java    | 1 -
 2 files changed, 2 deletions(-)
----------------------------------------------------------------------


http://git-wip-us.apache.org/repos/asf/beam/blob/cc7cfcce/sdks/java/core/src/main/java/org/apache/beam/sdk/options/PipelineOptionsFactory.java
----------------------------------------------------------------------
diff --git a/sdks/java/core/src/main/java/org/apache/beam/sdk/options/PipelineOptionsFactory.java b/sdks/java/core/src/main/java/org/apache/beam/sdk/options/PipelineOptionsFactory.java
index b947419..0f6bedf 100644
--- a/sdks/java/core/src/main/java/org/apache/beam/sdk/options/PipelineOptionsFactory.java
+++ b/sdks/java/core/src/main/java/org/apache/beam/sdk/options/PipelineOptionsFactory.java
@@ -1192,7 +1192,6 @@ public class PipelineOptionsFactory {
         method.property = propertyDescriptor;
         method.methodType = "setter";
         missingBeanMethods.add(method);
-        continue;
       }
     }
     throwForMissingBeanMethod(iface, missingBeanMethods);

http://git-wip-us.apache.org/repos/asf/beam/blob/cc7cfcce/sdks/java/core/src/test/java/org/apache/beam/sdk/io/range/ByteKeyRangeInterpolateKeyTest.java
----------------------------------------------------------------------
diff --git a/sdks/java/core/src/test/java/org/apache/beam/sdk/io/range/ByteKeyRangeInterpolateKeyTest.java b/sdks/java/core/src/test/java/org/apache/beam/sdk/io/range/ByteKeyRangeInterpolateKeyTest.java
index a6445eb..7e57a90 100644
--- a/sdks/java/core/src/test/java/org/apache/beam/sdk/io/range/ByteKeyRangeInterpolateKeyTest.java
+++ b/sdks/java/core/src/test/java/org/apache/beam/sdk/io/range/ByteKeyRangeInterpolateKeyTest.java
@@ -66,7 +66,6 @@ public class ByteKeyRangeInterpolateKeyTest {
         last = key;
       } catch (IllegalStateException e) {
         assertThat(message, e.getMessage(), containsString("near-empty ByteKeyRange"));
-        continue;
       }
     }
   }


[2/2] beam git commit: Remove useless continue statements

Posted by lc...@apache.org.
Remove useless continue statements

This closes #2787


Project: http://git-wip-us.apache.org/repos/asf/beam/repo
Commit: http://git-wip-us.apache.org/repos/asf/beam/commit/fc55d2f8
Tree: http://git-wip-us.apache.org/repos/asf/beam/tree/fc55d2f8
Diff: http://git-wip-us.apache.org/repos/asf/beam/diff/fc55d2f8

Branch: refs/heads/master
Commit: fc55d2f811a57d25e8d1e5f204d9a07e2c50a370
Parents: 33358eb cc7cfcc
Author: Lukasz Cwik <lc...@google.com>
Authored: Sat Apr 29 21:58:20 2017 -0700
Committer: Lukasz Cwik <lc...@google.com>
Committed: Sat Apr 29 21:58:20 2017 -0700

----------------------------------------------------------------------
 .../java/org/apache/beam/sdk/options/PipelineOptionsFactory.java    | 1 -
 .../apache/beam/sdk/io/range/ByteKeyRangeInterpolateKeyTest.java    | 1 -
 2 files changed, 2 deletions(-)
----------------------------------------------------------------------