You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@kudu.apache.org by gr...@apache.org on 2019/06/14 20:56:27 UTC
[kudu] branch branch-1.10.x updated: Optimize and sort imports
This is an automated email from the ASF dual-hosted git repository.
granthenke pushed a commit to branch branch-1.10.x
in repository https://gitbox.apache.org/repos/asf/kudu.git
The following commit(s) were added to refs/heads/branch-1.10.x by this push:
new c6b0180 Optimize and sort imports
c6b0180 is described below
commit c6b018094c41373297060b2df4be76f54598d52f
Author: Will Berkeley <wd...@gmail.com>
AuthorDate: Thu Jun 13 14:03:29 2019 -0700
Optimize and sort imports
I ran IntelliJ's "Optimize imports" function on the kudu-client module
after noticing how many violations of our import sorting rules were
reported by checkstyle.
Change-Id: Ida7cbcee5b40888ee2f78ded06206a2c51c5eed7
Reviewed-on: http://gerrit.cloudera.org:8080/13642
Reviewed-by: Grant Henke <gr...@apache.org>
Tested-by: Will Berkeley <wd...@gmail.com>
(cherry picked from commit ff6cefa1d1099c4e1784673628c2ae53a3612ff8)
Reviewed-on: http://gerrit.cloudera.org:8080/13654
Reviewed-by: Will Berkeley <wd...@gmail.com>
Tested-by: Kudu Jenkins
---
.../src/main/java/org/apache/kudu/Schema.java | 2 +-
.../org/apache/kudu/client/AlterTableOptions.java | 6 +--
.../org/apache/kudu/client/AsyncKuduClient.java | 8 ++--
.../org/apache/kudu/client/AsyncKuduScanner.java | 8 ++--
.../org/apache/kudu/client/AsyncKuduSession.java | 2 +-
.../org/apache/kudu/client/AuthzTokenCache.java | 21 +++++-----
.../main/java/org/apache/kudu/client/Batch.java | 4 +-
.../org/apache/kudu/client/ConnectToCluster.java | 2 +-
.../org/apache/kudu/client/ConnectionCache.java | 1 -
.../apache/kudu/client/GetTableSchemaRequest.java | 8 ++--
.../apache/kudu/client/GetTableSchemaResponse.java | 6 +--
.../apache/kudu/client/HiveMetastoreConfig.java | 4 +-
.../java/org/apache/kudu/client/HostAndPort.java | 4 +-
.../java/org/apache/kudu/client/KuduClient.java | 1 -
.../org/apache/kudu/client/KuduPartitioner.java | 8 ++--
.../java/org/apache/kudu/client/KuduPredicate.java | 5 +--
.../main/java/org/apache/kudu/client/KuduRpc.java | 2 +-
.../apache/kudu/client/KuduScannerIterator.java | 4 +-
.../org/apache/kudu/client/ListTablesRequest.java | 2 +-
.../java/org/apache/kudu/client/Operation.java | 4 +-
.../java/org/apache/kudu/client/PartialRow.java | 2 +-
.../java/org/apache/kudu/client/RowResult.java | 2 +-
.../main/java/org/apache/kudu/client/RpcProxy.java | 1 -
.../org/apache/kudu/client/SecurityContext.java | 7 ++--
.../apache/kudu/client/SplitKeyRangeResponse.java | 3 +-
.../java/org/apache/kudu/util/BloomFilter.java | 2 +-
.../java/org/apache/kudu/util/DataGenerator.java | 17 ++++----
.../java/org/apache/kudu/util/DecimalUtil.java | 6 +--
.../main/java/org/apache/kudu/util/NetUtil.java | 3 +-
.../java/org/apache/kudu/util/SchemaGenerator.java | 23 +++++-----
.../java/org/apache/kudu/util/SecurityUtil.java | 2 -
.../java/org/apache/kudu/util/TimestampUtil.java | 4 +-
.../java/org/apache/kudu/TestColumnSchema.java | 3 +-
.../test/java/org/apache/kudu/client/ITClient.java | 14 +++----
.../org/apache/kudu/client/ITClientStress.java | 14 +++----
.../apache/kudu/client/ITScannerMultiTablet.java | 4 +-
.../org/apache/kudu/client/TestAlterTable.java | 4 +-
.../apache/kudu/client/TestAsyncKuduClient.java | 8 ++--
.../apache/kudu/client/TestAsyncKuduSession.java | 4 +-
.../apache/kudu/client/TestAuthTokenReacquire.java | 23 +++++-----
.../kudu/client/TestAuthnTokenReacquireOpen.java | 9 ++--
.../apache/kudu/client/TestAuthzTokenCache.java | 31 +++++++-------
.../org/apache/kudu/client/TestBloomFilter.java | 2 +-
.../java/org/apache/kudu/client/TestBytes.java | 4 +-
.../kudu/client/TestClientFailoverSupport.java | 15 +++----
.../kudu/client/TestColumnRangePredicate.java | 4 +-
.../apache/kudu/client/TestConnectionCache.java | 4 +-
.../kudu/client/TestFlexiblePartitioning.java | 2 +-
.../org/apache/kudu/client/TestHandleTooBusy.java | 11 ++---
.../org/apache/kudu/client/TestHybridTime.java | 4 +-
.../org/apache/kudu/client/TestKeyEncoding.java | 4 +-
.../org/apache/kudu/client/TestKuduClient.java | 19 ++++-----
.../apache/kudu/client/TestKuduPartitioner.java | 24 +++++------
.../org/apache/kudu/client/TestKuduPredicate.java | 2 +-
.../org/apache/kudu/client/TestKuduScanner.java | 49 +++++++++++-----------
.../org/apache/kudu/client/TestKuduSession.java | 7 ++--
.../java/org/apache/kudu/client/TestKuduTable.java | 10 ++---
.../org/apache/kudu/client/TestLeaderFailover.java | 3 +-
.../org/apache/kudu/client/TestMasterFailover.java | 3 +-
.../kudu/client/TestMultiMasterAuthzTokens.java | 23 ++++++----
.../kudu/client/TestMultipleLeaderFailover.java | 9 ++--
.../org/apache/kudu/client/TestNegotiation.java | 15 +++----
.../apache/kudu/client/TestPartitionPruner.java | 2 +-
.../org/apache/kudu/client/TestRemoteTablet.java | 1 -
.../java/org/apache/kudu/client/TestRowErrors.java | 7 ++--
.../java/org/apache/kudu/client/TestRowResult.java | 4 +-
.../java/org/apache/kudu/client/TestRpcTraces.java | 4 +-
.../org/apache/kudu/client/TestScanPredicate.java | 2 +-
.../java/org/apache/kudu/client/TestScanToken.java | 32 +++++++-------
.../apache/kudu/client/TestScannerMultiTablet.java | 7 ++--
.../java/org/apache/kudu/client/TestSecurity.java | 25 ++++++-----
.../kudu/client/TestSecurityContextRealUser.java | 17 ++++----
.../org/apache/kudu/client/TestSplitKeyRange.java | 9 ++--
.../org/apache/kudu/client/TestStatistics.java | 8 ++--
.../kudu/client/TestTableLocationsCache.java | 2 +-
.../java/org/apache/kudu/client/TestTimeouts.java | 7 ++--
76 files changed, 318 insertions(+), 311 deletions(-)
diff --git a/java/kudu-client/src/main/java/org/apache/kudu/Schema.java b/java/kudu-client/src/main/java/org/apache/kudu/Schema.java
index 630a68d..3251550 100644
--- a/java/kudu-client/src/main/java/org/apache/kudu/Schema.java
+++ b/java/kudu-client/src/main/java/org/apache/kudu/Schema.java
@@ -24,10 +24,10 @@ import java.util.Map;
import com.google.common.base.Preconditions;
import com.google.common.collect.ImmutableList;
-import org.apache.kudu.Common.DataType;
import org.apache.yetus.audience.InterfaceAudience;
import org.apache.yetus.audience.InterfaceStability;
+import org.apache.kudu.Common.DataType;
import org.apache.kudu.client.Bytes;
import org.apache.kudu.client.PartialRow;
diff --git a/java/kudu-client/src/main/java/org/apache/kudu/client/AlterTableOptions.java b/java/kudu-client/src/main/java/org/apache/kudu/client/AlterTableOptions.java
index cb92c29..b4fbbab 100644
--- a/java/kudu-client/src/main/java/org/apache/kudu/client/AlterTableOptions.java
+++ b/java/kudu-client/src/main/java/org/apache/kudu/client/AlterTableOptions.java
@@ -17,13 +17,13 @@
package org.apache.kudu.client;
-import java.util.EnumSet;
-import java.util.Map;
-
import static org.apache.kudu.ColumnSchema.CompressionAlgorithm;
import static org.apache.kudu.ColumnSchema.Encoding;
import static org.apache.kudu.master.Master.AlterTableRequestPB;
+import java.util.EnumSet;
+import java.util.Map;
+
import com.google.common.base.Preconditions;
import com.google.protobuf.ByteString;
import org.apache.yetus.audience.InterfaceAudience;
diff --git a/java/kudu-client/src/main/java/org/apache/kudu/client/AsyncKuduClient.java b/java/kudu-client/src/main/java/org/apache/kudu/client/AsyncKuduClient.java
index e00888f..1de7306 100644
--- a/java/kudu-client/src/main/java/org/apache/kudu/client/AsyncKuduClient.java
+++ b/java/kudu-client/src/main/java/org/apache/kudu/client/AsyncKuduClient.java
@@ -27,7 +27,6 @@
package org.apache.kudu.client;
import static java.util.concurrent.TimeUnit.MILLISECONDS;
-
import static org.apache.kudu.client.ExternalConsistencyMode.CLIENT_PROPAGATED;
import java.net.InetAddress;
@@ -35,7 +34,6 @@ import java.net.UnknownHostException;
import java.security.cert.CertificateException;
import java.util.ArrayList;
import java.util.Arrays;
-import java.util.function.Consumer;
import java.util.HashSet;
import java.util.List;
import java.util.Random;
@@ -45,6 +43,7 @@ import java.util.concurrent.ConcurrentHashMap;
import java.util.concurrent.Executor;
import java.util.concurrent.Executors;
import java.util.concurrent.Semaphore;
+import java.util.function.Consumer;
import javax.annotation.Nonnull;
import javax.annotation.Nullable;
import javax.annotation.concurrent.GuardedBy;
@@ -57,7 +56,6 @@ import com.google.protobuf.ByteString;
import com.google.protobuf.Message;
import com.stumbleupon.async.Callback;
import com.stumbleupon.async.Deferred;
-import org.apache.kudu.security.Token;
import org.apache.yetus.audience.InterfaceAudience;
import org.apache.yetus.audience.InterfaceStability;
import org.jboss.netty.channel.socket.ClientSocketChannelFactory;
@@ -70,13 +68,13 @@ import org.jboss.netty.util.TimerTask;
import org.slf4j.Logger;
import org.slf4j.LoggerFactory;
-import org.apache.kudu.security.Token.SignedTokenPB;
import org.apache.kudu.Common;
import org.apache.kudu.Schema;
import org.apache.kudu.master.Master;
import org.apache.kudu.master.Master.GetTableLocationsResponsePB;
-import org.apache.kudu.master.Master.TableIdentifierPB;
import org.apache.kudu.master.Master.TSInfoPB;
+import org.apache.kudu.master.Master.TableIdentifierPB;
+import org.apache.kudu.security.Token.SignedTokenPB;
import org.apache.kudu.util.AsyncUtil;
import org.apache.kudu.util.NetUtil;
import org.apache.kudu.util.Pair;
diff --git a/java/kudu-client/src/main/java/org/apache/kudu/client/AsyncKuduScanner.java b/java/kudu-client/src/main/java/org/apache/kudu/client/AsyncKuduScanner.java
index f327faf..84368bd 100644
--- a/java/kudu-client/src/main/java/org/apache/kudu/client/AsyncKuduScanner.java
+++ b/java/kudu-client/src/main/java/org/apache/kudu/client/AsyncKuduScanner.java
@@ -43,10 +43,6 @@ import com.google.protobuf.Message;
import com.google.protobuf.UnsafeByteOperations;
import com.stumbleupon.async.Callback;
import com.stumbleupon.async.Deferred;
-import org.apache.kudu.security.Token;
-import org.apache.kudu.Type;
-import org.apache.kudu.tserver.Tserver.ScannerKeepAliveRequestPB;
-import org.apache.kudu.tserver.Tserver.ScannerKeepAliveResponsePB;
import org.apache.yetus.audience.InterfaceAudience;
import org.apache.yetus.audience.InterfaceStability;
import org.slf4j.Logger;
@@ -55,7 +51,11 @@ import org.slf4j.LoggerFactory;
import org.apache.kudu.ColumnSchema;
import org.apache.kudu.Common;
import org.apache.kudu.Schema;
+import org.apache.kudu.Type;
+import org.apache.kudu.security.Token;
import org.apache.kudu.tserver.Tserver;
+import org.apache.kudu.tserver.Tserver.ScannerKeepAliveRequestPB;
+import org.apache.kudu.tserver.Tserver.ScannerKeepAliveResponsePB;
import org.apache.kudu.util.Pair;
/**
diff --git a/java/kudu-client/src/main/java/org/apache/kudu/client/AsyncKuduSession.java b/java/kudu-client/src/main/java/org/apache/kudu/client/AsyncKuduSession.java
index ba52727..b4b2c37 100644
--- a/java/kudu-client/src/main/java/org/apache/kudu/client/AsyncKuduSession.java
+++ b/java/kudu-client/src/main/java/org/apache/kudu/client/AsyncKuduSession.java
@@ -37,7 +37,6 @@ import com.google.common.base.Preconditions;
import com.google.common.collect.ImmutableList;
import com.stumbleupon.async.Callback;
import com.stumbleupon.async.Deferred;
-import org.apache.kudu.client.AsyncKuduClient.LookupType;
import org.apache.yetus.audience.InterfaceAudience;
import org.apache.yetus.audience.InterfaceStability;
import org.jboss.netty.util.Timeout;
@@ -45,6 +44,7 @@ import org.jboss.netty.util.TimerTask;
import org.slf4j.Logger;
import org.slf4j.LoggerFactory;
+import org.apache.kudu.client.AsyncKuduClient.LookupType;
import org.apache.kudu.util.AsyncUtil;
import org.apache.kudu.util.Slice;
diff --git a/java/kudu-client/src/main/java/org/apache/kudu/client/AuthzTokenCache.java b/java/kudu-client/src/main/java/org/apache/kudu/client/AuthzTokenCache.java
index fdb2358..5e72b11 100644
--- a/java/kudu-client/src/main/java/org/apache/kudu/client/AuthzTokenCache.java
+++ b/java/kudu-client/src/main/java/org/apache/kudu/client/AuthzTokenCache.java
@@ -19,16 +19,6 @@
package org.apache.kudu.client;
-import com.google.common.base.Preconditions;
-import com.stumbleupon.async.Callback;
-import org.apache.kudu.security.Token;
-import org.apache.yetus.audience.InterfaceAudience;
-import org.slf4j.Logger;
-import org.slf4j.LoggerFactory;
-
-import javax.annotation.Nonnull;
-import javax.annotation.concurrent.GuardedBy;
-import javax.annotation.concurrent.ThreadSafe;
import java.util.ArrayList;
import java.util.Arrays;
import java.util.HashMap;
@@ -36,6 +26,17 @@ import java.util.List;
import java.util.Map;
import java.util.concurrent.ConcurrentHashMap;
import java.util.concurrent.atomic.AtomicInteger;
+import javax.annotation.Nonnull;
+import javax.annotation.concurrent.GuardedBy;
+import javax.annotation.concurrent.ThreadSafe;
+
+import com.google.common.base.Preconditions;
+import com.stumbleupon.async.Callback;
+import org.apache.yetus.audience.InterfaceAudience;
+import org.slf4j.Logger;
+import org.slf4j.LoggerFactory;
+
+import org.apache.kudu.security.Token;
/**
* Cache for authz tokens received from the master of unbounded capacity. A
diff --git a/java/kudu-client/src/main/java/org/apache/kudu/client/Batch.java b/java/kudu-client/src/main/java/org/apache/kudu/client/Batch.java
index a6f37f4..c2481e5 100644
--- a/java/kudu-client/src/main/java/org/apache/kudu/client/Batch.java
+++ b/java/kudu-client/src/main/java/org/apache/kudu/client/Batch.java
@@ -27,13 +27,13 @@ import com.google.common.base.MoreObjects;
import com.google.common.collect.Iterables;
import com.google.protobuf.Message;
import com.google.protobuf.UnsafeByteOperations;
-import org.apache.kudu.WireProtocol.AppStatusPB.ErrorCode;
-import org.apache.kudu.security.Token;
import org.apache.yetus.audience.InterfaceAudience;
import org.jboss.netty.util.Timer;
+import org.apache.kudu.WireProtocol.AppStatusPB.ErrorCode;
import org.apache.kudu.client.Statistics.Statistic;
import org.apache.kudu.client.Statistics.TabletStatistics;
+import org.apache.kudu.security.Token;
import org.apache.kudu.tserver.Tserver;
import org.apache.kudu.tserver.Tserver.TabletServerErrorPB;
import org.apache.kudu.util.Pair;
diff --git a/java/kudu-client/src/main/java/org/apache/kudu/client/ConnectToCluster.java b/java/kudu-client/src/main/java/org/apache/kudu/client/ConnectToCluster.java
index 0557b56..42a536e 100644
--- a/java/kudu-client/src/main/java/org/apache/kudu/client/ConnectToCluster.java
+++ b/java/kudu-client/src/main/java/org/apache/kudu/client/ConnectToCluster.java
@@ -31,9 +31,9 @@ import com.google.common.collect.Lists;
import com.stumbleupon.async.Callback;
import com.stumbleupon.async.Deferred;
import org.apache.yetus.audience.InterfaceAudience;
+import org.jboss.netty.util.Timer;
import org.slf4j.Logger;
import org.slf4j.LoggerFactory;
-import org.jboss.netty.util.Timer;
import org.apache.kudu.Common.HostPortPB;
import org.apache.kudu.consensus.Metadata.RaftPeerPB.Role;
diff --git a/java/kudu-client/src/main/java/org/apache/kudu/client/ConnectionCache.java b/java/kudu-client/src/main/java/org/apache/kudu/client/ConnectionCache.java
index 85be2cd..6da15b5 100644
--- a/java/kudu-client/src/main/java/org/apache/kudu/client/ConnectionCache.java
+++ b/java/kudu-client/src/main/java/org/apache/kudu/client/ConnectionCache.java
@@ -29,7 +29,6 @@ import com.google.common.collect.ImmutableList;
import com.stumbleupon.async.Deferred;
import org.apache.yetus.audience.InterfaceAudience;
import org.apache.yetus.audience.InterfaceStability;
-
import org.jboss.netty.channel.socket.ClientSocketChannelFactory;
import org.jboss.netty.util.HashedWheelTimer;
diff --git a/java/kudu-client/src/main/java/org/apache/kudu/client/GetTableSchemaRequest.java b/java/kudu-client/src/main/java/org/apache/kudu/client/GetTableSchemaRequest.java
index 0af0cb8..3e4f04c 100644
--- a/java/kudu-client/src/main/java/org/apache/kudu/client/GetTableSchemaRequest.java
+++ b/java/kudu-client/src/main/java/org/apache/kudu/client/GetTableSchemaRequest.java
@@ -21,20 +21,20 @@ import static org.apache.kudu.master.Master.GetTableSchemaRequestPB;
import static org.apache.kudu.master.Master.GetTableSchemaResponsePB;
import static org.apache.kudu.master.Master.TableIdentifierPB;
+import java.util.Collection;
+import java.util.List;
+
import com.google.common.base.Preconditions;
import com.google.common.collect.ImmutableList;
import com.google.protobuf.ByteString;
import com.google.protobuf.Message;
-import org.apache.kudu.master.Master;
import org.apache.yetus.audience.InterfaceAudience;
import org.jboss.netty.util.Timer;
import org.apache.kudu.Schema;
+import org.apache.kudu.master.Master;
import org.apache.kudu.util.Pair;
-import java.util.Collection;
-import java.util.List;
-
/**
* RPC to fetch a table's schema
*/
diff --git a/java/kudu-client/src/main/java/org/apache/kudu/client/GetTableSchemaResponse.java b/java/kudu-client/src/main/java/org/apache/kudu/client/GetTableSchemaResponse.java
index 7e4b557..22a1a2b 100644
--- a/java/kudu-client/src/main/java/org/apache/kudu/client/GetTableSchemaResponse.java
+++ b/java/kudu-client/src/main/java/org/apache/kudu/client/GetTableSchemaResponse.java
@@ -17,12 +17,12 @@
package org.apache.kudu.client;
+import java.util.Map;
+
import org.apache.yetus.audience.InterfaceAudience;
-import org.apache.kudu.security.Token.SignedTokenPB;
import org.apache.kudu.Schema;
-
-import java.util.Map;
+import org.apache.kudu.security.Token.SignedTokenPB;
@InterfaceAudience.Private
public class GetTableSchemaResponse extends KuduRpcResponse {
diff --git a/java/kudu-client/src/main/java/org/apache/kudu/client/HiveMetastoreConfig.java b/java/kudu-client/src/main/java/org/apache/kudu/client/HiveMetastoreConfig.java
index 77ba8b8..946655f 100644
--- a/java/kudu-client/src/main/java/org/apache/kudu/client/HiveMetastoreConfig.java
+++ b/java/kudu-client/src/main/java/org/apache/kudu/client/HiveMetastoreConfig.java
@@ -17,11 +17,11 @@
package org.apache.kudu.client;
+import javax.annotation.CheckForNull;
+
import org.apache.yetus.audience.InterfaceAudience;
import org.apache.yetus.audience.InterfaceStability;
-import javax.annotation.CheckForNull;
-
/**
* The Hive Metastore configuration of a Kudu cluster.
*/
diff --git a/java/kudu-client/src/main/java/org/apache/kudu/client/HostAndPort.java b/java/kudu-client/src/main/java/org/apache/kudu/client/HostAndPort.java
index ccec111..53917d8 100644
--- a/java/kudu-client/src/main/java/org/apache/kudu/client/HostAndPort.java
+++ b/java/kudu-client/src/main/java/org/apache/kudu/client/HostAndPort.java
@@ -16,11 +16,11 @@
// under the License.
package org.apache.kudu.client;
+import java.net.InetSocketAddress;
+
import com.google.common.base.Objects;
import org.apache.yetus.audience.InterfaceAudience;
-import java.net.InetSocketAddress;
-
/**
* A simple wrapper around InetSocketAddress to prevent
* accidentally introducing DNS lookups.
diff --git a/java/kudu-client/src/main/java/org/apache/kudu/client/KuduClient.java b/java/kudu-client/src/main/java/org/apache/kudu/client/KuduClient.java
index 24a6abb..acb2b69 100644
--- a/java/kudu-client/src/main/java/org/apache/kudu/client/KuduClient.java
+++ b/java/kudu-client/src/main/java/org/apache/kudu/client/KuduClient.java
@@ -22,7 +22,6 @@ import java.util.List;
import java.util.concurrent.Executor;
import com.google.common.base.Preconditions;
-import com.stumbleupon.async.Callback;
import com.stumbleupon.async.Deferred;
import org.apache.yetus.audience.InterfaceAudience;
import org.apache.yetus.audience.InterfaceStability;
diff --git a/java/kudu-client/src/main/java/org/apache/kudu/client/KuduPartitioner.java b/java/kudu-client/src/main/java/org/apache/kudu/client/KuduPartitioner.java
index 1649586..f282701 100644
--- a/java/kudu-client/src/main/java/org/apache/kudu/client/KuduPartitioner.java
+++ b/java/kudu-client/src/main/java/org/apache/kudu/client/KuduPartitioner.java
@@ -17,16 +17,16 @@
package org.apache.kudu.client;
-import com.google.common.base.Preconditions;
-import org.apache.yetus.audience.InterfaceAudience;
-import org.apache.yetus.audience.InterfaceStability;
+import static java.nio.charset.StandardCharsets.UTF_8;
import java.util.LinkedHashMap;
import java.util.Map;
import java.util.NavigableMap;
import java.util.TreeMap;
-import static java.nio.charset.StandardCharsets.UTF_8;
+import com.google.common.base.Preconditions;
+import org.apache.yetus.audience.InterfaceAudience;
+import org.apache.yetus.audience.InterfaceStability;
/**
* A KuduPartitioner allows clients to determine the target partition of a
diff --git a/java/kudu-client/src/main/java/org/apache/kudu/client/KuduPredicate.java b/java/kudu-client/src/main/java/org/apache/kudu/client/KuduPredicate.java
index dc37706..cfba040 100644
--- a/java/kudu-client/src/main/java/org/apache/kudu/client/KuduPredicate.java
+++ b/java/kudu-client/src/main/java/org/apache/kudu/client/KuduPredicate.java
@@ -20,9 +20,8 @@ package org.apache.kudu.client;
import java.io.ByteArrayInputStream;
import java.io.ByteArrayOutputStream;
import java.io.IOException;
-import java.math.BigInteger;
import java.math.BigDecimal;
-import java.nio.ByteBuffer;
+import java.math.BigInteger;
import java.sql.Timestamp;
import java.util.ArrayList;
import java.util.Arrays;
@@ -37,7 +36,6 @@ import com.google.common.base.Objects;
import com.google.common.base.Preconditions;
import com.google.common.primitives.UnsignedBytes;
import com.google.protobuf.ByteString;
-import org.apache.kudu.util.TimestampUtil;
import org.apache.yetus.audience.InterfaceAudience;
import org.apache.yetus.audience.InterfaceStability;
@@ -47,6 +45,7 @@ import org.apache.kudu.Common;
import org.apache.kudu.Schema;
import org.apache.kudu.Type;
import org.apache.kudu.util.DecimalUtil;
+import org.apache.kudu.util.TimestampUtil;
/**
* A predicate which can be used to filter rows based on the value of a column.
diff --git a/java/kudu-client/src/main/java/org/apache/kudu/client/KuduRpc.java b/java/kudu-client/src/main/java/org/apache/kudu/client/KuduRpc.java
index 1bfb725..acf0c44 100644
--- a/java/kudu-client/src/main/java/org/apache/kudu/client/KuduRpc.java
+++ b/java/kudu-client/src/main/java/org/apache/kudu/client/KuduRpc.java
@@ -39,7 +39,6 @@ import com.google.protobuf.CodedOutputStream;
import com.google.protobuf.InvalidProtocolBufferException;
import com.google.protobuf.Message;
import com.stumbleupon.async.Deferred;
-import org.apache.kudu.security.Token;
import org.apache.yetus.audience.InterfaceAudience;
import org.jboss.netty.buffer.ChannelBuffer;
import org.jboss.netty.buffer.ChannelBuffers;
@@ -49,6 +48,7 @@ import org.jboss.netty.util.TimerTask;
import org.slf4j.Logger;
import org.slf4j.LoggerFactory;
+import org.apache.kudu.security.Token;
import org.apache.kudu.util.Pair;
import org.apache.kudu.util.Slice;
diff --git a/java/kudu-client/src/main/java/org/apache/kudu/client/KuduScannerIterator.java b/java/kudu-client/src/main/java/org/apache/kudu/client/KuduScannerIterator.java
index 9c3ce5e..0f588b0 100644
--- a/java/kudu-client/src/main/java/org/apache/kudu/client/KuduScannerIterator.java
+++ b/java/kudu-client/src/main/java/org/apache/kudu/client/KuduScannerIterator.java
@@ -17,11 +17,11 @@
package org.apache.kudu.client;
+import java.util.Iterator;
+
import org.apache.yetus.audience.InterfaceAudience;
import org.apache.yetus.audience.InterfaceStability;
-import java.util.Iterator;
-
/**
* An iterator for the RowResults of a KuduScanner.
* Exhausting this iterator means that all of the rows from a KuduScanner have been read.
diff --git a/java/kudu-client/src/main/java/org/apache/kudu/client/ListTablesRequest.java b/java/kudu-client/src/main/java/org/apache/kudu/client/ListTablesRequest.java
index bc77664..9c7161f 100644
--- a/java/kudu-client/src/main/java/org/apache/kudu/client/ListTablesRequest.java
+++ b/java/kudu-client/src/main/java/org/apache/kudu/client/ListTablesRequest.java
@@ -21,10 +21,10 @@ import java.util.ArrayList;
import java.util.List;
import com.google.protobuf.Message;
-import org.apache.kudu.client.ListTablesResponse.TableInfo;
import org.apache.yetus.audience.InterfaceAudience;
import org.jboss.netty.util.Timer;
+import org.apache.kudu.client.ListTablesResponse.TableInfo;
import org.apache.kudu.master.Master;
import org.apache.kudu.util.Pair;
diff --git a/java/kudu-client/src/main/java/org/apache/kudu/client/Operation.java b/java/kudu-client/src/main/java/org/apache/kudu/client/Operation.java
index 998692b..5ebda0f 100644
--- a/java/kudu-client/src/main/java/org/apache/kudu/client/Operation.java
+++ b/java/kudu-client/src/main/java/org/apache/kudu/client/Operation.java
@@ -29,8 +29,6 @@ import com.google.common.collect.ImmutableList;
import com.google.protobuf.ByteString;
import com.google.protobuf.Message;
import com.google.protobuf.UnsafeByteOperations;
-import org.apache.kudu.WireProtocol.AppStatusPB.ErrorCode;
-import org.apache.kudu.security.Token;
import org.apache.yetus.audience.InterfaceAudience;
import org.apache.yetus.audience.InterfaceStability;
import org.jboss.netty.util.Timer;
@@ -38,10 +36,12 @@ import org.jboss.netty.util.Timer;
import org.apache.kudu.ColumnSchema;
import org.apache.kudu.Schema;
import org.apache.kudu.Type;
+import org.apache.kudu.WireProtocol.AppStatusPB.ErrorCode;
import org.apache.kudu.WireProtocol.RowOperationsPB;
import org.apache.kudu.client.ProtobufHelper.SchemaPBConversionFlags;
import org.apache.kudu.client.Statistics.Statistic;
import org.apache.kudu.client.Statistics.TabletStatistics;
+import org.apache.kudu.security.Token;
import org.apache.kudu.tserver.Tserver;
import org.apache.kudu.util.Pair;
diff --git a/java/kudu-client/src/main/java/org/apache/kudu/client/PartialRow.java b/java/kudu-client/src/main/java/org/apache/kudu/client/PartialRow.java
index d15e0e7..0a5ddd6 100644
--- a/java/kudu-client/src/main/java/org/apache/kudu/client/PartialRow.java
+++ b/java/kudu-client/src/main/java/org/apache/kudu/client/PartialRow.java
@@ -28,7 +28,6 @@ import java.util.ListIterator;
import com.google.common.base.Preconditions;
import com.google.common.collect.Lists;
-import org.apache.kudu.util.TimestampUtil;
import org.apache.yetus.audience.InterfaceAudience;
import org.apache.yetus.audience.InterfaceStability;
import org.jboss.netty.util.CharsetUtil;
@@ -39,6 +38,7 @@ import org.apache.kudu.Schema;
import org.apache.kudu.Type;
import org.apache.kudu.util.DecimalUtil;
import org.apache.kudu.util.StringUtil;
+import org.apache.kudu.util.TimestampUtil;
/**
* Class used to represent parts of a row along with its schema.<p>
diff --git a/java/kudu-client/src/main/java/org/apache/kudu/client/RowResult.java b/java/kudu-client/src/main/java/org/apache/kudu/client/RowResult.java
index 0d8d502..cf57160 100644
--- a/java/kudu-client/src/main/java/org/apache/kudu/client/RowResult.java
+++ b/java/kudu-client/src/main/java/org/apache/kudu/client/RowResult.java
@@ -23,7 +23,6 @@ import java.sql.Timestamp;
import java.util.Arrays;
import java.util.BitSet;
-import org.apache.kudu.util.TimestampUtil;
import org.apache.yetus.audience.InterfaceAudience;
import org.apache.yetus.audience.InterfaceStability;
@@ -32,6 +31,7 @@ import org.apache.kudu.ColumnTypeAttributes;
import org.apache.kudu.Schema;
import org.apache.kudu.Type;
import org.apache.kudu.util.Slice;
+import org.apache.kudu.util.TimestampUtil;
/**
* RowResult represents one row from a scanner.
diff --git a/java/kudu-client/src/main/java/org/apache/kudu/client/RpcProxy.java b/java/kudu-client/src/main/java/org/apache/kudu/client/RpcProxy.java
index 7b2f254..27e4110 100644
--- a/java/kudu-client/src/main/java/org/apache/kudu/client/RpcProxy.java
+++ b/java/kudu-client/src/main/java/org/apache/kudu/client/RpcProxy.java
@@ -32,7 +32,6 @@ import javax.annotation.Nonnull;
import com.google.common.base.Preconditions;
import com.google.protobuf.Message;
import com.stumbleupon.async.Callback;
-import org.apache.kudu.security.Token;
import org.apache.yetus.audience.InterfaceAudience;
import org.apache.yetus.audience.InterfaceStability;
import org.slf4j.Logger;
diff --git a/java/kudu-client/src/main/java/org/apache/kudu/client/SecurityContext.java b/java/kudu-client/src/main/java/org/apache/kudu/client/SecurityContext.java
index 2835015..0f28489 100644
--- a/java/kudu-client/src/main/java/org/apache/kudu/client/SecurityContext.java
+++ b/java/kudu-client/src/main/java/org/apache/kudu/client/SecurityContext.java
@@ -29,7 +29,6 @@ import java.util.Collections;
import java.util.List;
import java.util.concurrent.TimeUnit;
import java.util.concurrent.atomic.AtomicReference;
-
import javax.annotation.Nonnull;
import javax.annotation.Nullable;
import javax.annotation.concurrent.GuardedBy;
@@ -46,15 +45,15 @@ import com.google.common.collect.ImmutableList;
import com.google.common.collect.Lists;
import com.google.protobuf.ByteString;
import com.google.protobuf.InvalidProtocolBufferException;
+import org.apache.yetus.audience.InterfaceAudience;
+import org.slf4j.Logger;
+import org.slf4j.LoggerFactory;
import org.apache.kudu.client.Client.AuthenticationCredentialsPB;
import org.apache.kudu.security.Token.SignedTokenPB;
import org.apache.kudu.security.Token.TokenPB;
import org.apache.kudu.util.Pair;
import org.apache.kudu.util.SecurityUtil;
-import org.apache.yetus.audience.InterfaceAudience;
-import org.slf4j.Logger;
-import org.slf4j.LoggerFactory;
/**
* Class associated with a single AsyncKuduClient which stores security-related
diff --git a/java/kudu-client/src/main/java/org/apache/kudu/client/SplitKeyRangeResponse.java b/java/kudu-client/src/main/java/org/apache/kudu/client/SplitKeyRangeResponse.java
index 4b8c274..2b48218 100644
--- a/java/kudu-client/src/main/java/org/apache/kudu/client/SplitKeyRangeResponse.java
+++ b/java/kudu-client/src/main/java/org/apache/kudu/client/SplitKeyRangeResponse.java
@@ -19,9 +19,10 @@ package org.apache.kudu.client;
import java.util.List;
-import org.apache.kudu.Common.KeyRangePB;
import org.apache.yetus.audience.InterfaceAudience;
+import org.apache.kudu.Common.KeyRangePB;
+
/**
* Response type for SplitKeyRangeRequest.
*/
diff --git a/java/kudu-client/src/main/java/org/apache/kudu/util/BloomFilter.java b/java/kudu-client/src/main/java/org/apache/kudu/util/BloomFilter.java
index 049bc40..16d70c2 100644
--- a/java/kudu-client/src/main/java/org/apache/kudu/util/BloomFilter.java
+++ b/java/kudu-client/src/main/java/org/apache/kudu/util/BloomFilter.java
@@ -17,9 +17,9 @@
package org.apache.kudu.util;
-import javax.annotation.concurrent.NotThreadSafe;
import java.nio.charset.StandardCharsets;
import java.util.BitSet;
+import javax.annotation.concurrent.NotThreadSafe;
import com.google.common.base.Preconditions;
import com.sangupta.murmur.Murmur2;
diff --git a/java/kudu-client/src/main/java/org/apache/kudu/util/DataGenerator.java b/java/kudu-client/src/main/java/org/apache/kudu/util/DataGenerator.java
index d121b2d..d8ad50b 100644
--- a/java/kudu-client/src/main/java/org/apache/kudu/util/DataGenerator.java
+++ b/java/kudu-client/src/main/java/org/apache/kudu/util/DataGenerator.java
@@ -17,20 +17,21 @@
package org.apache.kudu.util;
+import java.math.BigDecimal;
+import java.math.BigInteger;
+import java.util.List;
+import java.util.Random;
+import javax.xml.bind.DatatypeConverter;
+
import com.google.common.base.Preconditions;
+import org.apache.yetus.audience.InterfaceAudience;
+import org.apache.yetus.audience.InterfaceStability;
+
import org.apache.kudu.ColumnSchema;
import org.apache.kudu.ColumnTypeAttributes;
import org.apache.kudu.Schema;
import org.apache.kudu.Type;
import org.apache.kudu.client.PartialRow;
-import org.apache.yetus.audience.InterfaceAudience;
-import org.apache.yetus.audience.InterfaceStability;
-
-import javax.xml.bind.DatatypeConverter;
-import java.math.BigDecimal;
-import java.math.BigInteger;
-import java.util.List;
-import java.util.Random;
/**
* A utility class to generate random data and rows.
diff --git a/java/kudu-client/src/main/java/org/apache/kudu/util/DecimalUtil.java b/java/kudu-client/src/main/java/org/apache/kudu/util/DecimalUtil.java
index d2c7b5a..f3fa51b 100644
--- a/java/kudu-client/src/main/java/org/apache/kudu/util/DecimalUtil.java
+++ b/java/kudu-client/src/main/java/org/apache/kudu/util/DecimalUtil.java
@@ -17,16 +17,16 @@
package org.apache.kudu.util;
+import static org.apache.kudu.Common.DataType;
+
import java.math.BigDecimal;
import java.math.BigInteger;
import java.math.RoundingMode;
import com.google.common.base.Strings;
-
-import org.apache.kudu.ColumnTypeAttributes;
import org.apache.yetus.audience.InterfaceAudience;
-import static org.apache.kudu.Common.DataType;
+import org.apache.kudu.ColumnTypeAttributes;
@InterfaceAudience.Private
public class DecimalUtil {
diff --git a/java/kudu-client/src/main/java/org/apache/kudu/util/NetUtil.java b/java/kudu-client/src/main/java/org/apache/kudu/util/NetUtil.java
index ced1b70..65d9b59 100644
--- a/java/kudu-client/src/main/java/org/apache/kudu/util/NetUtil.java
+++ b/java/kudu-client/src/main/java/org/apache/kudu/util/NetUtil.java
@@ -28,11 +28,12 @@ import com.google.common.base.Joiner;
import com.google.common.base.Splitter;
import com.google.common.collect.Iterables;
import com.google.common.collect.Lists;
-import org.apache.kudu.client.HostAndPort;
import org.apache.yetus.audience.InterfaceAudience;
import org.slf4j.Logger;
import org.slf4j.LoggerFactory;
+import org.apache.kudu.client.HostAndPort;
+
/**
* Networking related methods.
*/
diff --git a/java/kudu-client/src/main/java/org/apache/kudu/util/SchemaGenerator.java b/java/kudu-client/src/main/java/org/apache/kudu/util/SchemaGenerator.java
index bb5c13d..4116da8 100644
--- a/java/kudu-client/src/main/java/org/apache/kudu/util/SchemaGenerator.java
+++ b/java/kudu-client/src/main/java/org/apache/kudu/util/SchemaGenerator.java
@@ -17,7 +17,19 @@
package org.apache.kudu.util;
+import static org.apache.kudu.util.DataGenerator.randomBinary;
+import static org.apache.kudu.util.DataGenerator.randomDecimal;
+import static org.apache.kudu.util.DataGenerator.randomString;
+
+import java.util.ArrayList;
+import java.util.Arrays;
+import java.util.List;
+import java.util.Random;
+
import com.google.common.base.Preconditions;
+import org.apache.yetus.audience.InterfaceAudience;
+import org.apache.yetus.audience.InterfaceStability;
+
import org.apache.kudu.ColumnSchema;
import org.apache.kudu.ColumnSchema.ColumnSchemaBuilder;
import org.apache.kudu.ColumnSchema.CompressionAlgorithm;
@@ -27,17 +39,6 @@ import org.apache.kudu.Schema;
import org.apache.kudu.Type;
import org.apache.kudu.client.CreateTableOptions;
import org.apache.kudu.client.PartialRow;
-import org.apache.yetus.audience.InterfaceAudience;
-import org.apache.yetus.audience.InterfaceStability;
-
-import java.util.ArrayList;
-import java.util.Arrays;
-import java.util.List;
-import java.util.Random;
-
-import static org.apache.kudu.util.DataGenerator.randomBinary;
-import static org.apache.kudu.util.DataGenerator.randomDecimal;
-import static org.apache.kudu.util.DataGenerator.randomString;
/**
* A utility class to generate random schemas and schema components.
diff --git a/java/kudu-client/src/main/java/org/apache/kudu/util/SecurityUtil.java b/java/kudu-client/src/main/java/org/apache/kudu/util/SecurityUtil.java
index 4ec5734..9d317ea 100644
--- a/java/kudu-client/src/main/java/org/apache/kudu/util/SecurityUtil.java
+++ b/java/kudu-client/src/main/java/org/apache/kudu/util/SecurityUtil.java
@@ -23,7 +23,6 @@ import java.security.cert.X509Certificate;
import java.util.HashMap;
import java.util.Map;
import java.util.Set;
-
import javax.annotation.Nullable;
import javax.security.auth.Subject;
import javax.security.auth.kerberos.KerberosPrincipal;
@@ -35,7 +34,6 @@ import javax.security.auth.login.LoginException;
import com.google.common.base.Joiner;
import com.google.common.base.Preconditions;
-import com.google.common.base.Throwables;
import com.google.common.collect.ImmutableMap;
import org.apache.yetus.audience.InterfaceAudience;
import org.slf4j.Logger;
diff --git a/java/kudu-client/src/main/java/org/apache/kudu/util/TimestampUtil.java b/java/kudu-client/src/main/java/org/apache/kudu/util/TimestampUtil.java
index 959b8a5..a6dde24 100644
--- a/java/kudu-client/src/main/java/org/apache/kudu/util/TimestampUtil.java
+++ b/java/kudu-client/src/main/java/org/apache/kudu/util/TimestampUtil.java
@@ -17,14 +17,14 @@
package org.apache.kudu.util;
-import org.apache.yetus.audience.InterfaceAudience;
-
import java.sql.Timestamp;
import java.text.DateFormat;
import java.text.SimpleDateFormat;
import java.util.Date;
import java.util.TimeZone;
+import org.apache.yetus.audience.InterfaceAudience;
+
@InterfaceAudience.Private
public class TimestampUtil {
diff --git a/java/kudu-client/src/test/java/org/apache/kudu/TestColumnSchema.java b/java/kudu-client/src/test/java/org/apache/kudu/TestColumnSchema.java
index 3f85898..6ca8114 100644
--- a/java/kudu-client/src/test/java/org/apache/kudu/TestColumnSchema.java
+++ b/java/kudu-client/src/test/java/org/apache/kudu/TestColumnSchema.java
@@ -18,14 +18,13 @@ package org.apache.kudu;
import static org.junit.Assert.assertEquals;
import static org.junit.Assert.assertNotEquals;
-import static org.junit.Assert.fail;
import org.junit.Rule;
import org.junit.Test;
import org.apache.kudu.ColumnSchema.ColumnSchemaBuilder;
-import org.apache.kudu.util.DecimalUtil;
import org.apache.kudu.test.junit.RetryRule;
+import org.apache.kudu.util.DecimalUtil;
public class TestColumnSchema {
diff --git a/java/kudu-client/src/test/java/org/apache/kudu/client/ITClient.java b/java/kudu-client/src/test/java/org/apache/kudu/client/ITClient.java
index 09ce6c6..ac4c2e8 100644
--- a/java/kudu-client/src/test/java/org/apache/kudu/client/ITClient.java
+++ b/java/kudu-client/src/test/java/org/apache/kudu/client/ITClient.java
@@ -16,6 +16,11 @@
// under the License.
package org.apache.kudu.client;
+import static org.apache.kudu.test.ClientTestUtil.countRowsInScan;
+import static org.apache.kudu.test.ClientTestUtil.createBasicSchemaInsert;
+import static org.apache.kudu.test.ClientTestUtil.getBasicSchema;
+import static org.apache.kudu.test.KuduTestHarness.DEFAULT_SLEEP;
+
import java.util.ArrayList;
import java.util.List;
import java.util.Random;
@@ -23,9 +28,6 @@ import java.util.concurrent.CountDownLatch;
import java.util.concurrent.TimeUnit;
import com.google.common.collect.ImmutableList;
-import org.apache.kudu.test.KuduTestHarness;
-import org.apache.kudu.test.RandomUtils;
-import org.junit.After;
import org.junit.Assert;
import org.junit.Before;
import org.junit.Rule;
@@ -33,10 +35,8 @@ import org.junit.Test;
import org.slf4j.Logger;
import org.slf4j.LoggerFactory;
-import static org.apache.kudu.test.KuduTestHarness.DEFAULT_SLEEP;
-import static org.apache.kudu.test.ClientTestUtil.countRowsInScan;
-import static org.apache.kudu.test.ClientTestUtil.createBasicSchemaInsert;
-import static org.apache.kudu.test.ClientTestUtil.getBasicSchema;
+import org.apache.kudu.test.KuduTestHarness;
+import org.apache.kudu.test.RandomUtils;
/**
* Integration test for the client. RPCs are sent to Kudu from multiple threads while processes
diff --git a/java/kudu-client/src/test/java/org/apache/kudu/client/ITClientStress.java b/java/kudu-client/src/test/java/org/apache/kudu/client/ITClientStress.java
index f4e476b..9daab10 100644
--- a/java/kudu-client/src/test/java/org/apache/kudu/client/ITClientStress.java
+++ b/java/kudu-client/src/test/java/org/apache/kudu/client/ITClientStress.java
@@ -16,9 +16,9 @@
// under the License.
package org.apache.kudu.client;
-import static org.apache.kudu.test.KuduTestHarness.DEFAULT_SLEEP;
import static org.apache.kudu.test.ClientTestUtil.createFourTabletsTableWithNineRows;
import static org.apache.kudu.test.ClientTestUtil.getBasicCreateTableOptions;
+import static org.apache.kudu.test.KuduTestHarness.DEFAULT_SLEEP;
import static org.junit.Assert.assertFalse;
import static org.junit.Assert.assertTrue;
@@ -33,17 +33,17 @@ import java.util.concurrent.atomic.AtomicReference;
import com.google.common.base.Stopwatch;
import com.google.common.base.Supplier;
-
-import org.apache.kudu.Schema;
-import org.apache.kudu.test.KuduTestHarness;
-import org.apache.kudu.test.ClientTestUtil;
import org.junit.Rule;
import org.junit.Test;
-import org.apache.kudu.client.SessionConfiguration.FlushMode;
-import org.apache.kudu.test.CapturingLogAppender;
import org.slf4j.Logger;
import org.slf4j.LoggerFactory;
+import org.apache.kudu.Schema;
+import org.apache.kudu.client.SessionConfiguration.FlushMode;
+import org.apache.kudu.test.CapturingLogAppender;
+import org.apache.kudu.test.ClientTestUtil;
+import org.apache.kudu.test.KuduTestHarness;
+
public class ITClientStress {
private static final Logger LOG = LoggerFactory.getLogger(ITClientStress.class);
diff --git a/java/kudu-client/src/test/java/org/apache/kudu/client/ITScannerMultiTablet.java b/java/kudu-client/src/test/java/org/apache/kudu/client/ITScannerMultiTablet.java
index 58ed5f3..0e01b08 100644
--- a/java/kudu-client/src/test/java/org/apache/kudu/client/ITScannerMultiTablet.java
+++ b/java/kudu-client/src/test/java/org/apache/kudu/client/ITScannerMultiTablet.java
@@ -23,11 +23,11 @@ import static org.junit.Assert.assertTrue;
import java.util.Random;
import com.google.common.collect.Lists;
-import org.apache.kudu.test.KuduTestHarness;
import org.junit.Before;
+import org.junit.Rule;
import org.apache.kudu.Schema;
-import org.junit.Rule;
+import org.apache.kudu.test.KuduTestHarness;
/**
* Integration test that inserts enough data to trigger flushes and getting multiple data
diff --git a/java/kudu-client/src/test/java/org/apache/kudu/client/TestAlterTable.java b/java/kudu-client/src/test/java/org/apache/kudu/client/TestAlterTable.java
index 2f82bc5..85da8ae 100644
--- a/java/kudu-client/src/test/java/org/apache/kudu/client/TestAlterTable.java
+++ b/java/kudu-client/src/test/java/org/apache/kudu/client/TestAlterTable.java
@@ -27,13 +27,12 @@ import static org.junit.Assert.fail;
import java.util.ArrayList;
import java.util.Arrays;
import java.util.Collections;
-import java.util.List;
import java.util.HashMap;
+import java.util.List;
import java.util.Map;
import com.google.common.collect.ImmutableList;
import com.google.common.collect.Lists;
-import org.apache.kudu.test.KuduTestHarness;
import org.junit.Before;
import org.junit.Rule;
import org.junit.Test;
@@ -43,6 +42,7 @@ import org.apache.kudu.ColumnSchema.CompressionAlgorithm;
import org.apache.kudu.ColumnSchema.Encoding;
import org.apache.kudu.Schema;
import org.apache.kudu.Type;
+import org.apache.kudu.test.KuduTestHarness;
import org.apache.kudu.util.Pair;
public class TestAlterTable {
diff --git a/java/kudu-client/src/test/java/org/apache/kudu/client/TestAsyncKuduClient.java b/java/kudu-client/src/test/java/org/apache/kudu/client/TestAsyncKuduClient.java
index 3f5f27d..dd323e7 100644
--- a/java/kudu-client/src/test/java/org/apache/kudu/client/TestAsyncKuduClient.java
+++ b/java/kudu-client/src/test/java/org/apache/kudu/client/TestAsyncKuduClient.java
@@ -17,10 +17,10 @@
package org.apache.kudu.client;
import static java.nio.charset.StandardCharsets.UTF_8;
-import static org.apache.kudu.test.KuduTestHarness.DEFAULT_SLEEP;
import static org.apache.kudu.test.ClientTestUtil.countRowsInScan;
import static org.apache.kudu.test.ClientTestUtil.createBasicSchemaInsert;
import static org.apache.kudu.test.ClientTestUtil.getBasicCreateTableOptions;
+import static org.apache.kudu.test.KuduTestHarness.DEFAULT_SLEEP;
import static org.junit.Assert.assertEquals;
import static org.junit.Assert.assertNotEquals;
import static org.junit.Assert.assertTrue;
@@ -33,9 +33,6 @@ import java.util.concurrent.TimeUnit;
import com.google.common.base.Stopwatch;
import com.google.protobuf.ByteString;
import com.stumbleupon.async.Deferred;
-import org.apache.kudu.test.KuduTestHarness;
-import org.apache.kudu.test.ClientTestUtil;
-import org.apache.kudu.test.ProtobufUtils;
import org.junit.Before;
import org.junit.Rule;
import org.junit.Test;
@@ -46,6 +43,9 @@ import org.apache.kudu.Schema;
import org.apache.kudu.Type;
import org.apache.kudu.consensus.Metadata;
import org.apache.kudu.master.Master;
+import org.apache.kudu.test.ClientTestUtil;
+import org.apache.kudu.test.KuduTestHarness;
+import org.apache.kudu.test.ProtobufUtils;
public class TestAsyncKuduClient {
diff --git a/java/kudu-client/src/test/java/org/apache/kudu/client/TestAsyncKuduSession.java b/java/kudu-client/src/test/java/org/apache/kudu/client/TestAsyncKuduSession.java
index c6cf6cb..1099373 100644
--- a/java/kudu-client/src/test/java/org/apache/kudu/client/TestAsyncKuduSession.java
+++ b/java/kudu-client/src/test/java/org/apache/kudu/client/TestAsyncKuduSession.java
@@ -16,11 +16,11 @@
// under the License.
package org.apache.kudu.client;
-import static org.apache.kudu.test.KuduTestHarness.DEFAULT_SLEEP;
import static org.apache.kudu.test.ClientTestUtil.countRowsInTable;
import static org.apache.kudu.test.ClientTestUtil.createBasicSchemaInsert;
import static org.apache.kudu.test.ClientTestUtil.getBasicCreateTableOptions;
import static org.apache.kudu.test.ClientTestUtil.getBasicSchema;
+import static org.apache.kudu.test.KuduTestHarness.DEFAULT_SLEEP;
import static org.junit.Assert.assertEquals;
import static org.junit.Assert.assertFalse;
import static org.junit.Assert.assertTrue;
@@ -30,13 +30,13 @@ import java.util.ArrayList;
import java.util.List;
import com.stumbleupon.async.Deferred;
-import org.apache.kudu.test.KuduTestHarness;
import org.junit.Before;
import org.junit.Rule;
import org.junit.Test;
import org.apache.kudu.Schema;
import org.apache.kudu.WireProtocol.AppStatusPB;
+import org.apache.kudu.test.KuduTestHarness;
import org.apache.kudu.tserver.Tserver.TabletServerErrorPB;
public class TestAsyncKuduSession {
diff --git a/java/kudu-client/src/test/java/org/apache/kudu/client/TestAuthTokenReacquire.java b/java/kudu-client/src/test/java/org/apache/kudu/client/TestAuthTokenReacquire.java
index f58bfd7..2ab753c 100644
--- a/java/kudu-client/src/test/java/org/apache/kudu/client/TestAuthTokenReacquire.java
+++ b/java/kudu-client/src/test/java/org/apache/kudu/client/TestAuthTokenReacquire.java
@@ -17,12 +17,6 @@
package org.apache.kudu.client;
-import java.util.ArrayList;
-import java.util.Collections;
-import java.util.HashMap;
-import java.util.Map;
-import java.util.List;
-
import static org.apache.kudu.test.ClientTestUtil.countRowsInScan;
import static org.apache.kudu.test.ClientTestUtil.createBasicSchemaInsert;
import static org.apache.kudu.test.ClientTestUtil.getBasicCreateTableOptions;
@@ -32,17 +26,24 @@ import static org.junit.Assert.assertNotNull;
import static org.junit.Assert.assertTrue;
import static org.junit.Assert.fail;
-import org.apache.kudu.Schema;
-import org.apache.kudu.security.Token;
-import org.apache.kudu.test.cluster.MiniKuduCluster.MiniKuduClusterBuilder;
-import org.apache.kudu.test.KuduTestHarness;
-import org.apache.kudu.test.ClientTestUtil;
+import java.util.ArrayList;
+import java.util.Collections;
+import java.util.HashMap;
+import java.util.List;
+import java.util.Map;
+
import org.junit.Before;
import org.junit.Rule;
import org.junit.Test;
import org.slf4j.Logger;
import org.slf4j.LoggerFactory;
+import org.apache.kudu.Schema;
+import org.apache.kudu.security.Token;
+import org.apache.kudu.test.ClientTestUtil;
+import org.apache.kudu.test.KuduTestHarness;
+import org.apache.kudu.test.cluster.MiniKuduCluster.MiniKuduClusterBuilder;
+
/**
* This test contains scenarios to verify that clients re-acquire tokens upon
* expiration of the current one and automatically retries affected calls.
diff --git a/java/kudu-client/src/test/java/org/apache/kudu/client/TestAuthnTokenReacquireOpen.java b/java/kudu-client/src/test/java/org/apache/kudu/client/TestAuthnTokenReacquireOpen.java
index 32a499c..00f84ef 100644
--- a/java/kudu-client/src/test/java/org/apache/kudu/client/TestAuthnTokenReacquireOpen.java
+++ b/java/kudu-client/src/test/java/org/apache/kudu/client/TestAuthnTokenReacquireOpen.java
@@ -24,14 +24,15 @@ import static org.junit.Assert.assertFalse;
import static org.junit.Assert.assertNotNull;
import static org.junit.Assert.assertTrue;
-import org.apache.kudu.Schema;
-import org.apache.kudu.test.cluster.MiniKuduCluster.MiniKuduClusterBuilder;
-import org.apache.kudu.test.KuduTestHarness;
-import org.apache.kudu.test.ClientTestUtil;
import org.junit.Before;
import org.junit.Rule;
import org.junit.Test;
+import org.apache.kudu.Schema;
+import org.apache.kudu.test.ClientTestUtil;
+import org.apache.kudu.test.KuduTestHarness;
+import org.apache.kudu.test.cluster.MiniKuduCluster.MiniKuduClusterBuilder;
+
/**
* This test contains a special scenario to make sure the automatic authn token re-acquisition works
* in the case when the client has established a connection to the master using secondary
diff --git a/java/kudu-client/src/test/java/org/apache/kudu/client/TestAuthzTokenCache.java b/java/kudu-client/src/test/java/org/apache/kudu/client/TestAuthzTokenCache.java
index 0b229a9..b697b2f 100644
--- a/java/kudu-client/src/test/java/org/apache/kudu/client/TestAuthzTokenCache.java
+++ b/java/kudu-client/src/test/java/org/apache/kudu/client/TestAuthzTokenCache.java
@@ -17,15 +17,12 @@
package org.apache.kudu.client;
-import com.stumbleupon.async.Deferred;
-import org.apache.kudu.security.Token;
-import org.apache.kudu.test.KuduTestHarness;
-import org.apache.kudu.test.cluster.MiniKuduCluster;
-import org.junit.Before;
-import org.junit.Rule;
-import org.junit.Test;
-import org.slf4j.Logger;
-import org.slf4j.LoggerFactory;
+import static org.apache.kudu.test.ClientTestUtil.getBasicCreateTableOptions;
+import static org.apache.kudu.test.ClientTestUtil.getBasicSchema;
+import static org.junit.Assert.assertEquals;
+import static org.junit.Assert.assertFalse;
+import static org.junit.Assert.assertNotNull;
+import static org.junit.Assert.assertTrue;
import java.util.ArrayList;
import java.util.List;
@@ -34,12 +31,16 @@ import java.util.concurrent.ExecutorService;
import java.util.concurrent.Executors;
import java.util.concurrent.Future;
-import static org.apache.kudu.test.ClientTestUtil.getBasicCreateTableOptions;
-import static org.apache.kudu.test.ClientTestUtil.getBasicSchema;
-import static org.junit.Assert.assertEquals;
-import static org.junit.Assert.assertFalse;
-import static org.junit.Assert.assertNotNull;
-import static org.junit.Assert.assertTrue;
+import com.stumbleupon.async.Deferred;
+import org.junit.Before;
+import org.junit.Rule;
+import org.junit.Test;
+import org.slf4j.Logger;
+import org.slf4j.LoggerFactory;
+
+import org.apache.kudu.security.Token;
+import org.apache.kudu.test.KuduTestHarness;
+import org.apache.kudu.test.cluster.MiniKuduCluster;
public class TestAuthzTokenCache {
private static final Logger LOG = LoggerFactory.getLogger(TestAuthzTokenCache.class);
diff --git a/java/kudu-client/src/test/java/org/apache/kudu/client/TestBloomFilter.java b/java/kudu-client/src/test/java/org/apache/kudu/client/TestBloomFilter.java
index d2e2a85..22fe72b 100644
--- a/java/kudu-client/src/test/java/org/apache/kudu/client/TestBloomFilter.java
+++ b/java/kudu-client/src/test/java/org/apache/kudu/client/TestBloomFilter.java
@@ -24,8 +24,8 @@ import java.util.Random;
import org.junit.Rule;
import org.junit.Test;
-import org.apache.kudu.util.BloomFilter;
import org.apache.kudu.test.junit.RetryRule;
+import org.apache.kudu.util.BloomFilter;
public class TestBloomFilter {
diff --git a/java/kudu-client/src/test/java/org/apache/kudu/client/TestBytes.java b/java/kudu-client/src/test/java/org/apache/kudu/client/TestBytes.java
index a27c1da..e475163 100644
--- a/java/kudu-client/src/test/java/org/apache/kudu/client/TestBytes.java
+++ b/java/kudu-client/src/test/java/org/apache/kudu/client/TestBytes.java
@@ -24,12 +24,12 @@ import java.math.BigInteger;
import java.math.MathContext;
import java.math.RoundingMode;
-import org.junit.Rule;
import org.junit.Assert;
+import org.junit.Rule;
import org.junit.Test;
-import org.apache.kudu.util.DecimalUtil;
import org.apache.kudu.test.junit.RetryRule;
+import org.apache.kudu.util.DecimalUtil;
public class TestBytes {
diff --git a/java/kudu-client/src/test/java/org/apache/kudu/client/TestClientFailoverSupport.java b/java/kudu-client/src/test/java/org/apache/kudu/client/TestClientFailoverSupport.java
index e24cb5e..16d2618 100644
--- a/java/kudu-client/src/test/java/org/apache/kudu/client/TestClientFailoverSupport.java
+++ b/java/kudu-client/src/test/java/org/apache/kudu/client/TestClientFailoverSupport.java
@@ -16,11 +16,11 @@
// under the License.
package org.apache.kudu.client;
-import static org.apache.kudu.test.KuduTestHarness.DEFAULT_SLEEP;
-import static org.apache.kudu.test.junit.AssertHelpers.assertEventuallyTrue;
import static org.apache.kudu.test.ClientTestUtil.countRowsInScan;
import static org.apache.kudu.test.ClientTestUtil.createBasicSchemaInsert;
import static org.apache.kudu.test.ClientTestUtil.getBasicCreateTableOptions;
+import static org.apache.kudu.test.KuduTestHarness.DEFAULT_SLEEP;
+import static org.apache.kudu.test.junit.AssertHelpers.assertEventuallyTrue;
import static org.junit.Assert.assertEquals;
import static org.junit.Assert.assertFalse;
@@ -28,16 +28,17 @@ import java.io.Closeable;
import java.io.IOException;
import java.util.List;
-import org.apache.kudu.Schema;
-import org.apache.kudu.test.KuduTestHarness;
-import org.apache.kudu.test.junit.AssertHelpers.BooleanExpression;
-import org.apache.kudu.test.CapturingLogAppender;
-import org.apache.kudu.test.ClientTestUtil;
import org.junit.After;
import org.junit.Before;
import org.junit.Rule;
import org.junit.Test;
+import org.apache.kudu.Schema;
+import org.apache.kudu.test.CapturingLogAppender;
+import org.apache.kudu.test.ClientTestUtil;
+import org.apache.kudu.test.KuduTestHarness;
+import org.apache.kudu.test.junit.AssertHelpers.BooleanExpression;
+
public class TestClientFailoverSupport {
private static final Schema basicSchema = ClientTestUtil.getBasicSchema();
diff --git a/java/kudu-client/src/test/java/org/apache/kudu/client/TestColumnRangePredicate.java b/java/kudu-client/src/test/java/org/apache/kudu/client/TestColumnRangePredicate.java
index e1f004d..3732921 100644
--- a/java/kudu-client/src/test/java/org/apache/kudu/client/TestColumnRangePredicate.java
+++ b/java/kudu-client/src/test/java/org/apache/kudu/client/TestColumnRangePredicate.java
@@ -24,14 +24,14 @@ import java.math.BigDecimal;
import java.util.List;
import com.google.common.collect.Lists;
-import org.junit.Test;
import org.junit.Rule;
+import org.junit.Test;
import org.apache.kudu.ColumnSchema;
import org.apache.kudu.ColumnTypeAttributes;
import org.apache.kudu.Type;
-import org.apache.kudu.tserver.Tserver;
import org.apache.kudu.test.junit.RetryRule;
+import org.apache.kudu.tserver.Tserver;
public class TestColumnRangePredicate {
diff --git a/java/kudu-client/src/test/java/org/apache/kudu/client/TestConnectionCache.java b/java/kudu-client/src/test/java/org/apache/kudu/client/TestConnectionCache.java
index 8750d41..f111053 100644
--- a/java/kudu-client/src/test/java/org/apache/kudu/client/TestConnectionCache.java
+++ b/java/kudu-client/src/test/java/org/apache/kudu/client/TestConnectionCache.java
@@ -23,11 +23,11 @@ import static org.junit.Assert.assertNotSame;
import static org.junit.Assert.assertTrue;
import com.stumbleupon.async.Deferred;
-import org.apache.kudu.test.cluster.MiniKuduCluster;
-import org.apache.kudu.test.junit.RetryRule;
import org.junit.Rule;
import org.junit.Test;
+import org.apache.kudu.test.cluster.MiniKuduCluster;
+import org.apache.kudu.test.junit.RetryRule;
import org.apache.kudu.util.NetUtil;
public class TestConnectionCache {
diff --git a/java/kudu-client/src/test/java/org/apache/kudu/client/TestFlexiblePartitioning.java b/java/kudu-client/src/test/java/org/apache/kudu/client/TestFlexiblePartitioning.java
index 7c135fb..517bb99 100644
--- a/java/kudu-client/src/test/java/org/apache/kudu/client/TestFlexiblePartitioning.java
+++ b/java/kudu-client/src/test/java/org/apache/kudu/client/TestFlexiblePartitioning.java
@@ -31,7 +31,6 @@ import com.google.common.base.Predicates;
import com.google.common.collect.ComparisonChain;
import com.google.common.collect.ImmutableList;
import com.google.common.collect.Sets;
-import org.apache.kudu.test.KuduTestHarness;
import org.junit.Before;
import org.junit.Rule;
import org.junit.Test;
@@ -41,6 +40,7 @@ import org.slf4j.LoggerFactory;
import org.apache.kudu.ColumnSchema;
import org.apache.kudu.Schema;
import org.apache.kudu.Type;
+import org.apache.kudu.test.KuduTestHarness;
public class TestFlexiblePartitioning {
private static final Logger LOG = LoggerFactory.getLogger(TestKuduClient.class);
diff --git a/java/kudu-client/src/test/java/org/apache/kudu/client/TestHandleTooBusy.java b/java/kudu-client/src/test/java/org/apache/kudu/client/TestHandleTooBusy.java
index d4d8687..c540867 100644
--- a/java/kudu-client/src/test/java/org/apache/kudu/client/TestHandleTooBusy.java
+++ b/java/kudu-client/src/test/java/org/apache/kudu/client/TestHandleTooBusy.java
@@ -16,6 +16,8 @@
// under the License.
package org.apache.kudu.client;
+import static org.apache.kudu.test.ClientTestUtil.getBasicCreateTableOptions;
+
import java.util.List;
import java.util.concurrent.Callable;
import java.util.concurrent.ExecutorService;
@@ -23,14 +25,13 @@ import java.util.concurrent.Executors;
import java.util.concurrent.Future;
import com.google.common.collect.Lists;
-import org.apache.kudu.Schema;
-import org.apache.kudu.test.cluster.MiniKuduCluster.MiniKuduClusterBuilder;
-import org.apache.kudu.test.KuduTestHarness;
-import org.apache.kudu.test.ClientTestUtil;
import org.junit.Rule;
import org.junit.Test;
-import static org.apache.kudu.test.ClientTestUtil.getBasicCreateTableOptions;
+import org.apache.kudu.Schema;
+import org.apache.kudu.test.ClientTestUtil;
+import org.apache.kudu.test.KuduTestHarness;
+import org.apache.kudu.test.cluster.MiniKuduCluster.MiniKuduClusterBuilder;
/**
* Tests which provoke RPC queue overflow errors on the server side
diff --git a/java/kudu-client/src/test/java/org/apache/kudu/client/TestHybridTime.java b/java/kudu-client/src/test/java/org/apache/kudu/client/TestHybridTime.java
index cf2d0bc..160d23b 100644
--- a/java/kudu-client/src/test/java/org/apache/kudu/client/TestHybridTime.java
+++ b/java/kudu-client/src/test/java/org/apache/kudu/client/TestHybridTime.java
@@ -32,8 +32,6 @@ import java.util.concurrent.TimeUnit;
import com.google.common.collect.ImmutableList;
import com.google.common.collect.Ordering;
-import org.apache.kudu.test.cluster.MiniKuduCluster.MiniKuduClusterBuilder;
-import org.apache.kudu.test.KuduTestHarness;
import org.junit.Before;
import org.junit.Rule;
import org.junit.Test;
@@ -42,6 +40,8 @@ import org.slf4j.LoggerFactory;
import org.apache.kudu.ColumnSchema;
import org.apache.kudu.Schema;
+import org.apache.kudu.test.KuduTestHarness;
+import org.apache.kudu.test.cluster.MiniKuduCluster.MiniKuduClusterBuilder;
/**
* Tests client propagated timestamps. All the work for commit wait is done and tested on the
diff --git a/java/kudu-client/src/test/java/org/apache/kudu/client/TestKeyEncoding.java b/java/kudu-client/src/test/java/org/apache/kudu/client/TestKeyEncoding.java
index 2b80d4a..376351f 100644
--- a/java/kudu-client/src/test/java/org/apache/kudu/client/TestKeyEncoding.java
+++ b/java/kudu-client/src/test/java/org/apache/kudu/client/TestKeyEncoding.java
@@ -22,11 +22,9 @@ import static org.junit.Assert.assertTrue;
import java.math.BigDecimal;
import java.util.ArrayList;
-import java.util.EnumSet;
import java.util.List;
import com.google.common.collect.ImmutableList;
-import org.apache.kudu.test.KuduTestHarness;
import org.junit.Before;
import org.junit.Rule;
import org.junit.Test;
@@ -38,7 +36,7 @@ import org.apache.kudu.Schema;
import org.apache.kudu.Type;
import org.apache.kudu.client.PartitionSchema.HashBucketSchema;
import org.apache.kudu.client.PartitionSchema.RangeSchema;
-import org.apache.kudu.client.ProtobufHelper.SchemaPBConversionFlags;
+import org.apache.kudu.test.KuduTestHarness;
import org.apache.kudu.util.DecimalUtil;
public class TestKeyEncoding {
diff --git a/java/kudu-client/src/test/java/org/apache/kudu/client/TestKuduClient.java b/java/kudu-client/src/test/java/org/apache/kudu/client/TestKuduClient.java
index 0f71b5f..288dd81 100644
--- a/java/kudu-client/src/test/java/org/apache/kudu/client/TestKuduClient.java
+++ b/java/kudu-client/src/test/java/org/apache/kudu/client/TestKuduClient.java
@@ -46,9 +46,9 @@ import java.lang.reflect.Method;
import java.math.BigDecimal;
import java.util.ArrayList;
import java.util.Collections;
+import java.util.HashMap;
import java.util.List;
import java.util.Map;
-import java.util.HashMap;
import java.util.Random;
import java.util.concurrent.Callable;
import java.util.concurrent.ExecutorService;
@@ -57,24 +57,23 @@ import java.util.concurrent.Future;
import com.google.common.collect.ImmutableList;
import com.stumbleupon.async.Deferred;
-
-import org.apache.kudu.test.ClientTestUtil;
-import org.apache.kudu.test.KuduTestHarness;
-import org.apache.kudu.test.KuduTestHarness.LocationConfig;
-import org.apache.kudu.test.KuduTestHarness.TabletServerConfig;
-import org.apache.kudu.test.RandomUtils;
-import org.apache.kudu.util.TimestampUtil;
import org.junit.Before;
import org.junit.Rule;
import org.junit.Test;
+import org.slf4j.Logger;
+import org.slf4j.LoggerFactory;
import org.apache.kudu.ColumnSchema;
import org.apache.kudu.Schema;
import org.apache.kudu.Type;
import org.apache.kudu.test.CapturingLogAppender;
+import org.apache.kudu.test.ClientTestUtil;
+import org.apache.kudu.test.KuduTestHarness;
+import org.apache.kudu.test.KuduTestHarness.LocationConfig;
+import org.apache.kudu.test.KuduTestHarness.TabletServerConfig;
+import org.apache.kudu.test.RandomUtils;
import org.apache.kudu.util.DecimalUtil;
-import org.slf4j.Logger;
-import org.slf4j.LoggerFactory;
+import org.apache.kudu.util.TimestampUtil;
public class TestKuduClient {
private static final Logger LOG = LoggerFactory.getLogger(TestKuduClient.class);
diff --git a/java/kudu-client/src/test/java/org/apache/kudu/client/TestKuduPartitioner.java b/java/kudu-client/src/test/java/org/apache/kudu/client/TestKuduPartitioner.java
index 8a61a8f..743926c 100644
--- a/java/kudu-client/src/test/java/org/apache/kudu/client/TestKuduPartitioner.java
+++ b/java/kudu-client/src/test/java/org/apache/kudu/client/TestKuduPartitioner.java
@@ -17,25 +17,23 @@
package org.apache.kudu.client;
-import org.apache.kudu.Schema;
-import org.apache.kudu.test.KuduTestHarness;
-import org.junit.Before;
-import org.junit.Rule;
-import org.junit.Test;
-import org.slf4j.Logger;
-import org.slf4j.LoggerFactory;
+import static org.apache.kudu.test.ClientTestUtil.getBasicSchema;
+import static org.apache.kudu.test.ClientTestUtil.getBasicTableOptionsWithNonCoveredRange;
+import static org.junit.Assert.assertEquals;
+import static org.junit.Assert.assertTrue;
+import static org.junit.Assert.fail;
import java.util.ArrayList;
import java.util.Arrays;
import java.util.Collections;
import java.util.List;
-import static org.apache.kudu.test.ClientTestUtil.getBasicCreateTableOptions;
-import static org.apache.kudu.test.ClientTestUtil.getBasicSchema;
-import static org.apache.kudu.test.ClientTestUtil.getBasicTableOptionsWithNonCoveredRange;
-import static org.junit.Assert.assertEquals;
-import static org.junit.Assert.assertTrue;
-import static org.junit.Assert.fail;
+import org.junit.Before;
+import org.junit.Rule;
+import org.junit.Test;
+
+import org.apache.kudu.Schema;
+import org.apache.kudu.test.KuduTestHarness;
public class TestKuduPartitioner {
diff --git a/java/kudu-client/src/test/java/org/apache/kudu/client/TestKuduPredicate.java b/java/kudu-client/src/test/java/org/apache/kudu/client/TestKuduPredicate.java
index 7aec32e..a14ea94 100644
--- a/java/kudu-client/src/test/java/org/apache/kudu/client/TestKuduPredicate.java
+++ b/java/kudu-client/src/test/java/org/apache/kudu/client/TestKuduPredicate.java
@@ -36,8 +36,8 @@ import org.junit.Test;
import org.apache.kudu.ColumnSchema;
import org.apache.kudu.Type;
-import org.apache.kudu.util.DecimalUtil;
import org.apache.kudu.test.junit.RetryRule;
+import org.apache.kudu.util.DecimalUtil;
public class TestKuduPredicate {
diff --git a/java/kudu-client/src/test/java/org/apache/kudu/client/TestKuduScanner.java b/java/kudu-client/src/test/java/org/apache/kudu/client/TestKuduScanner.java
index 1047ae0..155e226 100644
--- a/java/kudu-client/src/test/java/org/apache/kudu/client/TestKuduScanner.java
+++ b/java/kudu-client/src/test/java/org/apache/kudu/client/TestKuduScanner.java
@@ -16,21 +16,15 @@
// under the License.
package org.apache.kudu.client;
-import org.apache.kudu.ColumnSchema;
-import org.apache.kudu.Common.DataType;
-import org.apache.kudu.Schema;
-import org.apache.kudu.Type;
-import org.apache.kudu.client.Operation.ChangeType;
-import org.apache.kudu.test.CapturingLogAppender;
-import org.apache.kudu.test.KuduTestHarness;
-import org.apache.kudu.test.RandomUtils;
-import org.apache.kudu.util.DataGenerator;
-import org.apache.kudu.util.Pair;
-import org.junit.Before;
-import org.junit.Rule;
-import org.junit.Test;
-import org.slf4j.Logger;
-import org.slf4j.LoggerFactory;
+import static org.apache.kudu.client.AsyncKuduScanner.DEFAULT_IS_DELETED_COL_NAME;
+import static org.apache.kudu.test.ClientTestUtil.getBasicCreateTableOptions;
+import static org.apache.kudu.test.ClientTestUtil.getBasicSchema;
+import static org.apache.kudu.test.ClientTestUtil.loadDefaultTable;
+import static org.junit.Assert.assertEquals;
+import static org.junit.Assert.assertFalse;
+import static org.junit.Assert.assertNull;
+import static org.junit.Assert.assertTrue;
+import static org.junit.Assert.fail;
import java.io.Closeable;
import java.util.ArrayList;
@@ -43,15 +37,22 @@ import java.util.Map;
import java.util.Random;
import java.util.Set;
-import static org.apache.kudu.client.AsyncKuduScanner.DEFAULT_IS_DELETED_COL_NAME;
-import static org.apache.kudu.test.ClientTestUtil.getBasicCreateTableOptions;
-import static org.apache.kudu.test.ClientTestUtil.getBasicSchema;
-import static org.apache.kudu.test.ClientTestUtil.loadDefaultTable;
-import static org.junit.Assert.assertEquals;
-import static org.junit.Assert.assertFalse;
-import static org.junit.Assert.assertNull;
-import static org.junit.Assert.assertTrue;
-import static org.junit.Assert.fail;
+import org.junit.Before;
+import org.junit.Rule;
+import org.junit.Test;
+import org.slf4j.Logger;
+import org.slf4j.LoggerFactory;
+
+import org.apache.kudu.ColumnSchema;
+import org.apache.kudu.Common.DataType;
+import org.apache.kudu.Schema;
+import org.apache.kudu.Type;
+import org.apache.kudu.client.Operation.ChangeType;
+import org.apache.kudu.test.CapturingLogAppender;
+import org.apache.kudu.test.KuduTestHarness;
+import org.apache.kudu.test.RandomUtils;
+import org.apache.kudu.util.DataGenerator;
+import org.apache.kudu.util.Pair;
public class TestKuduScanner {
diff --git a/java/kudu-client/src/test/java/org/apache/kudu/client/TestKuduSession.java b/java/kudu-client/src/test/java/org/apache/kudu/client/TestKuduSession.java
index b293a52..af3b5ed 100644
--- a/java/kudu-client/src/test/java/org/apache/kudu/client/TestKuduSession.java
+++ b/java/kudu-client/src/test/java/org/apache/kudu/client/TestKuduSession.java
@@ -32,13 +32,14 @@ import java.util.ArrayList;
import java.util.List;
import com.google.common.collect.ImmutableList;
-import org.apache.kudu.Schema;
-import org.apache.kudu.test.KuduTestHarness;
-import org.apache.kudu.test.ClientTestUtil;
import org.junit.Before;
import org.junit.Rule;
import org.junit.Test;
+import org.apache.kudu.Schema;
+import org.apache.kudu.test.ClientTestUtil;
+import org.apache.kudu.test.KuduTestHarness;
+
public class TestKuduSession {
private static final String tableName = "TestKuduSession";
diff --git a/java/kudu-client/src/test/java/org/apache/kudu/client/TestKuduTable.java b/java/kudu-client/src/test/java/org/apache/kudu/client/TestKuduTable.java
index a3d3d2c..d1332e3 100644
--- a/java/kudu-client/src/test/java/org/apache/kudu/client/TestKuduTable.java
+++ b/java/kudu-client/src/test/java/org/apache/kudu/client/TestKuduTable.java
@@ -16,12 +16,12 @@
// under the License.
package org.apache.kudu.client;
-import static org.apache.kudu.test.KuduTestHarness.DEFAULT_SLEEP;
import static org.apache.kudu.test.ClientTestUtil.createBasicSchemaInsert;
import static org.apache.kudu.test.ClientTestUtil.getBasicCreateTableOptions;
import static org.apache.kudu.test.ClientTestUtil.getBasicSchema;
import static org.apache.kudu.test.ClientTestUtil.getBasicTableOptionsWithNonCoveredRange;
import static org.apache.kudu.test.ClientTestUtil.scanTableToStrings;
+import static org.apache.kudu.test.KuduTestHarness.DEFAULT_SLEEP;
import static org.junit.Assert.assertArrayEquals;
import static org.junit.Assert.assertEquals;
import static org.junit.Assert.assertFalse;
@@ -34,17 +34,17 @@ import java.util.List;
import com.google.common.collect.ImmutableList;
import com.google.common.collect.Lists;
-import org.apache.kudu.test.KuduTestHarness;
-import org.apache.kudu.test.ClientTestUtil;
import org.junit.Before;
import org.junit.Rule;
import org.junit.Test;
+import org.slf4j.Logger;
+import org.slf4j.LoggerFactory;
import org.apache.kudu.ColumnSchema;
import org.apache.kudu.Schema;
import org.apache.kudu.Type;
-import org.slf4j.Logger;
-import org.slf4j.LoggerFactory;
+import org.apache.kudu.test.ClientTestUtil;
+import org.apache.kudu.test.KuduTestHarness;
public class TestKuduTable {
private static final Logger LOG = LoggerFactory.getLogger(TestKuduTable.class);
diff --git a/java/kudu-client/src/test/java/org/apache/kudu/client/TestLeaderFailover.java b/java/kudu-client/src/test/java/org/apache/kudu/client/TestLeaderFailover.java
index 663d512..78f722b 100644
--- a/java/kudu-client/src/test/java/org/apache/kudu/client/TestLeaderFailover.java
+++ b/java/kudu-client/src/test/java/org/apache/kudu/client/TestLeaderFailover.java
@@ -23,11 +23,12 @@ import static org.apache.kudu.test.ClientTestUtil.getBasicSchema;
import static org.junit.Assert.assertEquals;
import static org.junit.Assert.fail;
-import org.apache.kudu.test.KuduTestHarness;
import org.junit.Before;
import org.junit.Rule;
import org.junit.Test;
+import org.apache.kudu.test.KuduTestHarness;
+
public class TestLeaderFailover {
private static final String TABLE_NAME =
diff --git a/java/kudu-client/src/test/java/org/apache/kudu/client/TestMasterFailover.java b/java/kudu-client/src/test/java/org/apache/kudu/client/TestMasterFailover.java
index 74004b9..d0dcebf 100644
--- a/java/kudu-client/src/test/java/org/apache/kudu/client/TestMasterFailover.java
+++ b/java/kudu-client/src/test/java/org/apache/kudu/client/TestMasterFailover.java
@@ -21,10 +21,11 @@ import static org.apache.kudu.test.ClientTestUtil.getBasicCreateTableOptions;
import static org.apache.kudu.test.ClientTestUtil.getBasicSchema;
import static org.junit.Assert.assertEquals;
-import org.apache.kudu.test.KuduTestHarness;
import org.junit.Rule;
import org.junit.Test;
+import org.apache.kudu.test.KuduTestHarness;
+
/**
* Tests {@link AsyncKuduClient} with multiple masters.
*/
diff --git a/java/kudu-client/src/test/java/org/apache/kudu/client/TestMultiMasterAuthzTokens.java b/java/kudu-client/src/test/java/org/apache/kudu/client/TestMultiMasterAuthzTokens.java
index b13264f..edcc73e 100644
--- a/java/kudu-client/src/test/java/org/apache/kudu/client/TestMultiMasterAuthzTokens.java
+++ b/java/kudu-client/src/test/java/org/apache/kudu/client/TestMultiMasterAuthzTokens.java
@@ -17,11 +17,14 @@
package org.apache.kudu.client;
-import org.apache.kudu.test.KuduTestHarness;
-import org.apache.kudu.test.cluster.MiniKuduCluster;
-import org.junit.Before;
-import org.junit.Rule;
-import org.junit.Test;
+import static org.apache.kudu.client.SessionConfiguration.FlushMode.AUTO_FLUSH_BACKGROUND;
+import static org.apache.kudu.client.SessionConfiguration.FlushMode.AUTO_FLUSH_SYNC;
+import static org.apache.kudu.test.ClientTestUtil.countRowsInTable;
+import static org.apache.kudu.test.ClientTestUtil.createBasicSchemaInsert;
+import static org.apache.kudu.test.ClientTestUtil.createBasicSchemaUpsert;
+import static org.apache.kudu.test.ClientTestUtil.getBasicCreateTableOptions;
+import static org.apache.kudu.test.ClientTestUtil.getBasicSchema;
+import static org.junit.Assert.assertEquals;
import java.util.ArrayList;
import java.util.List;
@@ -31,10 +34,12 @@ import java.util.concurrent.ExecutorService;
import java.util.concurrent.Executors;
import java.util.concurrent.Future;
-import static org.apache.kudu.client.SessionConfiguration.FlushMode.AUTO_FLUSH_BACKGROUND;
-import static org.apache.kudu.client.SessionConfiguration.FlushMode.AUTO_FLUSH_SYNC;
-import static org.apache.kudu.test.ClientTestUtil.*;
-import static org.junit.Assert.assertEquals;
+import org.junit.Before;
+import org.junit.Rule;
+import org.junit.Test;
+
+import org.apache.kudu.test.KuduTestHarness;
+import org.apache.kudu.test.cluster.MiniKuduCluster;
public class TestMultiMasterAuthzTokens {
private static final MiniKuduCluster.MiniKuduClusterBuilder clusterBuilder =
diff --git a/java/kudu-client/src/test/java/org/apache/kudu/client/TestMultipleLeaderFailover.java b/java/kudu-client/src/test/java/org/apache/kudu/client/TestMultipleLeaderFailover.java
index 5d47fc7..52edd32 100644
--- a/java/kudu-client/src/test/java/org/apache/kudu/client/TestMultipleLeaderFailover.java
+++ b/java/kudu-client/src/test/java/org/apache/kudu/client/TestMultipleLeaderFailover.java
@@ -16,22 +16,23 @@
// under the License.
package org.apache.kudu.client;
-import static org.apache.kudu.test.KuduTestHarness.DEFAULT_SLEEP;
-import static org.apache.kudu.test.junit.AssertHelpers.assertEventuallyTrue;
import static org.apache.kudu.test.ClientTestUtil.countRowsInScan;
import static org.apache.kudu.test.ClientTestUtil.createBasicSchemaInsert;
import static org.apache.kudu.test.ClientTestUtil.getBasicCreateTableOptions;
import static org.apache.kudu.test.ClientTestUtil.getBasicSchema;
+import static org.apache.kudu.test.KuduTestHarness.DEFAULT_SLEEP;
+import static org.apache.kudu.test.junit.AssertHelpers.assertEventuallyTrue;
import static org.junit.Assert.assertEquals;
import static org.junit.Assert.fail;
import java.util.List;
-import org.apache.kudu.test.KuduTestHarness;
-import org.apache.kudu.test.junit.AssertHelpers.BooleanExpression;
import org.junit.Rule;
import org.junit.Test;
+import org.apache.kudu.test.KuduTestHarness;
+import org.apache.kudu.test.junit.AssertHelpers.BooleanExpression;
+
public class TestMultipleLeaderFailover {
@Rule
diff --git a/java/kudu-client/src/test/java/org/apache/kudu/client/TestNegotiation.java b/java/kudu-client/src/test/java/org/apache/kudu/client/TestNegotiation.java
index bfcf716..89ff3da 100644
--- a/java/kudu-client/src/test/java/org/apache/kudu/client/TestNegotiation.java
+++ b/java/kudu-client/src/test/java/org/apache/kudu/client/TestNegotiation.java
@@ -17,16 +17,17 @@
package org.apache.kudu.client;
-import org.apache.kudu.test.cluster.MiniKuduCluster;
-import org.apache.kudu.test.junit.RetryRule;
-import org.apache.kudu.test.cluster.FakeDNS;
-import org.apache.kudu.test.CapturingLogAppender;
-import org.junit.Rule;
-import org.junit.Test;
+import static junit.framework.TestCase.assertTrue;
import java.io.Closeable;
-import static junit.framework.TestCase.assertTrue;
+import org.junit.Rule;
+import org.junit.Test;
+
+import org.apache.kudu.test.CapturingLogAppender;
+import org.apache.kudu.test.cluster.FakeDNS;
+import org.apache.kudu.test.cluster.MiniKuduCluster;
+import org.apache.kudu.test.junit.RetryRule;
public class TestNegotiation {
diff --git a/java/kudu-client/src/test/java/org/apache/kudu/client/TestPartitionPruner.java b/java/kudu-client/src/test/java/org/apache/kudu/client/TestPartitionPruner.java
index 34a83ed..c7b2e3b 100644
--- a/java/kudu-client/src/test/java/org/apache/kudu/client/TestPartitionPruner.java
+++ b/java/kudu-client/src/test/java/org/apache/kudu/client/TestPartitionPruner.java
@@ -22,7 +22,6 @@ import java.util.ArrayList;
import java.util.List;
import com.google.common.collect.ImmutableList;
-import org.apache.kudu.test.KuduTestHarness;
import org.junit.Before;
import org.junit.Rule;
import org.junit.Test;
@@ -31,6 +30,7 @@ import org.apache.kudu.ColumnSchema;
import org.apache.kudu.Schema;
import org.apache.kudu.Type;
import org.apache.kudu.client.KuduPredicate.ComparisonOp;
+import org.apache.kudu.test.KuduTestHarness;
public class TestPartitionPruner {
diff --git a/java/kudu-client/src/test/java/org/apache/kudu/client/TestRemoteTablet.java b/java/kudu-client/src/test/java/org/apache/kudu/client/TestRemoteTablet.java
index c4372c1..3c58df0 100644
--- a/java/kudu-client/src/test/java/org/apache/kudu/client/TestRemoteTablet.java
+++ b/java/kudu-client/src/test/java/org/apache/kudu/client/TestRemoteTablet.java
@@ -27,7 +27,6 @@ import java.net.UnknownHostException;
import java.util.ArrayList;
import java.util.List;
-import com.google.protobuf.ByteString;
import org.junit.Rule;
import org.junit.Test;
diff --git a/java/kudu-client/src/test/java/org/apache/kudu/client/TestRowErrors.java b/java/kudu-client/src/test/java/org/apache/kudu/client/TestRowErrors.java
index 5391e82..c45d8e9 100644
--- a/java/kudu-client/src/test/java/org/apache/kudu/client/TestRowErrors.java
+++ b/java/kudu-client/src/test/java/org/apache/kudu/client/TestRowErrors.java
@@ -16,21 +16,22 @@
// under the License.
package org.apache.kudu.client;
-import static org.apache.kudu.test.KuduTestHarness.DEFAULT_SLEEP;
import static org.apache.kudu.test.ClientTestUtil.createBasicSchemaInsert;
import static org.apache.kudu.test.ClientTestUtil.createFourTabletsTableWithNineRows;
import static org.apache.kudu.test.ClientTestUtil.getBasicCreateTableOptions;
import static org.apache.kudu.test.ClientTestUtil.getBasicSchema;
+import static org.apache.kudu.test.KuduTestHarness.DEFAULT_SLEEP;
import static org.junit.Assert.assertEquals;
import static org.junit.Assert.assertTrue;
import java.util.List;
-import org.apache.kudu.Schema;
-import org.apache.kudu.test.KuduTestHarness;
import org.junit.Rule;
import org.junit.Test;
+import org.apache.kudu.Schema;
+import org.apache.kudu.test.KuduTestHarness;
+
public class TestRowErrors {
private static final Schema basicSchema = getBasicSchema();
diff --git a/java/kudu-client/src/test/java/org/apache/kudu/client/TestRowResult.java b/java/kudu-client/src/test/java/org/apache/kudu/client/TestRowResult.java
index 7b367e2..fc8375b 100644
--- a/java/kudu-client/src/test/java/org/apache/kudu/client/TestRowResult.java
+++ b/java/kudu-client/src/test/java/org/apache/kudu/client/TestRowResult.java
@@ -28,13 +28,13 @@ import java.math.BigDecimal;
import java.nio.ByteBuffer;
import java.sql.Timestamp;
-import org.apache.kudu.Schema;
-import org.apache.kudu.test.KuduTestHarness;
import org.junit.Before;
import org.junit.Rule;
import org.junit.Test;
+import org.apache.kudu.Schema;
import org.apache.kudu.Type;
+import org.apache.kudu.test.KuduTestHarness;
public class TestRowResult {
diff --git a/java/kudu-client/src/test/java/org/apache/kudu/client/TestRpcTraces.java b/java/kudu-client/src/test/java/org/apache/kudu/client/TestRpcTraces.java
index dad90dd..c125d47 100644
--- a/java/kudu-client/src/test/java/org/apache/kudu/client/TestRpcTraces.java
+++ b/java/kudu-client/src/test/java/org/apache/kudu/client/TestRpcTraces.java
@@ -26,11 +26,11 @@ import java.net.InetAddress;
import java.util.ArrayList;
import java.util.List;
-import org.apache.kudu.client.RpcTraceFrame.Action;
-import org.apache.kudu.client.RpcTraceFrame.RpcTraceFrameBuilder;
import org.junit.Rule;
import org.junit.Test;
+import org.apache.kudu.client.RpcTraceFrame.Action;
+import org.apache.kudu.client.RpcTraceFrame.RpcTraceFrameBuilder;
import org.apache.kudu.test.junit.RetryRule;
public class TestRpcTraces {
diff --git a/java/kudu-client/src/test/java/org/apache/kudu/client/TestScanPredicate.java b/java/kudu-client/src/test/java/org/apache/kudu/client/TestScanPredicate.java
index 1ec209d..5426125 100644
--- a/java/kudu-client/src/test/java/org/apache/kudu/client/TestScanPredicate.java
+++ b/java/kudu-client/src/test/java/org/apache/kudu/client/TestScanPredicate.java
@@ -25,7 +25,6 @@ import java.util.TreeSet;
import com.google.common.collect.ImmutableList;
import com.google.common.collect.ImmutableSortedSet;
-import org.apache.kudu.test.KuduTestHarness;
import org.junit.Assert;
import org.junit.Before;
import org.junit.Rule;
@@ -35,6 +34,7 @@ import org.apache.kudu.ColumnSchema;
import org.apache.kudu.Schema;
import org.apache.kudu.Type;
import org.apache.kudu.client.KuduPredicate.ComparisonOp;
+import org.apache.kudu.test.KuduTestHarness;
import org.apache.kudu.util.DecimalUtil;
public class TestScanPredicate {
diff --git a/java/kudu-client/src/test/java/org/apache/kudu/client/TestScanToken.java b/java/kudu-client/src/test/java/org/apache/kudu/client/TestScanToken.java
index c203e95..e8057c0 100644
--- a/java/kudu-client/src/test/java/org/apache/kudu/client/TestScanToken.java
+++ b/java/kudu-client/src/test/java/org/apache/kudu/client/TestScanToken.java
@@ -17,22 +17,6 @@
package org.apache.kudu.client;
-import com.google.common.collect.ImmutableList;
-import org.apache.kudu.ColumnSchema;
-import org.apache.kudu.client.KuduScanner;
-import org.apache.kudu.Schema;
-import org.apache.kudu.Type;
-import org.apache.kudu.test.KuduTestHarness;
-import org.junit.Before;
-import org.junit.Rule;
-import org.junit.Test;
-import org.slf4j.Logger;
-import org.slf4j.LoggerFactory;
-
-import java.io.IOException;
-import java.util.ArrayList;
-import java.util.List;
-
import static org.apache.kudu.test.ClientTestUtil.countRowsInScan;
import static org.apache.kudu.test.ClientTestUtil.countScanTokenRows;
import static org.apache.kudu.test.ClientTestUtil.createBasicSchemaInsert;
@@ -44,6 +28,22 @@ import static org.junit.Assert.assertEquals;
import static org.junit.Assert.assertTrue;
import static org.junit.Assert.fail;
+import java.io.IOException;
+import java.util.ArrayList;
+import java.util.List;
+
+import com.google.common.collect.ImmutableList;
+import org.junit.Before;
+import org.junit.Rule;
+import org.junit.Test;
+import org.slf4j.Logger;
+import org.slf4j.LoggerFactory;
+
+import org.apache.kudu.ColumnSchema;
+import org.apache.kudu.Schema;
+import org.apache.kudu.Type;
+import org.apache.kudu.test.KuduTestHarness;
+
public class TestScanToken {
private static final Logger LOG = LoggerFactory.getLogger(TestKuduClient.class);
diff --git a/java/kudu-client/src/test/java/org/apache/kudu/client/TestScannerMultiTablet.java b/java/kudu-client/src/test/java/org/apache/kudu/client/TestScannerMultiTablet.java
index 4ddcdd5..9b9b648 100644
--- a/java/kudu-client/src/test/java/org/apache/kudu/client/TestScannerMultiTablet.java
+++ b/java/kudu-client/src/test/java/org/apache/kudu/client/TestScannerMultiTablet.java
@@ -17,8 +17,8 @@
package org.apache.kudu.client;
import static org.apache.kudu.Type.STRING;
-import static org.apache.kudu.test.KuduTestHarness.DEFAULT_SLEEP;
import static org.apache.kudu.test.ClientTestUtil.countRowsInScan;
+import static org.apache.kudu.test.KuduTestHarness.DEFAULT_SLEEP;
import static org.junit.Assert.assertEquals;
import static org.junit.Assert.assertNotEquals;
import static org.junit.Assert.assertNull;
@@ -29,16 +29,15 @@ import java.util.ArrayList;
import com.google.common.collect.ImmutableList;
import com.google.common.collect.Lists;
import com.stumbleupon.async.Deferred;
-
-import org.apache.kudu.test.KuduTestHarness;
import org.junit.Before;
import org.junit.Rule;
import org.junit.Test;
-import org.apache.kudu.client.Client.ScanTokenPB;
import org.apache.kudu.ColumnSchema;
import org.apache.kudu.Common;
import org.apache.kudu.Schema;
+import org.apache.kudu.client.Client.ScanTokenPB;
+import org.apache.kudu.test.KuduTestHarness;
public class TestScannerMultiTablet {
// Generate a unique table name
diff --git a/java/kudu-client/src/test/java/org/apache/kudu/client/TestSecurity.java b/java/kudu-client/src/test/java/org/apache/kudu/client/TestSecurity.java
index 77f947a..3c4de25 100644
--- a/java/kudu-client/src/test/java/org/apache/kudu/client/TestSecurity.java
+++ b/java/kudu-client/src/test/java/org/apache/kudu/client/TestSecurity.java
@@ -13,10 +13,10 @@
*/
package org.apache.kudu.client;
-import static org.apache.kudu.test.junit.AssertHelpers.assertEventuallyTrue;
import static org.apache.kudu.test.ClientTestUtil.createBasicSchemaInsert;
import static org.apache.kudu.test.ClientTestUtil.getBasicCreateTableOptions;
import static org.apache.kudu.test.ClientTestUtil.getBasicSchema;
+import static org.apache.kudu.test.junit.AssertHelpers.assertEventuallyTrue;
import static org.junit.Assert.assertNotNull;
import java.io.Closeable;
@@ -26,9 +26,18 @@ import java.security.PrivilegedExceptionAction;
import java.util.List;
import java.util.Set;
import java.util.concurrent.TimeUnit;
-
import javax.security.auth.Subject;
+import com.google.common.base.Stopwatch;
+import com.google.common.collect.ImmutableSet;
+import com.stumbleupon.async.Deferred;
+import org.hamcrest.CoreMatchers;
+import org.junit.After;
+import org.junit.Assert;
+import org.junit.Before;
+import org.junit.Rule;
+import org.junit.Test;
+
import org.apache.kudu.client.Client.AuthenticationCredentialsPB;
import org.apache.kudu.master.Master.ConnectToMasterResponsePB;
import org.apache.kudu.test.CapturingLogAppender;
@@ -36,19 +45,9 @@ import org.apache.kudu.test.cluster.FakeDNS;
import org.apache.kudu.test.cluster.MiniKuduCluster;
import org.apache.kudu.test.cluster.MiniKuduCluster.MiniKuduClusterBuilder;
import org.apache.kudu.test.junit.AssertHelpers;
-import org.apache.kudu.test.junit.RetryRule;
import org.apache.kudu.test.junit.AssertHelpers.BooleanExpression;
+import org.apache.kudu.test.junit.RetryRule;
import org.apache.kudu.util.SecurityUtil;
-import org.hamcrest.CoreMatchers;
-import org.junit.After;
-import org.junit.Assert;
-import org.junit.Before;
-import org.junit.Rule;
-import org.junit.Test;
-
-import com.google.common.base.Stopwatch;
-import com.google.common.collect.ImmutableSet;
-import com.stumbleupon.async.Deferred;
public class TestSecurity {
private static final String TABLE_NAME = "TestSecurity-table";
diff --git a/java/kudu-client/src/test/java/org/apache/kudu/client/TestSecurityContextRealUser.java b/java/kudu-client/src/test/java/org/apache/kudu/client/TestSecurityContextRealUser.java
index d4dc753..7910c31 100644
--- a/java/kudu-client/src/test/java/org/apache/kudu/client/TestSecurityContextRealUser.java
+++ b/java/kudu-client/src/test/java/org/apache/kudu/client/TestSecurityContextRealUser.java
@@ -17,20 +17,21 @@
package org.apache.kudu.client;
-import org.apache.kudu.test.cluster.MiniKuduCluster.MiniKuduClusterBuilder;
-import org.apache.kudu.test.KuduTestHarness;
-import org.junit.Before;
-import org.junit.Rule;
-import org.junit.Test;
-
-import java.util.ArrayList;
-
import static org.apache.kudu.test.ClientTestUtil.getBasicSchema;
import static org.apache.kudu.test.ClientTestUtil.scanTableToStrings;
import static org.junit.Assert.assertEquals;
import static org.junit.Assert.assertTrue;
import static org.junit.Assert.fail;
+import java.util.ArrayList;
+
+import org.junit.Before;
+import org.junit.Rule;
+import org.junit.Test;
+
+import org.apache.kudu.test.KuduTestHarness;
+import org.apache.kudu.test.cluster.MiniKuduCluster.MiniKuduClusterBuilder;
+
/**
* Tests that the 'real user' field of the security context is used for
* SASL PLAIN negotiations, and is imported from the SecurityCredentialsPB.
diff --git a/java/kudu-client/src/test/java/org/apache/kudu/client/TestSplitKeyRange.java b/java/kudu-client/src/test/java/org/apache/kudu/client/TestSplitKeyRange.java
index cb839aa..62f0ec9 100644
--- a/java/kudu-client/src/test/java/org/apache/kudu/client/TestSplitKeyRange.java
+++ b/java/kudu-client/src/test/java/org/apache/kudu/client/TestSplitKeyRange.java
@@ -22,13 +22,14 @@ import static org.apache.kudu.test.KuduTestHarness.DEFAULT_SLEEP;
import static org.junit.Assert.assertEquals;
import static org.junit.Assert.assertTrue;
-import org.apache.kudu.Schema;
-import org.apache.kudu.test.KuduTestHarness;
-import org.apache.kudu.test.KuduTestHarness.TabletServerConfig;
+import java.util.List;
+
import org.junit.Rule;
import org.junit.Test;
-import java.util.List;
+import org.apache.kudu.Schema;
+import org.apache.kudu.test.KuduTestHarness;
+import org.apache.kudu.test.KuduTestHarness.TabletServerConfig;
public class TestSplitKeyRange {
// Generate a unique table name
diff --git a/java/kudu-client/src/test/java/org/apache/kudu/client/TestStatistics.java b/java/kudu-client/src/test/java/org/apache/kudu/client/TestStatistics.java
index 1dfa319..08e13b2 100644
--- a/java/kudu-client/src/test/java/org/apache/kudu/client/TestStatistics.java
+++ b/java/kudu-client/src/test/java/org/apache/kudu/client/TestStatistics.java
@@ -21,15 +21,15 @@ import static org.apache.kudu.test.ClientTestUtil.getBasicCreateTableOptions;
import static org.apache.kudu.test.ClientTestUtil.getBasicSchema;
import static org.junit.Assert.assertEquals;
-import org.apache.kudu.test.KuduTestHarness;
+import java.util.ArrayList;
+import java.util.List;
+
import org.junit.Before;
import org.junit.Rule;
import org.junit.Test;
import org.apache.kudu.client.Statistics.Statistic;
-
-import java.util.ArrayList;
-import java.util.List;
+import org.apache.kudu.test.KuduTestHarness;
public class TestStatistics {
diff --git a/java/kudu-client/src/test/java/org/apache/kudu/client/TestTableLocationsCache.java b/java/kudu-client/src/test/java/org/apache/kudu/client/TestTableLocationsCache.java
index c231bbd..ff3213d 100644
--- a/java/kudu-client/src/test/java/org/apache/kudu/client/TestTableLocationsCache.java
+++ b/java/kudu-client/src/test/java/org/apache/kudu/client/TestTableLocationsCache.java
@@ -16,7 +16,7 @@
// under the License.
package org.apache.kudu.client;
-import static org.junit.Assert.*;
+import static org.junit.Assert.assertEquals;
import java.util.List;
diff --git a/java/kudu-client/src/test/java/org/apache/kudu/client/TestTimeouts.java b/java/kudu-client/src/test/java/org/apache/kudu/client/TestTimeouts.java
index 4af6ef6..cfad30c 100644
--- a/java/kudu-client/src/test/java/org/apache/kudu/client/TestTimeouts.java
+++ b/java/kudu-client/src/test/java/org/apache/kudu/client/TestTimeouts.java
@@ -23,13 +23,14 @@ import static org.junit.Assert.assertFalse;
import static org.junit.Assert.assertTrue;
import static org.junit.Assert.fail;
-import org.apache.kudu.test.KuduTestHarness;
-import org.apache.kudu.test.KuduTestHarness.MasterServerConfig;
-import org.apache.kudu.test.KuduTestHarness.TabletServerConfig;
import org.junit.Ignore;
import org.junit.Rule;
import org.junit.Test;
+import org.apache.kudu.test.KuduTestHarness;
+import org.apache.kudu.test.KuduTestHarness.MasterServerConfig;
+import org.apache.kudu.test.KuduTestHarness.TabletServerConfig;
+
public class TestTimeouts {
private static final String TABLE_NAME =