You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@calcite.apache.org by jh...@apache.org on 2014/08/25 20:42:28 UTC

[3/8] git commit: Fix 'mvn site' on JDK 1.8 and enable in Travis-CI.

Fix 'mvn site' on JDK 1.8 and enable in Travis-CI.

Fix several javadoc errors and warnings.

Remove some duplicate package-info.java files.

Remove versions of maven plugins where apache pom already specifies a version.


Project: http://git-wip-us.apache.org/repos/asf/incubator-optiq/repo
Commit: http://git-wip-us.apache.org/repos/asf/incubator-optiq/commit/55fbf047
Tree: http://git-wip-us.apache.org/repos/asf/incubator-optiq/tree/55fbf047
Diff: http://git-wip-us.apache.org/repos/asf/incubator-optiq/diff/55fbf047

Branch: refs/heads/master
Commit: 55fbf047681affa637825c3108df1d1a80855063
Parents: 0d84e62
Author: Julian Hyde <jh...@apache.org>
Authored: Tue Aug 19 12:28:55 2014 -0700
Committer: Julian Hyde <jh...@apache.org>
Committed: Fri Aug 22 00:15:23 2014 -0700

----------------------------------------------------------------------
 .travis.yml                                     |  4 ++-
 .../hydromatic/avatica/AvaticaConnection.java   |  1 +
 .../net/hydromatic/avatica/AvaticaFactory.java  |  1 +
 .../avatica/AvaticaPreparedStatement.java       |  3 ++
 .../java/net/hydromatic/avatica/Cursor.java     |  5 ++-
 .../avatica/test/ConnectStringParserTest.java   |  2 --
 .../eigenbase/rel/metadata/RelMdPredicates.java | 17 ++++++-----
 .../java/org/eigenbase/test/DiffRepository.java |  3 +-
 .../org/eigenbase/test/SqlToRelTestBase.java    |  1 +
 .../eigenbase/test/concurrent/package-info.java | 11 +++----
 .../net/hydromatic/optiq/test/package-info.java | 23 --------------
 pom.xml                                         | 32 +-------------------
 .../net/hydromatic/optiq/test/package-info.java | 23 --------------
 .../optiq/impl/splunk/search/package-info.java  |  4 +--
 .../optiq/impl/splunk/util/package-info.java    |  4 +--
 .../net/hydromatic/optiq/test/package-info.java | 23 --------------
 src/main/config/checkstyle/suppressions.xml     |  3 ++
 .../net/hydromatic/optiq/StatementTest.java     | 16 ++++++++++
 .../java/net/hydromatic/optiq/package-info.java | 25 ---------------
 19 files changed, 53 insertions(+), 148 deletions(-)
----------------------------------------------------------------------


http://git-wip-us.apache.org/repos/asf/incubator-optiq/blob/55fbf047/.travis.yml
----------------------------------------------------------------------
diff --git a/.travis.yml b/.travis.yml
index caceb4e..720e293 100644
--- a/.travis.yml
+++ b/.travis.yml
@@ -24,5 +24,7 @@ jdk:
 branches:
   only:
     - master
-
+    - javadoc
+script:
+  mvn test site
 # End .travis.yml

