Partially address lesv feedback.

Author: kurtisvg

cloud-sql/mysql/servlet/pom.xml

@@ -37,14 +37,6 @@
   </properties>
 
   <dependencies>
-
-    <!-- Required only for AppEngine Standard -->
-    <dependency>
-      <groupId>com.google.appengine</groupId>
-      <artifactId>appengine-api-1.0-sdk</artifactId>
-      <version>1.9.64</version>
-    </dependency>
-
     <dependency>
       <groupId>javax.servlet</groupId>
       <artifactId>javax.servlet-api</artifactId>
@@ -79,6 +71,7 @@
           <scanIntervalSeconds>1</scanIntervalSeconds>
         </configuration>
       </plugin>
+      <!-- Only required for AppEngine Deployments -->
       <plugin>
         <groupId>com.google.cloud.tools</groupId>
         <artifactId>appengine-maven-plugin</artifactId>

cloud-sql/mysql/servlet/src/main/java/com/example/cloudsql/ConnectionPoolContextListener.java

@@ -26,7 +26,7 @@
 import javax.servlet.annotation.WebListener;
 import javax.sql.DataSource;
 
-@WebListener
+@WebListener("Creates a connection pool that is stored in the Servlet's context for later use.")
 public class ConnectionPoolContextListener implements ServletContextListener {
 
   private static final String CLOUD_SQL_INSTANCE_NAME = System.getenv("CLOUD_SQL_INSTANCE_NAME");
@@ -74,7 +74,9 @@ private DataSource mysqlConnectionPool() {
 
     // [START cloud_sql_connection_lifetime]
     // maxLifetime is the maximum possible lifetime of a connection in the pool. Connections that
-    // live longer than this many milliseconds will be closed and reestablished between uses.
+    // live longer than this many milliseconds will be closed and reestablished between uses. This
+    // value should be several minutes shorter than the database's timeout value to avoid unexpected
+    // terminations.
     config.setMaxLifetime(1800000); // 30 minutes
     // [END cloud_sql_connection_lifetime]
 

cloud-sql/mysql/servlet/src/main/webapp/WEB-INF/appengine-web.xml

@@ -23,4 +23,4 @@
     <env-var name="DB_PASS" value="my-db-password" />
     <env-var name="DB_NAME" value="my_db" />
   </env-variables>
-</appengine-web-app>
+</appengine-web-app>