---

mziccard · mziccard · commit 1f24d6552b75 · 2015-10-28T15:18:12.000-07:00
yaml --- r: 6383 b: refs/heads/tswast-patch-1 c: ff9f60c h: refs/heads/master i: 6381: 9edd0b7 6379: e853439 6375: c2f1b5d 6367: 47b578b
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: 22ca3f92e81a6fa1522dc922178652fd103bf312
+refs/heads/tswast-patch-1: ff9f60cb609f18da1c40f69fd5db5e8c6ae97ecd
 refs/heads/pubsub-streaming-pull: 19262b752ee874eb2ca3b950eb2aef44d5a5267b
diff --git a/branches/tswast-patch-1/gcloud-java-core/src/main/java/com/google/gcloud/ServiceOptions.java b/branches/tswast-patch-1/gcloud-java-core/src/main/java/com/google/gcloud/ServiceOptions.java
@@ -411,8 +411,6 @@ private static boolean isWindows() {
   }
 
   protected static String getAppEngineProjectId() {
-    // TODO(ozarov): An alternative to reflection would be to depend on AE api jar:
-    // http://mvnrepository.com/artifact/com.google.appengine/appengine-api-1.0-sdk/1.2.0
     try {
       Class<?> factoryClass =
           Class.forName("com.google.appengine.api.appidentity.AppIdentityServiceFactory");
diff --git a/branches/tswast-patch-1/gcloud-java-datastore/src/main/java/com/google/gcloud/datastore/DatastoreOptions.java b/branches/tswast-patch-1/gcloud-java-datastore/src/main/java/com/google/gcloud/datastore/DatastoreOptions.java
@@ -172,8 +172,6 @@ public String namespace() {
   }
 
   private static String defaultNamespace() {
-    // TODO(ozarov): An alternative to reflection would be to depend on AE api jar:
-    // http://mvnrepository.com/artifact/com.google.appengine/appengine-api-1.0-sdk/1.2.0
     try {
       Class<?> clazz = Class.forName("com.google.appengine.api.NamespaceManager");
       Method method = clazz.getMethod("get");
diff --git a/branches/tswast-patch-1/gcloud-java-datastore/src/test/java/com/google/gcloud/datastore/DatastoreTest.java b/branches/tswast-patch-1/gcloud-java-datastore/src/test/java/com/google/gcloud/datastore/DatastoreTest.java
@@ -459,7 +459,6 @@ public void testRunStructuredQuery() {
     assertEquals("Dan", entity.getString("name"));
     assertEquals(2, entity.properties().size());
     assertFalse(results4.hasNext());
-    // TODO(ozarov): construct a test to verify nextQuery/pagination
   }
 
   @Test

Original file line number	Diff line number	Diff line change
`@@ -411,8 +411,6 @@ private static boolean isWindows() {`
`411`	`411`	`}`
`412`	`412`
`413`	`413`	`protected static String getAppEngineProjectId() {`
`414`		`- // TODO(ozarov): An alternative to reflection would be to depend on AE api jar:`
`415`		`- // http://mvnrepository.com/artifact/com.google.appengine/appengine-api-1.0-sdk/1.2.0`
`416`	`414`	`try {`
`417`	`415`	`Class<?> factoryClass =`
`418`	`416`	`Class.forName("com.google.appengine.api.appidentity.AppIdentityServiceFactory");`
Original file line number	Diff line number	Diff line change
`@@ -172,8 +172,6 @@ public String namespace() {`
`172`	`172`	`}`
`173`	`173`
`174`	`174`	`private static String defaultNamespace() {`
`175`		`- // TODO(ozarov): An alternative to reflection would be to depend on AE api jar:`
`176`		`- // http://mvnrepository.com/artifact/com.google.appengine/appengine-api-1.0-sdk/1.2.0`
`177`	`175`	`try {`
`178`	`176`	`Class<?> clazz = Class.forName("com.google.appengine.api.NamespaceManager");`
`179`	`177`	`Method method = clazz.getMethod("get");`
Original file line number	Diff line number	Diff line change
`@@ -459,7 +459,6 @@ public void testRunStructuredQuery() {`
`459`	`459`	`assertEquals("Dan", entity.getString("name"));`
`460`	`460`	`assertEquals(2, entity.properties().size());`
`461`	`461`	`assertFalse(results4.hasNext());`
`462`		`- // TODO(ozarov): construct a test to verify nextQuery/pagination`
`463`	`462`	`}`
`464`	`463`
`465`	`464`	`@Test`