You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@beam.apache.org by ke...@apache.org on 2017/10/18 20:09:15 UTC

[3/7] beam git commit: Improve GcsFileSystem errors messages slightly

Improve GcsFileSystem errors messages slightly


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

Branch: refs/heads/master
Commit: 7409ca042f4cc7f57c02d2ab2843a3bbc833a49a
Parents: 2acdc74
Author: Kenneth Knowles <kl...@google.com>
Authored: Tue Oct 10 21:55:33 2017 -0700
Committer: Kenneth Knowles <ke...@apache.org>
Committed: Wed Oct 18 13:02:24 2017 -0700

----------------------------------------------------------------------
 .../apache/beam/sdk/extensions/gcp/storage/GcsFileSystem.java   | 5 +++--
 1 file changed, 3 insertions(+), 2 deletions(-)
----------------------------------------------------------------------


http://git-wip-us.apache.org/repos/asf/beam/blob/7409ca04/sdks/java/extensions/google-cloud-platform-core/src/main/java/org/apache/beam/sdk/extensions/gcp/storage/GcsFileSystem.java
----------------------------------------------------------------------
diff --git a/sdks/java/extensions/google-cloud-platform-core/src/main/java/org/apache/beam/sdk/extensions/gcp/storage/GcsFileSystem.java b/sdks/java/extensions/google-cloud-platform-core/src/main/java/org/apache/beam/sdk/extensions/gcp/storage/GcsFileSystem.java
index 6db0a01..f35c62a 100644
--- a/sdks/java/extensions/google-cloud-platform-core/src/main/java/org/apache/beam/sdk/extensions/gcp/storage/GcsFileSystem.java
+++ b/sdks/java/extensions/google-cloud-platform-core/src/main/java/org/apache/beam/sdk/extensions/gcp/storage/GcsFileSystem.java
@@ -88,10 +88,11 @@ class GcsFileSystem extends FileSystem<GcsResourceId> {
     ImmutableList.Builder<MatchResult> ret = ImmutableList.builder();
     for (Boolean isGlob : isGlobBooleans) {
       if (isGlob) {
-        checkState(globsMatchResults.hasNext(), "Expect globsMatchResults has next.");
+        checkState(globsMatchResults.hasNext(), "Expect globsMatchResults has next: %s", globs);
         ret.add(globsMatchResults.next());
       } else {
-        checkState(nonGlobsMatchResults.hasNext(), "Expect nonGlobsMatchResults has next.");
+        checkState(
+            nonGlobsMatchResults.hasNext(), "Expect nonGlobsMatchResults has next: %s", nonGlobs);
         ret.add(nonGlobsMatchResults.next());
       }
     }