---

Ajay Kannan · Ajay Kannan · commit c72e474ba551 · 2016-01-25T12:44:49.000-08:00
yaml --- r: 6847 b: refs/heads/tswast-patch-1 c: 03ca1bf h: refs/heads/master i: 6845: aca9ee4 6843: 67548a1 6839: fbf1fc9 6831: 0e7c814 6815: a203d4f 6783: 2e110e9
diff --git a/[refs] b/[refs]
@@ -57,5 +57,5 @@ refs/tags/v0.18.0: 9d193c4c4b9d1c6f21515dd8e50836b9194ec9bb
 refs/tags/v0.19.0: e67b56e4d8dad5f9a7b38c9b2107c23c828f2ed5
 refs/tags/v0.20.0: 839f7fb7156535146aa1cb2c5aadd8d375d854e8
 refs/tags/v0.20.1: 370471f437f1f4f68a11e068df5cd6bf39edb1fa
-refs/heads/tswast-patch-1: f11195303f281414fa10b74ca402c0a99bf77313
+refs/heads/tswast-patch-1: 03ca1bfbf214900f5c0dbd06fb982fe396425426
 refs/heads/pubsub-streaming-pull: 19262b752ee874eb2ca3b950eb2aef44d5a5267b
diff --git a/branches/tswast-patch-1/gcloud-java-datastore/src/main/java/com/google/gcloud/datastore/Cursor.java b/branches/tswast-patch-1/gcloud-java-datastore/src/main/java/com/google/gcloud/datastore/Cursor.java
@@ -17,20 +17,14 @@
 package com.google.gcloud.datastore;
 
 import static com.google.common.base.Preconditions.checkNotNull;
-import static java.nio.charset.StandardCharsets.UTF_8;
 
 import com.google.api.services.datastore.DatastoreV1;
 import com.google.api.services.datastore.DatastoreV1.Value;
 import com.google.common.base.MoreObjects;
 import com.google.common.base.MoreObjects.ToStringHelper;
+import com.google.common.io.BaseEncoding;
 import com.google.protobuf.ByteString;
 import com.google.protobuf.InvalidProtocolBufferException;
-import com.google.protobuf.TextFormat;
-import com.google.protobuf.TextFormat.ParseException;
-
-import java.io.UnsupportedEncodingException;
-import java.net.URLDecoder;
-import java.net.URLEncoder;
 
 /**
  * A Google Cloud Datastore cursor.
@@ -74,23 +68,16 @@ ByteString byteString() {
    * Returns the cursor in an encoded form that can be used as part of a URL.
    */
   public String toUrlSafe() {
-    try {
-      return URLEncoder.encode(TextFormat.printToString(toPb()), UTF_8.name());
-    } catch (UnsupportedEncodingException e) {
-      throw new IllegalStateException("Unexpected encoding exception", e);
-    }
+    return BaseEncoding.base64Url().encode(byteString.toByteArray());
   }
 
   /**
    * Create a {@code Cursor} given its URL safe encoded form.
    */
   public static Cursor fromUrlSafe(String urlSafe) {
     try {
-      String utf8Str = URLDecoder.decode(urlSafe, UTF_8.name());
-      DatastoreV1.Value.Builder builder = DatastoreV1.Value.newBuilder();
-      TextFormat.merge(utf8Str, builder);
-      return fromPb(builder.build());
-    } catch (UnsupportedEncodingException | ParseException e) {
+      return Cursor.copyFrom(BaseEncoding.base64Url().decode(urlSafe));
+    } catch (IllegalArgumentException e) {
       throw new IllegalStateException("Unexpected decoding exception", e);
     }
   }
diff --git a/branches/tswast-patch-1/gcloud-java-datastore/src/main/java/com/google/gcloud/datastore/StructuredQuery.java b/branches/tswast-patch-1/gcloud-java-datastore/src/main/java/com/google/gcloud/datastore/StructuredQuery.java
@@ -782,7 +782,7 @@ public StructuredQuery<V> build() {
     }
   }
 
-  static final class Builder<V> extends BaseBuilder<V, Builder<V>> {
+  public static final class Builder<V> extends BaseBuilder<V, Builder<V>> {
 
     Builder(ResultType<V> resultType) {
       super(resultType);

Original file line number	Diff line number	Diff line change
`@@ -782,7 +782,7 @@ public StructuredQuery<V> build() {`
`782`	`782`	`}`
`783`	`783`	`}`
`784`	`784`
`785`		`- static final class Builder<V> extends BaseBuilder<V, Builder<V>> {`
	`785`	`+ public static final class Builder<V> extends BaseBuilder<V, Builder<V>> {`
`786`	`786`
`787`	`787`	`Builder(ResultType<V> resultType) {`
`788`	`788`	`super(resultType);`