Update of Flexible folder and the top-level parent pom. (#1004)

* Updated flexible/analytics.

* Updated flexible/async-rest

* Update flexible/cloud-tasks.

* Updated flexible/cloudsql

* Updated flexible/cloudstorage

* Updated flexible/cron

* Updated flexible/disk

* Updated flexible/endpoints

* Updated flexible/errorreporting

* Updated flexible/extending-runtime

* Updated flexible/gaeinfo

* Updated flexible/helloworld.

* Updated flexible/mailgun

* Updated flexible/mailjet

* Updated flexible/memcache.

* Updated flexible/postgres

* Updated flexible/pubsub.

* Updated flexible/sendgrid

* Updated flexible/sparkjava

* Update flexible/static-files.

* Update flexible/twilio.

* Updated flexible folder and a few missed violations.

* Updated storage/xmp-api/cmdline-sample

* Updated storage/xml-api/serviceaccount-appengine/sample

* Updated vision/beta/cloud-client

* Updated final pom.xml.

Author: kurtisvg

flexible/README.md

@@ -1,5 +1,8 @@
 # Google App Engine Flexible Environment Java Samples
 
+<a href="https://console.cloud.google.com/cloudshell/open?git_repo=https://github.com/GoogleCloudPlatform/java-docs-samples&page=editor&open_in_editor=flexible/README.md">
+<img alt="Open in Cloud Shell" src ="http://gstatic.com/cloudssh/images/open-btn.png"></a>
+
 This is a repository that contains Java code samples for [Google App Engine
 flexible environment][aeflex-docs].
 

flexible/analytics/pom.xml

@@ -20,17 +20,22 @@
   <groupId>com.example.flexible</groupId>
   <artifactId>flexible-analytics</artifactId>
 
+  <!--
+    The parent pom defines common style checks and testing strategies for our samples.
+    Removing or replacing it should not affect the execution of the samples in anyway.
+  -->
   <parent>
-    <artifactId>appengine-flexible</artifactId>
-    <groupId>com.google.cloud</groupId>
-    <version>1.0.0</version>
-    <relativePath>..</relativePath>
+    <groupId>com.google.cloud.samples</groupId>
+    <artifactId>shared-configuration</artifactId>
+    <version>1.0.8</version>
   </parent>
 
   <properties>
+    <maven.compiler.target>1.8</maven.compiler.target>
+    <maven.compiler.source>1.8</maven.compiler.source>
+
     <appengine.maven.plugin>1.3.2</appengine.maven.plugin>
     <jetty>9.4.4.v20170414</jetty>
-
     <failOnMissingWebXml>false</failOnMissingWebXml> <!-- REQUIRED -->
   </properties>
 

flexible/analytics/src/main/java/com/example/analytics/AnalyticsServlet.java

@@ -1,16 +1,19 @@
 /*
  * Copyright 2015 Google Inc.
  *
- * <p>Licensed under the Apache License, Version 2.0 (the "License"); you may not use this file
- * except in compliance with the License. You may obtain a copy of the License at
+ * Licensed under the Apache License, Version 2.0 (the "License");
+ * you may not use this file except in compliance with the License.
+ * You may obtain a copy of the License at
  *
- * <p>http://www.apache.org/licenses/LICENSE-2.0
+ * http://www.apache.org/licenses/LICENSE-2.0
  *
- * <p>Unless required by applicable law or agreed to in writing, software distributed under the
- * License is distributed on an "AS IS" BASIS, WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either
- * express or implied. See the License for the specific language governing permissions and
+ * Unless required by applicable law or agreed to in writing, software
+ * distributed under the License is distributed on an "AS IS" BASIS,
+ * WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied.
+ * See the License for the specific language governing permissions and
  * limitations under the License.
  */
+
 package com.example.analytics;
 
 import java.io.IOException;

flexible/async-rest/README.md

@@ -1,6 +1,10 @@
 
 # Example Web App Using Asynchronous Servlets #
 
+<a href="https://console.cloud.google.com/cloudshell/open?git_repo=https://github.com/GoogleCloudPlatform/java-docs-samples&page=editor&open_in_editor=flexible/async-rest/README.md">
+<img alt="Open in Cloud Shell" src ="http://gstatic.com/cloudssh/images/open-btn.png"></a>
+
+
 This web app demonstrates using asynchronous servlet techniques to reduce server resources.
 
 The code for this tutorial is [here](https://github.com/GoogleCloudPlatform/java-docs-samples/tree/master/flexible/async-rest).

flexible/async-rest/pom.xml

@@ -21,11 +21,14 @@
   <version>1.0.0-SNAPSHOT</version>
   <packaging>war</packaging>
 
+  <!--
+    The parent pom defines common style checks and testing strategies for our samples.
+    Removing or replacing it should not affect the execution of the samples in anyway.
+  -->
   <parent>
-    <artifactId>appengine-flexible</artifactId>
-    <groupId>com.google.cloud</groupId>
-    <version>1.0.0</version>
-    <relativePath>..</relativePath>
+    <groupId>com.google.cloud.samples</groupId>
+    <artifactId>shared-configuration</artifactId>
+    <version>1.0.8</version>
   </parent>
 
   <properties>

flexible/async-rest/src/main/java/com/google/appengine/demos/DumpServlet.java

@@ -19,7 +19,6 @@
 import java.io.IOException;
 import java.io.PrintWriter;
 import java.util.Collections;
-
 import javax.servlet.ServletException;
 import javax.servlet.http.HttpServlet;
 import javax.servlet.http.HttpServletRequest;

flexible/async-rest/src/main/java/com/google/appengine/demos/asyncrest/AbstractRestServlet.java

@@ -23,7 +23,6 @@
 import java.util.Iterator;
 import java.util.Map;
 import java.util.Queue;
-
 import javax.servlet.ServletConfig;
 import javax.servlet.ServletException;
 import javax.servlet.UnavailableException;

flexible/async-rest/src/main/java/com/google/appengine/demos/asyncrest/AsyncRestServlet.java

@@ -16,28 +16,26 @@
 
 package com.google.appengine.demos.asyncrest;
 
-import org.eclipse.jetty.client.HttpClient;
-import org.eclipse.jetty.client.api.Response;
-import org.eclipse.jetty.client.api.Result;
-import org.eclipse.jetty.http.HttpMethod;
-import org.eclipse.jetty.util.BufferUtil;
-import org.eclipse.jetty.util.Utf8StringBuilder;
-import org.eclipse.jetty.util.ajax.JSON;
-import org.eclipse.jetty.util.ssl.SslContextFactory;
-
 import java.io.IOException;
 import java.io.PrintWriter;
 import java.nio.ByteBuffer;
 import java.util.Map;
 import java.util.Queue;
 import java.util.concurrent.ConcurrentLinkedQueue;
 import java.util.concurrent.atomic.AtomicInteger;
-
 import javax.servlet.AsyncContext;
 import javax.servlet.ServletConfig;
 import javax.servlet.ServletException;
 import javax.servlet.http.HttpServletRequest;
 import javax.servlet.http.HttpServletResponse;
+import org.eclipse.jetty.client.HttpClient;
+import org.eclipse.jetty.client.api.Response;
+import org.eclipse.jetty.client.api.Result;
+import org.eclipse.jetty.http.HttpMethod;
+import org.eclipse.jetty.util.BufferUtil;
+import org.eclipse.jetty.util.Utf8StringBuilder;
+import org.eclipse.jetty.util.ajax.JSON;
+import org.eclipse.jetty.util.ssl.SslContextFactory;
 
 /**
  * Servlet which makes REST calls asynchronously.
@@ -134,8 +132,6 @@ void doComplete() {
 
     // We have results!
     // Generate the response
-    String thumbs = generateResults(results);
-
     response.setContentType("text/html");
     PrintWriter out = response.getWriter();
     out.println("<html><head>");
@@ -176,6 +172,8 @@ void doComplete() {
 
     out.println("<br/>");
     out.print("First 5 results of " + results.size() + ":<br/>");
+
+    String thumbs = generateResults(results);
     if ("".equals(thumbs)) {
       out.print("<i>No results. Ensure " + APPKEY + " property is set correctly.</i>");
     } else {

flexible/async-rest/src/main/java/com/google/appengine/demos/asyncrest/SerialRestServlet.java

@@ -16,8 +16,6 @@
 
 package com.google.appengine.demos.asyncrest;
 
-import org.eclipse.jetty.util.ajax.JSON;
-
 import java.io.BufferedReader;
 import java.io.IOException;
 import java.io.InputStreamReader;
@@ -27,10 +25,10 @@
 import java.util.LinkedList;
 import java.util.Map;
 import java.util.Queue;
-
 import javax.servlet.ServletException;
 import javax.servlet.http.HttpServletRequest;
 import javax.servlet.http.HttpServletResponse;
+import org.eclipse.jetty.util.ajax.JSON;
 
 /**
  * Servlet which makes REST calls serially.
@@ -41,17 +39,21 @@
  * <dd>The Google app key to use</dd>
  * </dl>
  */
+
 public class SerialRestServlet extends AbstractRestServlet {
 
+  //CHECKSTYLE OFF: VariableDeclarationUsageDistance
   @Override
   protected void doGet(HttpServletRequest request, HttpServletResponse response)
       throws ServletException, IOException {
     if (key == null) {
       response.sendError(500, APPKEY + " not set");
       return;
     }
+
     long start = System.nanoTime();
 
+
     String loc = sanitize(request.getParameter(LOC_PARAM));
     String lat = sanitize(request.getParameter(LATITUDE_PARAM));
     String longitude = sanitize(request.getParameter(LONGITUDE_PARAM));
@@ -118,6 +120,7 @@ protected void doGet(HttpServletRequest request, HttpServletResponse response)
     out.println("</body></html>");
     out.close();
   }
+  //CHECKSTYLE ON: VariableDeclarationUsageDistance
 
   /**
    * Handle HTTP POST request.
@@ -130,4 +133,4 @@ protected void doPost(HttpServletRequest request, HttpServletResponse response)
     doGet(request, response);
   }
 
-}
+}

flexible/cloud-tasks/README.md

@@ -1,5 +1,8 @@
 # Google Cloud Tasks App Engine Flexible Queue Samples
 
+<a href="https://console.cloud.google.com/cloudshell/open?git_repo=https://github.com/GoogleCloudPlatform/java-docs-samples&page=editor&open_in_editor=flexible/cloud-tasks/README.md">
+<img alt="Open in Cloud Shell" src ="http://gstatic.com/cloudssh/images/open-btn.png"></a>
+
 This is a sample AppEngine app demonstrating use of the Cloud Tasks API
 using App Engine Queues.