http://git-wip-us.apache.org/repos/asf/incubator-optiq/blob/55fbf047/avatica/src/main/java/net/hydromatic/avatica/AvaticaConnection.java
----------------------------------------------------------------------
diff --git a/avatica/src/main/java/net/hydromatic/avatica/AvaticaConnection.java b/avatica/src/main/java/net/hydromatic/avatica/AvaticaConnection.java
index 9b9994d..c05e89f 100644
--- a/avatica/src/main/java/net/hydromatic/avatica/AvaticaConnection.java
+++ b/avatica/src/main/java/net/hydromatic/avatica/AvaticaConnection.java
@@ -371,6 +371,7 @@ public abstract class AvaticaConnection implements Connection {
   /**
    * Executes a parsed query, closing any previously open result set.
    *
+   * @param statement     Statement
    * @param prepareResult Parsed query
    * @return Result set
    * @throws java.sql.SQLException if a database error occurs

http://git-wip-us.apache.org/repos/asf/incubator-optiq/blob/55fbf047/avatica/src/main/java/net/hydromatic/avatica/AvaticaFactory.java
----------------------------------------------------------------------
diff --git a/avatica/src/main/java/net/hydromatic/avatica/AvaticaFactory.java b/avatica/src/main/java/net/hydromatic/avatica/AvaticaFactory.java
index e48e968..bdb9c0f 100644
--- a/avatica/src/main/java/net/hydromatic/avatica/AvaticaFactory.java
+++ b/avatica/src/main/java/net/hydromatic/avatica/AvaticaFactory.java
@@ -57,6 +57,7 @@ public interface AvaticaFactory {
    *
    * @param statement Statement
    * @param prepareResult Prepared statement
+   * @param timeZone Time zone
    * @return Result set
    */
   AvaticaResultSet newResultSet(

http://git-wip-us.apache.org/repos/asf/incubator-optiq/blob/55fbf047/avatica/src/main/java/net/hydromatic/avatica/AvaticaPreparedStatement.java
----------------------------------------------------------------------
diff --git a/avatica/src/main/java/net/hydromatic/avatica/AvaticaPreparedStatement.java b/avatica/src/main/java/net/hydromatic/avatica/AvaticaPreparedStatement.java
index b507b60..04534b8 100644
--- a/avatica/src/main/java/net/hydromatic/avatica/AvaticaPreparedStatement.java
+++ b/avatica/src/main/java/net/hydromatic/avatica/AvaticaPreparedStatement.java
@@ -43,6 +43,9 @@ public abstract class AvaticaPreparedStatement
    *
    * @param connection Connection
    * @param prepareResult Result of preparing statement
+   * @param resultSetType Result set type
+   * @param resultSetConcurrency Result set concurrency
+   * @param resultSetHoldability Result set holdability
    * @throws SQLException If fails due to underlying implementation reasons.
    */
   protected AvaticaPreparedStatement(

http://git-wip-us.apache.org/repos/asf/incubator-optiq/blob/55fbf047/avatica/src/main/java/net/hydromatic/avatica/Cursor.java
----------------------------------------------------------------------
diff --git a/avatica/src/main/java/net/hydromatic/avatica/Cursor.java b/avatica/src/main/java/net/hydromatic/avatica/Cursor.java
index 1978cc2..f719852 100644
--- a/avatica/src/main/java/net/hydromatic/avatica/Cursor.java
+++ b/avatica/src/main/java/net/hydromatic/avatica/Cursor.java
@@ -45,6 +45,8 @@ public interface Cursor extends Closeable {
    * Moves to the next row.
    *
    * @return Whether moved
+   *
+   * @throws SQLException on database error
    */
   boolean next() throws SQLException;
 
@@ -55,7 +57,8 @@ public interface Cursor extends Closeable {
 
   /**
    * Returns whether the last value returned was null.
-   * @throws SQLException
+   *
+   * @throws SQLException on database error
    */
   boolean wasNull() throws SQLException;
 

http://git-wip-us.apache.org/repos/asf/incubator-optiq/blob/55fbf047/avatica/src/test/java/net/hydromatic/avatica/test/ConnectStringParserTest.java
----------------------------------------------------------------------
diff --git a/avatica/src/test/java/net/hydromatic/avatica/test/ConnectStringParserTest.java b/avatica/src/test/java/net/hydromatic/avatica/test/ConnectStringParserTest.java
index 2d1d93c..38dd5ee 100644
--- a/avatica/src/test/java/net/hydromatic/avatica/test/ConnectStringParserTest.java
+++ b/avatica/src/test/java/net/hydromatic/avatica/test/ConnectStringParserTest.java
@@ -138,8 +138,6 @@ public class ConnectStringParserTest {
    * href="http://msdn.microsoft.com/library/default.asp?url=/library/en-us/oledb/htm/oledbconnectionstringsyntax.asp">
    * OLE DB spec</a>. Omitted are cases for Window handles, returning multiple
    * values, and special handling of "Provider" keyword.
-   *
-   * @throws Throwable
    */
   @Test public void testOleDbExamples() throws Throwable {
     // test the parser with examples from OLE DB documentation

http://git-wip-us.apache.org/repos/asf/incubator-optiq/blob/55fbf047/core/src/main/java/org/eigenbase/rel/metadata/RelMdPredicates.java
----------------------------------------------------------------------
diff --git a/core/src/main/java/org/eigenbase/rel/metadata/RelMdPredicates.java b/core/src/main/java/org/eigenbase/rel/metadata/RelMdPredicates.java
index 44490de..eddaeff 100644
--- a/core/src/main/java/org/eigenbase/rel/metadata/RelMdPredicates.java
+++ b/core/src/main/java/org/eigenbase/rel/metadata/RelMdPredicates.java
@@ -79,14 +79,17 @@ import com.google.common.collect.Lists;
  * Grouping Keys. This can be extended to infer predicates on Aggregation
  * expressions from  expressions on the aggregated columns. For e.g.
  * <pre>
- * select a, max(b) from R1 where b > 7 => max(b) > 7 or max(b) is null
+ * select a, max(b) from R1 where b &gt; 7
+ *   &rarr; max(b) &gt; 7 or max(b) is null
  * </pre>
  *
  * <li> For Projections we only look at columns that are projected without
  * any function applied. So:
  * <pre>
- * select a from R1 where a > 7 -> a > 7 is pulledUp from the Projection.
- * select a + 1 from R1 where a + 1 > 7 -> a + 1 > 7 is not pulledUp
+ * select a from R1 where a &gt; 7
+ *   &rarr; "a &gt; 7" is pulled up from the Projection.
+ * select a + 1 from R1 where a + 1 &gt; 7
+ *   &rarr; "a + 1 gt; 7" is not pulled up
  * </pre>
  *
  * <li> There are several restrictions on Joins:
@@ -135,9 +138,9 @@ public class RelMdPredicates {
    * is not in the projection list.
    *
    * <pre>
-   * childPullUpExprs:      {a > 7, b + c < 10, a + e = 9}
+   * childPullUpExprs:      {a &gt; 7, b + c &lt; 10, a + e = 9}
    * projectionExprs:       {a, b, c, e / 2}
-   * projectionPullupExprs: {a > 7, b + c < 10}
+   * projectionPullupExprs: {a &gt; 7, b + c &lt; 10}
    * </pre>
    *
    * </ol>
@@ -213,9 +216,9 @@ public class RelMdPredicates {
    * GroupSet. For e.g.
    *
    * <pre>
-   * childPullUpExprs : { a > 7, b + c < 10, a + e = 9}
+   * childPullUpExprs : { a &gt; 7, b + c &lt; 10, a + e = 9}
    * groupSet         : { a, b}
-   * pulledUpExprs    : { a > 7}
+   * pulledUpExprs    : { a &gt; 7}
    * </pre>
    */
   public RelOptPredicateList getPredicates(AggregateRelBase agg) {

http://git-wip-us.apache.org/repos/asf/incubator-optiq/blob/55fbf047/core/src/test/java/org/eigenbase/test/DiffRepository.java
----------------------------------------------------------------------
diff --git a/core/src/test/java/org/eigenbase/test/DiffRepository.java b/core/src/test/java/org/eigenbase/test/DiffRepository.java
index 358482c..97f38e2 100644
--- a/core/src/test/java/org/eigenbase/test/DiffRepository.java
+++ b/core/src/test/java/org/eigenbase/test/DiffRepository.java
@@ -686,7 +686,8 @@ public class DiffRepository {
      * @param testCaseName   Test case name
      * @param tag            Tag being expanded
      * @param text           Text being expanded
-     * @param expanded       Expanded text @return Expanded text after filtering
+     * @param expanded       Expanded text
+     * @return Expanded text after filtering
      */
     String filter(
         DiffRepository diffRepository,

http://git-wip-us.apache.org/repos/asf/incubator-optiq/blob/55fbf047/core/src/test/java/org/eigenbase/test/SqlToRelTestBase.java
----------------------------------------------------------------------
diff --git a/core/src/test/java/org/eigenbase/test/SqlToRelTestBase.java b/core/src/test/java/org/eigenbase/test/SqlToRelTestBase.java
index 5c761dc..64d3b6f 100644
--- a/core/src/test/java/org/eigenbase/test/SqlToRelTestBase.java
+++ b/core/src/test/java/org/eigenbase/test/SqlToRelTestBase.java
@@ -147,6 +147,7 @@ public abstract class SqlToRelTestBase {
      *
      * @param sql  SQL query
      * @param plan Expected plan
+     * @param trim Whether to trim columns that are not needed
      */
     void assertConvertsTo(
         String sql,

http://git-wip-us.apache.org/repos/asf/incubator-optiq/blob/55fbf047/core/src/test/java/org/eigenbase/test/concurrent/package-info.java
----------------------------------------------------------------------
diff --git a/core/src/test/java/org/eigenbase/test/concurrent/package-info.java b/core/src/test/java/org/eigenbase/test/concurrent/package-info.java
index df06aad..ca78773 100644
--- a/core/src/test/java/org/eigenbase/test/concurrent/package-info.java
+++ b/core/src/test/java/org/eigenbase/test/concurrent/package-info.java
@@ -40,7 +40,7 @@
  * <p>
  *     The syntactic structure of an <i>mtsql</i> script is:
  *
- * <p><pre>
+ * <pre>
  *     &lt;directive&gt;*
  *     &lt;setup section&gt;?
  *     &lt;cleanup section&gt;?
@@ -74,8 +74,6 @@
  *     Next all the thread sections are run at once, each in its own thread.
  *     When all these threads complete, the cleanup section (if any) is run.
  *
- * <p>
- *
  * <h4>Synchronization:</h4>
  *
  * <p>The threads are synchronized by inserting synchronization points
@@ -112,11 +110,11 @@
  *
  * <h4>Basic Commands (allowed in any section):</h4>
  *
- * <p><pre>
+ * <pre>
  * &lt;SQL statement&gt;:
  *     An SQL statement terminated by a semicolon. The statement can span lines.
  * </pre>
- * <p><pre>
+ * <pre>
  * &#64;include FILE
  *   Reads and executes the contents of FILE, another mtsql script.
  *   Inclusions may nest.
@@ -124,7 +122,7 @@
  *
  * <h4>Threaded Commands (allowed only in a &#64;thread section):</h4>
  *
- * <p><pre>
+ * <pre>
  * &#64;sleep N        -- thread sleeps for N milliseconds
  * &#64;echo MESSAGE   -- prints the message to stdout
  *
@@ -176,7 +174,6 @@
  *     useful to
  *     parameterize SQL statements, timeout values etc.
  *
- * <p>
  * <ul>
  *     <li>Variable Expansion: If VAR is a declared variable, $VAR is replaced
  *         by the value of VAR. Quotes are ignored.  $$ expands to $. A variable

http://git-wip-us.apache.org/repos/asf/incubator-optiq/blob/55fbf047/mongodb/src/test/java/net/hydromatic/optiq/test/package-info.java
----------------------------------------------------------------------
diff --git a/mongodb/src/test/java/net/hydromatic/optiq/test/package-info.java b/mongodb/src/test/java/net/hydromatic/optiq/test/package-info.java
deleted file mode 100644
index 90df564..0000000
--- a/mongodb/src/test/java/net/hydromatic/optiq/test/package-info.java
+++ /dev/null
@@ -1,23 +0,0 @@
-/*
-// Licensed to the Apache Software Foundation (ASF) under one or more
-// contributor license agreements.  See the NOTICE file distributed with
-// this work for additional information regarding copyright ownership.
-// The ASF licenses this file to you 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
-//
-// http://www.apache.org/licenses/LICENSE-2.0
-//
-// 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.
-*/
-
-/**
- * Provides utility classes.
- */
-package org.eigenbase.util;
-
-// End package-info.java

http://git-wip-us.apache.org/repos/asf/incubator-optiq/blob/55fbf047/pom.xml
----------------------------------------------------------------------
diff --git a/pom.xml b/pom.xml
index 1416efb..0268d46 100644
--- a/pom.xml
+++ b/pom.xml
@@ -239,7 +239,6 @@ limitations under the License.
       <plugin>
         <groupId>org.apache.maven.plugins</groupId>
         <artifactId>maven-source-plugin</artifactId>
-        <version>2.2.1</version>
         <executions>
           <execution>
             <id>attach-sources</id>
@@ -253,7 +252,6 @@ limitations under the License.
       </plugin>
       <plugin>
         <artifactId>maven-compiler-plugin</artifactId>
-        <version>2.3.2</version>
         <configuration>
           <source>1.5</source>
           <target>1.5</target>
@@ -283,7 +281,6 @@ limitations under the License.
       </plugin>
       <plugin>
         <artifactId>maven-release-plugin</artifactId>
-        <version>2.4.2</version>
         <dependencies>
           <dependency>
             <groupId>org.apache.maven.scm</groupId>
@@ -309,33 +306,8 @@ limitations under the License.
         </plugin>
         <plugin>
           <groupId>org.apache.maven.plugins</groupId>
-          <artifactId>maven-compiler-plugin</artifactId>
-          <version>2.3.2</version>
-        </plugin>
-        <plugin>
-          <groupId>org.apache.maven.plugins</groupId>
-          <artifactId>maven-jar-plugin</artifactId>
-          <version>2.2</version>
-        </plugin>
-        <plugin>
-          <groupId>org.apache.maven.plugins</groupId>
-          <artifactId>maven-release-plugin</artifactId>
-          <version>2.4.1</version>
-        </plugin>
-        <plugin>
-          <groupId>org.apache.maven.plugins</groupId>
           <artifactId>maven-shade-plugin</artifactId>
-          <version>2.0</version>
-        </plugin>
-        <plugin>
-          <groupId>org.apache.maven.plugins</groupId>
-          <artifactId>maven-source-plugin</artifactId>
-          <version>2.2.1</version>
-        </plugin>
-        <plugin>
-          <groupId>org.apache.maven.plugins</groupId>
-          <artifactId>maven-surefire-plugin</artifactId>
-          <version>2.16</version>
+          <version>2.1</version>
         </plugin>
         <plugin>
           <groupId>org.codehaus.mojo</groupId>
@@ -362,7 +334,6 @@ limitations under the License.
       <plugin>
         <groupId>org.apache.maven.plugins</groupId>
         <artifactId>maven-javadoc-plugin</artifactId>
-        <version>2.8.1</version>
         <configuration>
           <links>
             <link>http://docs.oracle.com/javase/8/docs/api/</link>
@@ -471,7 +442,6 @@ limitations under the License.
           <plugin>
             <groupId>org.apache.rat</groupId>
             <artifactId>apache-rat-plugin</artifactId>
-            <version>0.10</version>
             <configuration>
               <excludes>
                 <exclude>*.md</exclude>

http://git-wip-us.apache.org/repos/asf/incubator-optiq/blob/55fbf047/spark/src/test/java/net/hydromatic/optiq/test/package-info.java
----------------------------------------------------------------------
diff --git a/spark/src/test/java/net/hydromatic/optiq/test/package-info.java b/spark/src/test/java/net/hydromatic/optiq/test/package-info.java
deleted file mode 100644
index efa92d2..0000000
--- a/spark/src/test/java/net/hydromatic/optiq/test/package-info.java
+++ /dev/null
@@ -1,23 +0,0 @@
-/*
-// Licensed to the Apache Software Foundation (ASF) under one or more
-// contributor license agreements.  See the NOTICE file distributed with
-// this work for additional information regarding copyright ownership.
-// The ASF licenses this file to you 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
-//
-// http://www.apache.org/licenses/LICENSE-2.0
-//
-// 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.
-*/
-
-/**
- * Unit tests for Optiq.
- */
-package net.hydromatic.optiq.test;
-
-// End package-info.java

http://git-wip-us.apache.org/repos/asf/incubator-optiq/blob/55fbf047/splunk/src/main/java/net/hydromatic/optiq/impl/splunk/search/package-info.java
----------------------------------------------------------------------
diff --git a/splunk/src/main/java/net/hydromatic/optiq/impl/splunk/search/package-info.java b/splunk/src/main/java/net/hydromatic/optiq/impl/splunk/search/package-info.java
index 90df564..cb2b178 100644
--- a/splunk/src/main/java/net/hydromatic/optiq/impl/splunk/search/package-info.java
+++ b/splunk/src/main/java/net/hydromatic/optiq/impl/splunk/search/package-info.java
@@ -16,8 +16,8 @@
 */
 
 /**
- * Provides utility classes.
+ * Executes queries via Splunk's REST API.
  */
-package org.eigenbase.util;
+package net.hydromatic.optiq.impl.splunk.search;
 
 // End package-info.java

http://git-wip-us.apache.org/repos/asf/incubator-optiq/blob/55fbf047/splunk/src/main/java/net/hydromatic/optiq/impl/splunk/util/package-info.java
----------------------------------------------------------------------
diff --git a/splunk/src/main/java/net/hydromatic/optiq/impl/splunk/util/package-info.java b/splunk/src/main/java/net/hydromatic/optiq/impl/splunk/util/package-info.java
index 90df564..41e9ed4 100644
--- a/splunk/src/main/java/net/hydromatic/optiq/impl/splunk/util/package-info.java
+++ b/splunk/src/main/java/net/hydromatic/optiq/impl/splunk/util/package-info.java
@@ -16,8 +16,8 @@
 */
 
 /**
- * Provides utility classes.
+ * Utilities for RPC to Splunk.
  */
-package org.eigenbase.util;
+package net.hydromatic.optiq.impl.splunk.util;
 
 // End package-info.java

http://git-wip-us.apache.org/repos/asf/incubator-optiq/blob/55fbf047/splunk/src/test/java/net/hydromatic/optiq/test/package-info.java
----------------------------------------------------------------------
diff --git a/splunk/src/test/java/net/hydromatic/optiq/test/package-info.java b/splunk/src/test/java/net/hydromatic/optiq/test/package-info.java
deleted file mode 100644
index 90df564..0000000
--- a/splunk/src/test/java/net/hydromatic/optiq/test/package-info.java
+++ /dev/null
@@ -1,23 +0,0 @@
-/*
-// Licensed to the Apache Software Foundation (ASF) under one or more
-// contributor license agreements.  See the NOTICE file distributed with
-// this work for additional information regarding copyright ownership.
-// The ASF licenses this file to you 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
-//
-// http://www.apache.org/licenses/LICENSE-2.0
-//
-// 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.
-*/
-
-/**
- * Provides utility classes.
- */
-package org.eigenbase.util;
-
-// End package-info.java

http://git-wip-us.apache.org/repos/asf/incubator-optiq/blob/55fbf047/src/main/config/checkstyle/suppressions.xml
----------------------------------------------------------------------
diff --git a/src/main/config/checkstyle/suppressions.xml b/src/main/config/checkstyle/suppressions.xml
index 18cfabe..c33229c 100644
--- a/src/main/config/checkstyle/suppressions.xml
+++ b/src/main/config/checkstyle/suppressions.xml
@@ -43,6 +43,9 @@ limitations under the License.
   <!-- Suppress JavadocPackage in the test packages -->
   <suppress checks="JavadocPackage" files="src[/\\]test[/\\]java[/\\]"/>
 
+  <!-- And likewise in ubenchmark -->
+  <suppress checks="JavadocPackage" files="StatementTest.java"/>
+
   <!-- Method names in Resource can have underscores -->
   <suppress checks="MethodName" files="EigenbaseNewResource.java"/>
 

http://git-wip-us.apache.org/repos/asf/incubator-optiq/blob/55fbf047/ubenchmark/src/main/java/net/hydromatic/optiq/StatementTest.java
----------------------------------------------------------------------
diff --git a/ubenchmark/src/main/java/net/hydromatic/optiq/StatementTest.java b/ubenchmark/src/main/java/net/hydromatic/optiq/StatementTest.java
index 67a9d60..29080dc 100644
--- a/ubenchmark/src/main/java/net/hydromatic/optiq/StatementTest.java
+++ b/ubenchmark/src/main/java/net/hydromatic/optiq/StatementTest.java
@@ -26,6 +26,22 @@ import java.util.*;
 
 /**
  * Compares {@link java.sql.Statement} vs {@link java.sql.PreparedStatement}.
+ *
+ * <p>This package contains micro-benchmarks to test optiq performance.
+ *
+ * <p>To run this and other benchmarks:
+ *
+ * <blockquote>
+ *   <code>mvn package &amp;&amp;
+ *   java -jar ./target/ubenchmarks.jar -wi 5 -i 5 -f 1</code>
+ * </blockquote>
+ *
+ * <p>To run with profiling:
+ *
+ * <blockquote>
+ *   <code>java -Djmh.stack.lines=10 -jar ./target/ubenchmarks.jar
+ *     -prof hs_comp,hs_gc,stack -f 1 -wi 5</code>
+ * </blockquote>
  */
 public class StatementTest {
 

http://git-wip-us.apache.org/repos/asf/incubator-optiq/blob/55fbf047/ubenchmark/src/main/java/net/hydromatic/optiq/package-info.java
----------------------------------------------------------------------
diff --git a/ubenchmark/src/main/java/net/hydromatic/optiq/package-info.java b/ubenchmark/src/main/java/net/hydromatic/optiq/package-info.java
deleted file mode 100644
index a24acca..0000000
--- a/ubenchmark/src/main/java/net/hydromatic/optiq/package-info.java
+++ /dev/null
@@ -1,25 +0,0 @@
-/*
-// Licensed to the Apache Software Foundation (ASF) under one or more
-// contributor license agreements.  See the NOTICE file distributed with
-// this work for additional information regarding copyright ownership.
-// The ASF licenses this file to you 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
-//
-// http://www.apache.org/licenses/LICENSE-2.0
-//
-// 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.
-*/
-/**
- * Microbenchmarks to test optiq performance.
- * The way to run is
- * {@code mvn package && java -jar ./target/ubenchmarks.jar -wi 5 -i 5 -f 1}.
- * <p>
- * To run with profiling, use {@code java -Djmh.stack.lines=10 -jar
- * ./target/ubenchmarks.jar -prof hs_comp,hs_gc,stack -f 1 -wi 5}.
- */
-package net.hydromatic.optiq;