createProxy(
diff --git a/atlasdb-service/src/main/java/com/palantir/atlasdb/api/AtlasDbService.java b/atlasdb-service/src/main/java/com/palantir/atlasdb/api/AtlasDbService.java
index 79502c5a4a3..643baf7f16a 100644
--- a/atlasdb-service/src/main/java/com/palantir/atlasdb/api/AtlasDbService.java
+++ b/atlasdb-service/src/main/java/com/palantir/atlasdb/api/AtlasDbService.java
@@ -15,8 +15,9 @@
*/
package com.palantir.atlasdb.api;
+import com.palantir.atlasdb.table.description.TableMetadata;
+import com.palantir.common.annotation.Idempotent;
import java.util.Set;
-
import javax.ws.rs.Consumes;
import javax.ws.rs.GET;
import javax.ws.rs.POST;
@@ -25,9 +26,6 @@
import javax.ws.rs.Produces;
import javax.ws.rs.core.MediaType;
-import com.palantir.atlasdb.table.description.TableMetadata;
-import com.palantir.common.annotation.Idempotent;
-
@Path("/atlasdb")
public interface AtlasDbService {
diff --git a/atlasdb-service/src/main/java/com/palantir/atlasdb/api/RangeToken.java b/atlasdb-service/src/main/java/com/palantir/atlasdb/api/RangeToken.java
index 0efdbb57c84..9d997914d44 100644
--- a/atlasdb-service/src/main/java/com/palantir/atlasdb/api/RangeToken.java
+++ b/atlasdb-service/src/main/java/com/palantir/atlasdb/api/RangeToken.java
@@ -15,9 +15,8 @@
*/
package com.palantir.atlasdb.api;
-import javax.annotation.Nullable;
-
import com.palantir.logsafe.Preconditions;
+import javax.annotation.Nullable;
/*
*
diff --git a/atlasdb-service/src/main/java/com/palantir/atlasdb/api/TableCellVal.java b/atlasdb-service/src/main/java/com/palantir/atlasdb/api/TableCellVal.java
index 84a18d50570..4f6052be6b2 100644
--- a/atlasdb-service/src/main/java/com/palantir/atlasdb/api/TableCellVal.java
+++ b/atlasdb-service/src/main/java/com/palantir/atlasdb/api/TableCellVal.java
@@ -15,10 +15,9 @@
*/
package com.palantir.atlasdb.api;
-import java.util.Map;
-
import com.palantir.atlasdb.keyvalue.api.Cell;
import com.palantir.logsafe.Preconditions;
+import java.util.Map;
/**
* For tables with named columns,
diff --git a/atlasdb-service/src/main/java/com/palantir/atlasdb/api/TableRange.java b/atlasdb-service/src/main/java/com/palantir/atlasdb/api/TableRange.java
index aff88e16d92..c55287e5ca6 100644
--- a/atlasdb-service/src/main/java/com/palantir/atlasdb/api/TableRange.java
+++ b/atlasdb-service/src/main/java/com/palantir/atlasdb/api/TableRange.java
@@ -15,9 +15,8 @@
*/
package com.palantir.atlasdb.api;
-import java.util.Arrays;
-
import com.palantir.logsafe.Preconditions;
+import java.util.Arrays;
/**
*
diff --git a/atlasdb-service/src/main/java/com/palantir/atlasdb/api/TableRowResult.java b/atlasdb-service/src/main/java/com/palantir/atlasdb/api/TableRowResult.java
index 92d6a049b00..4dc5f30452b 100644
--- a/atlasdb-service/src/main/java/com/palantir/atlasdb/api/TableRowResult.java
+++ b/atlasdb-service/src/main/java/com/palantir/atlasdb/api/TableRowResult.java
@@ -15,10 +15,9 @@
*/
package com.palantir.atlasdb.api;
-import java.util.Collection;
-
import com.palantir.atlasdb.keyvalue.api.RowResult;
import com.palantir.logsafe.Preconditions;
+import java.util.Collection;
/**
* For tables with named columns,
diff --git a/atlasdb-service/src/main/java/com/palantir/atlasdb/impl/AtlasDbServiceImpl.java b/atlasdb-service/src/main/java/com/palantir/atlasdb/impl/AtlasDbServiceImpl.java
index 47c376f3ebd..323d1135038 100644
--- a/atlasdb-service/src/main/java/com/palantir/atlasdb/impl/AtlasDbServiceImpl.java
+++ b/atlasdb-service/src/main/java/com/palantir/atlasdb/impl/AtlasDbServiceImpl.java
@@ -15,16 +15,6 @@
*/
package com.palantir.atlasdb.impl;
-import java.util.Collection;
-import java.util.List;
-import java.util.Map;
-import java.util.Set;
-import java.util.UUID;
-import java.util.concurrent.TimeUnit;
-import java.util.stream.Collectors;
-
-import javax.inject.Inject;
-
import com.google.common.cache.Cache;
import com.google.common.cache.CacheBuilder;
import com.google.common.collect.ImmutableSet;
@@ -54,6 +44,14 @@
import com.palantir.common.base.BatchingVisitable;
import com.palantir.common.base.BatchingVisitables;
import com.palantir.logsafe.Preconditions;
+import java.util.Collection;
+import java.util.List;
+import java.util.Map;
+import java.util.Set;
+import java.util.UUID;
+import java.util.concurrent.TimeUnit;
+import java.util.stream.Collectors;
+import javax.inject.Inject;
public class AtlasDbServiceImpl implements AtlasDbService {
private static final TableMetadata RAW_METADATA = TableMetadata.builder()
diff --git a/atlasdb-service/src/main/java/com/palantir/atlasdb/impl/TableMetadataCache.java b/atlasdb-service/src/main/java/com/palantir/atlasdb/impl/TableMetadataCache.java
index 84364390256..e489244b81d 100644
--- a/atlasdb-service/src/main/java/com/palantir/atlasdb/impl/TableMetadataCache.java
+++ b/atlasdb-service/src/main/java/com/palantir/atlasdb/impl/TableMetadataCache.java
@@ -15,17 +15,15 @@
*/
package com.palantir.atlasdb.impl;
-import java.util.concurrent.TimeUnit;
-
-import javax.annotation.CheckForNull;
-import javax.inject.Inject;
-
import com.google.common.cache.CacheBuilder;
import com.google.common.cache.CacheLoader;
import com.google.common.cache.LoadingCache;
import com.palantir.atlasdb.keyvalue.api.KeyValueService;
import com.palantir.atlasdb.keyvalue.api.TableReference;
import com.palantir.atlasdb.table.description.TableMetadata;
+import java.util.concurrent.TimeUnit;
+import javax.annotation.CheckForNull;
+import javax.inject.Inject;
public class TableMetadataCache {
private final LoadingCache cache;
diff --git a/atlasdb-service/src/main/java/com/palantir/atlasdb/jackson/AtlasDeserializers.java b/atlasdb-service/src/main/java/com/palantir/atlasdb/jackson/AtlasDeserializers.java
index 7608818a63e..bed1014e1c0 100644
--- a/atlasdb-service/src/main/java/com/palantir/atlasdb/jackson/AtlasDeserializers.java
+++ b/atlasdb-service/src/main/java/com/palantir/atlasdb/jackson/AtlasDeserializers.java
@@ -15,12 +15,6 @@
*/
package com.palantir.atlasdb.jackson;
-import java.util.Collection;
-import java.util.Iterator;
-import java.util.List;
-import java.util.Map;
-import java.util.Map.Entry;
-
import com.fasterxml.jackson.databind.JsonNode;
import com.google.common.base.Function;
import com.google.common.base.Preconditions;
@@ -49,6 +43,11 @@
import com.palantir.atlasdb.table.description.NamedColumnDescription;
import com.palantir.atlasdb.table.description.TableMetadata;
import com.palantir.common.base.Throwables;
+import java.util.Collection;
+import java.util.Iterator;
+import java.util.List;
+import java.util.Map;
+import java.util.Map.Entry;
public final class AtlasDeserializers {
diff --git a/atlasdb-service/src/main/java/com/palantir/atlasdb/jackson/AtlasJacksonModule.java b/atlasdb-service/src/main/java/com/palantir/atlasdb/jackson/AtlasJacksonModule.java
index 9d164a59c28..0e807606095 100644
--- a/atlasdb-service/src/main/java/com/palantir/atlasdb/jackson/AtlasJacksonModule.java
+++ b/atlasdb-service/src/main/java/com/palantir/atlasdb/jackson/AtlasJacksonModule.java
@@ -15,8 +15,6 @@
*/
package com.palantir.atlasdb.jackson;
-import javax.inject.Inject;
-
import com.fasterxml.jackson.databind.Module;
import com.fasterxml.jackson.databind.module.SimpleModule;
import com.palantir.atlasdb.api.RangeToken;
@@ -27,6 +25,7 @@
import com.palantir.atlasdb.api.TableRowSelection;
import com.palantir.atlasdb.impl.TableMetadataCache;
import com.palantir.atlasdb.table.description.TableMetadata;
+import javax.inject.Inject;
public class AtlasJacksonModule {
private final TableMetadataCache cache;
diff --git a/atlasdb-service/src/main/java/com/palantir/atlasdb/jackson/AtlasSerializers.java b/atlasdb-service/src/main/java/com/palantir/atlasdb/jackson/AtlasSerializers.java
index 67b1e065866..a039f148e59 100644
--- a/atlasdb-service/src/main/java/com/palantir/atlasdb/jackson/AtlasSerializers.java
+++ b/atlasdb-service/src/main/java/com/palantir/atlasdb/jackson/AtlasSerializers.java
@@ -15,9 +15,6 @@
*/
package com.palantir.atlasdb.jackson;
-import java.io.IOException;
-import java.util.Set;
-
import com.fasterxml.jackson.core.JsonGenerator;
import com.google.common.io.BaseEncoding;
import com.google.common.primitives.UnsignedBytes;
@@ -33,6 +30,8 @@
import com.palantir.atlasdb.table.description.NameMetadataDescription;
import com.palantir.atlasdb.table.description.NamedColumnDescription;
import com.palantir.util.Pair;
+import java.io.IOException;
+import java.util.Set;
public final class AtlasSerializers {
diff --git a/atlasdb-service/src/main/java/com/palantir/atlasdb/jackson/RangeTokenDeserializer.java b/atlasdb-service/src/main/java/com/palantir/atlasdb/jackson/RangeTokenDeserializer.java
index 5ac68576bbe..74ba3218aa7 100644
--- a/atlasdb-service/src/main/java/com/palantir/atlasdb/jackson/RangeTokenDeserializer.java
+++ b/atlasdb-service/src/main/java/com/palantir/atlasdb/jackson/RangeTokenDeserializer.java
@@ -15,8 +15,6 @@
*/
package com.palantir.atlasdb.jackson;
-import java.io.IOException;
-
import com.fasterxml.jackson.core.JsonParser;
import com.fasterxml.jackson.core.JsonToken;
import com.fasterxml.jackson.databind.DeserializationContext;
@@ -25,6 +23,7 @@
import com.palantir.atlasdb.api.TableCell;
import com.palantir.atlasdb.api.TableRange;
import com.palantir.atlasdb.api.TableRowResult;
+import java.io.IOException;
public class RangeTokenDeserializer extends StdDeserializer {
private static final long serialVersionUID = 1L;
diff --git a/atlasdb-service/src/main/java/com/palantir/atlasdb/jackson/RangeTokenSerializer.java b/atlasdb-service/src/main/java/com/palantir/atlasdb/jackson/RangeTokenSerializer.java
index 460d3b51403..0d5f0381913 100644
--- a/atlasdb-service/src/main/java/com/palantir/atlasdb/jackson/RangeTokenSerializer.java
+++ b/atlasdb-service/src/main/java/com/palantir/atlasdb/jackson/RangeTokenSerializer.java
@@ -15,14 +15,12 @@
*/
package com.palantir.atlasdb.jackson;
-import java.io.IOException;
-
-import javax.inject.Inject;
-
import com.fasterxml.jackson.core.JsonGenerator;
import com.fasterxml.jackson.databind.SerializerProvider;
import com.fasterxml.jackson.databind.ser.std.StdSerializer;
import com.palantir.atlasdb.api.RangeToken;
+import java.io.IOException;
+import javax.inject.Inject;
public class RangeTokenSerializer extends StdSerializer {
private static final long serialVersionUID = 1L;
diff --git a/atlasdb-service/src/main/java/com/palantir/atlasdb/jackson/TableCellDeserializer.java b/atlasdb-service/src/main/java/com/palantir/atlasdb/jackson/TableCellDeserializer.java
index 20fae15fc3d..1f76b62fc2b 100644
--- a/atlasdb-service/src/main/java/com/palantir/atlasdb/jackson/TableCellDeserializer.java
+++ b/atlasdb-service/src/main/java/com/palantir/atlasdb/jackson/TableCellDeserializer.java
@@ -15,8 +15,6 @@
*/
package com.palantir.atlasdb.jackson;
-import java.io.IOException;
-
import com.fasterxml.jackson.core.JsonParser;
import com.fasterxml.jackson.databind.DeserializationContext;
import com.fasterxml.jackson.databind.JsonNode;
@@ -25,6 +23,7 @@
import com.palantir.atlasdb.impl.TableMetadataCache;
import com.palantir.atlasdb.keyvalue.api.Cell;
import com.palantir.atlasdb.table.description.TableMetadata;
+import java.io.IOException;
public class TableCellDeserializer extends StdDeserializer {
private static final long serialVersionUID = 1L;
diff --git a/atlasdb-service/src/main/java/com/palantir/atlasdb/jackson/TableCellSerializer.java b/atlasdb-service/src/main/java/com/palantir/atlasdb/jackson/TableCellSerializer.java
index af0ad5307ae..943158770e5 100644
--- a/atlasdb-service/src/main/java/com/palantir/atlasdb/jackson/TableCellSerializer.java
+++ b/atlasdb-service/src/main/java/com/palantir/atlasdb/jackson/TableCellSerializer.java
@@ -15,8 +15,6 @@
*/
package com.palantir.atlasdb.jackson;
-import java.io.IOException;
-
import com.fasterxml.jackson.core.JsonGenerator;
import com.fasterxml.jackson.databind.SerializerProvider;
import com.fasterxml.jackson.databind.ser.std.StdSerializer;
@@ -29,6 +27,7 @@
import com.palantir.atlasdb.table.description.DynamicColumnDescription;
import com.palantir.atlasdb.table.description.NamedColumnDescription;
import com.palantir.atlasdb.table.description.TableMetadata;
+import java.io.IOException;
public class TableCellSerializer extends StdSerializer {
private static final long serialVersionUID = 1L;
diff --git a/atlasdb-service/src/main/java/com/palantir/atlasdb/jackson/TableCellValDeserializer.java b/atlasdb-service/src/main/java/com/palantir/atlasdb/jackson/TableCellValDeserializer.java
index 79f4a5b722b..d1968baf98e 100644
--- a/atlasdb-service/src/main/java/com/palantir/atlasdb/jackson/TableCellValDeserializer.java
+++ b/atlasdb-service/src/main/java/com/palantir/atlasdb/jackson/TableCellValDeserializer.java
@@ -15,9 +15,6 @@
*/
package com.palantir.atlasdb.jackson;
-import java.io.IOException;
-import java.util.Map;
-
import com.fasterxml.jackson.core.JsonParser;
import com.fasterxml.jackson.databind.DeserializationContext;
import com.fasterxml.jackson.databind.JsonNode;
@@ -26,6 +23,8 @@
import com.palantir.atlasdb.impl.TableMetadataCache;
import com.palantir.atlasdb.keyvalue.api.Cell;
import com.palantir.atlasdb.table.description.TableMetadata;
+import java.io.IOException;
+import java.util.Map;
public class TableCellValDeserializer extends StdDeserializer {
private static final long serialVersionUID = 1L;
diff --git a/atlasdb-service/src/main/java/com/palantir/atlasdb/jackson/TableCellValSerializer.java b/atlasdb-service/src/main/java/com/palantir/atlasdb/jackson/TableCellValSerializer.java
index f498f433f43..766b4165ab1 100644
--- a/atlasdb-service/src/main/java/com/palantir/atlasdb/jackson/TableCellValSerializer.java
+++ b/atlasdb-service/src/main/java/com/palantir/atlasdb/jackson/TableCellValSerializer.java
@@ -15,10 +15,6 @@
*/
package com.palantir.atlasdb.jackson;
-import java.io.IOException;
-import java.util.Map.Entry;
-import java.util.Set;
-
import com.fasterxml.jackson.core.JsonGenerator;
import com.fasterxml.jackson.databind.SerializerProvider;
import com.fasterxml.jackson.databind.ser.std.StdSerializer;
@@ -31,6 +27,9 @@
import com.palantir.atlasdb.table.description.DynamicColumnDescription;
import com.palantir.atlasdb.table.description.NamedColumnDescription;
import com.palantir.atlasdb.table.description.TableMetadata;
+import java.io.IOException;
+import java.util.Map.Entry;
+import java.util.Set;
public class TableCellValSerializer extends StdSerializer {
private static final long serialVersionUID = 1L;
diff --git a/atlasdb-service/src/main/java/com/palantir/atlasdb/jackson/TableMetadataDeserializer.java b/atlasdb-service/src/main/java/com/palantir/atlasdb/jackson/TableMetadataDeserializer.java
index dbe92dfedf0..459d6ff6f34 100644
--- a/atlasdb-service/src/main/java/com/palantir/atlasdb/jackson/TableMetadataDeserializer.java
+++ b/atlasdb-service/src/main/java/com/palantir/atlasdb/jackson/TableMetadataDeserializer.java
@@ -15,11 +15,6 @@
*/
package com.palantir.atlasdb.jackson;
-import java.io.IOException;
-import java.util.Collection;
-import java.util.List;
-import java.util.Optional;
-
import com.fasterxml.jackson.core.JsonParser;
import com.fasterxml.jackson.databind.DeserializationContext;
import com.fasterxml.jackson.databind.JsonNode;
@@ -42,6 +37,10 @@
import com.palantir.atlasdb.table.description.ValueType;
import com.palantir.atlasdb.transaction.api.ConflictHandler;
import com.palantir.common.persist.Persistable;
+import java.io.IOException;
+import java.util.Collection;
+import java.util.List;
+import java.util.Optional;
public class TableMetadataDeserializer extends StdDeserializer {
private static final long serialVersionUID = 1L;
diff --git a/atlasdb-service/src/main/java/com/palantir/atlasdb/jackson/TableMetadataSerializer.java b/atlasdb-service/src/main/java/com/palantir/atlasdb/jackson/TableMetadataSerializer.java
index 28e3e3245a1..6e40a84f007 100644
--- a/atlasdb-service/src/main/java/com/palantir/atlasdb/jackson/TableMetadataSerializer.java
+++ b/atlasdb-service/src/main/java/com/palantir/atlasdb/jackson/TableMetadataSerializer.java
@@ -15,8 +15,6 @@
*/
package com.palantir.atlasdb.jackson;
-import java.io.IOException;
-
import com.fasterxml.jackson.core.JsonGenerator;
import com.fasterxml.jackson.databind.SerializerProvider;
import com.fasterxml.jackson.databind.ser.std.StdSerializer;
@@ -33,6 +31,7 @@
import com.palantir.atlasdb.table.description.NameMetadataDescription;
import com.palantir.atlasdb.table.description.NamedColumnDescription;
import com.palantir.atlasdb.table.description.TableMetadata;
+import java.io.IOException;
public class TableMetadataSerializer extends StdSerializer {
private static final long serialVersionUID = 1L;
diff --git a/atlasdb-service/src/main/java/com/palantir/atlasdb/jackson/TableRangeDeserializer.java b/atlasdb-service/src/main/java/com/palantir/atlasdb/jackson/TableRangeDeserializer.java
index 9c77de430d1..a38d480670f 100644
--- a/atlasdb-service/src/main/java/com/palantir/atlasdb/jackson/TableRangeDeserializer.java
+++ b/atlasdb-service/src/main/java/com/palantir/atlasdb/jackson/TableRangeDeserializer.java
@@ -15,8 +15,6 @@
*/
package com.palantir.atlasdb.jackson;
-import java.io.IOException;
-
import com.fasterxml.jackson.core.JsonParser;
import com.fasterxml.jackson.databind.DeserializationContext;
import com.fasterxml.jackson.databind.JsonNode;
@@ -25,6 +23,7 @@
import com.palantir.atlasdb.impl.TableMetadataCache;
import com.palantir.atlasdb.keyvalue.api.RangeRequests;
import com.palantir.atlasdb.table.description.TableMetadata;
+import java.io.IOException;
public class TableRangeDeserializer extends StdDeserializer {
private static final long serialVersionUID = 1L;
diff --git a/atlasdb-service/src/main/java/com/palantir/atlasdb/jackson/TableRangeSerializer.java b/atlasdb-service/src/main/java/com/palantir/atlasdb/jackson/TableRangeSerializer.java
index 42b499015bd..7f9eddee614 100644
--- a/atlasdb-service/src/main/java/com/palantir/atlasdb/jackson/TableRangeSerializer.java
+++ b/atlasdb-service/src/main/java/com/palantir/atlasdb/jackson/TableRangeSerializer.java
@@ -15,8 +15,6 @@
*/
package com.palantir.atlasdb.jackson;
-import java.io.IOException;
-
import com.fasterxml.jackson.core.JsonGenerator;
import com.fasterxml.jackson.databind.SerializerProvider;
import com.fasterxml.jackson.databind.ser.std.StdSerializer;
@@ -24,6 +22,7 @@
import com.palantir.atlasdb.api.TableRange;
import com.palantir.atlasdb.impl.TableMetadataCache;
import com.palantir.atlasdb.table.description.TableMetadata;
+import java.io.IOException;
public class TableRangeSerializer extends StdSerializer {
private static final long serialVersionUID = 1L;
diff --git a/atlasdb-service/src/main/java/com/palantir/atlasdb/jackson/TableRowResultDeserializer.java b/atlasdb-service/src/main/java/com/palantir/atlasdb/jackson/TableRowResultDeserializer.java
index 431618cc4a0..9ba606ca168 100644
--- a/atlasdb-service/src/main/java/com/palantir/atlasdb/jackson/TableRowResultDeserializer.java
+++ b/atlasdb-service/src/main/java/com/palantir/atlasdb/jackson/TableRowResultDeserializer.java
@@ -15,10 +15,6 @@
*/
package com.palantir.atlasdb.jackson;
-import java.io.IOException;
-import java.nio.charset.StandardCharsets;
-import java.util.Collection;
-
import com.fasterxml.jackson.core.JsonParser;
import com.fasterxml.jackson.databind.DeserializationContext;
import com.fasterxml.jackson.databind.JsonNode;
@@ -31,6 +27,9 @@
import com.palantir.atlasdb.keyvalue.api.RowResult;
import com.palantir.atlasdb.table.description.NamedColumnDescription;
import com.palantir.atlasdb.table.description.TableMetadata;
+import java.io.IOException;
+import java.nio.charset.StandardCharsets;
+import java.util.Collection;
public class TableRowResultDeserializer extends StdDeserializer {
private static final long serialVersionUID = 1L;
diff --git a/atlasdb-service/src/main/java/com/palantir/atlasdb/jackson/TableRowResultSerializer.java b/atlasdb-service/src/main/java/com/palantir/atlasdb/jackson/TableRowResultSerializer.java
index 59662dc94c5..1b2a94ea90e 100644
--- a/atlasdb-service/src/main/java/com/palantir/atlasdb/jackson/TableRowResultSerializer.java
+++ b/atlasdb-service/src/main/java/com/palantir/atlasdb/jackson/TableRowResultSerializer.java
@@ -15,11 +15,6 @@
*/
package com.palantir.atlasdb.jackson;
-import java.io.IOException;
-import java.util.Map.Entry;
-import java.util.Set;
-import java.util.SortedMap;
-
import com.fasterxml.jackson.core.JsonGenerator;
import com.fasterxml.jackson.databind.SerializerProvider;
import com.fasterxml.jackson.databind.ser.std.StdSerializer;
@@ -32,6 +27,10 @@
import com.palantir.atlasdb.table.description.DynamicColumnDescription;
import com.palantir.atlasdb.table.description.NamedColumnDescription;
import com.palantir.atlasdb.table.description.TableMetadata;
+import java.io.IOException;
+import java.util.Map.Entry;
+import java.util.Set;
+import java.util.SortedMap;
public class TableRowResultSerializer extends StdSerializer {
private static final long serialVersionUID = 1L;
diff --git a/atlasdb-service/src/main/java/com/palantir/atlasdb/jackson/TableRowSelectionDeserializer.java b/atlasdb-service/src/main/java/com/palantir/atlasdb/jackson/TableRowSelectionDeserializer.java
index 5658992a327..280d05a79e9 100644
--- a/atlasdb-service/src/main/java/com/palantir/atlasdb/jackson/TableRowSelectionDeserializer.java
+++ b/atlasdb-service/src/main/java/com/palantir/atlasdb/jackson/TableRowSelectionDeserializer.java
@@ -15,8 +15,6 @@
*/
package com.palantir.atlasdb.jackson;
-import java.io.IOException;
-
import com.fasterxml.jackson.core.JsonParser;
import com.fasterxml.jackson.databind.DeserializationContext;
import com.fasterxml.jackson.databind.JsonNode;
@@ -26,6 +24,7 @@
import com.palantir.atlasdb.impl.TableMetadataCache;
import com.palantir.atlasdb.keyvalue.api.ColumnSelection;
import com.palantir.atlasdb.table.description.TableMetadata;
+import java.io.IOException;
public class TableRowSelectionDeserializer extends StdDeserializer {
private static final long serialVersionUID = 1L;
diff --git a/atlasdb-service/src/main/java/com/palantir/atlasdb/jackson/TableRowSelectionSerializer.java b/atlasdb-service/src/main/java/com/palantir/atlasdb/jackson/TableRowSelectionSerializer.java
index b34b9776c3a..56e4e5441b9 100644
--- a/atlasdb-service/src/main/java/com/palantir/atlasdb/jackson/TableRowSelectionSerializer.java
+++ b/atlasdb-service/src/main/java/com/palantir/atlasdb/jackson/TableRowSelectionSerializer.java
@@ -15,8 +15,6 @@
*/
package com.palantir.atlasdb.jackson;
-import java.io.IOException;
-
import com.fasterxml.jackson.core.JsonGenerator;
import com.fasterxml.jackson.databind.SerializerProvider;
import com.fasterxml.jackson.databind.ser.std.StdSerializer;
@@ -24,6 +22,7 @@
import com.palantir.atlasdb.api.TableRowSelection;
import com.palantir.atlasdb.impl.TableMetadataCache;
import com.palantir.atlasdb.table.description.TableMetadata;
+import java.io.IOException;
public class TableRowSelectionSerializer extends StdSerializer {
private static final long serialVersionUID = 1L;
diff --git a/atlasdb-service/src/main/java/com/palantir/atlasdb/proto/fork/ForkedJsonFormat.java b/atlasdb-service/src/main/java/com/palantir/atlasdb/proto/fork/ForkedJsonFormat.java
index 5dc91d5a481..136ac0f75fb 100644
--- a/atlasdb-service/src/main/java/com/palantir/atlasdb/proto/fork/ForkedJsonFormat.java
+++ b/atlasdb-service/src/main/java/com/palantir/atlasdb/proto/fork/ForkedJsonFormat.java
@@ -15,18 +15,6 @@
*/
package com.palantir.atlasdb.proto.fork;
-import java.io.IOException;
-import java.math.BigInteger;
-import java.nio.CharBuffer;
-import java.text.CharacterIterator;
-import java.text.StringCharacterIterator;
-import java.util.Iterator;
-import java.util.List;
-import java.util.Locale;
-import java.util.Map;
-import java.util.regex.Matcher;
-import java.util.regex.Pattern;
-
import com.google.common.io.BaseEncoding;
import com.google.protobuf.ByteString;
import com.google.protobuf.Descriptors.Descriptor;
@@ -39,6 +27,17 @@
import com.google.protobuf.UnknownFieldSet;
import com.palantir.logsafe.exceptions.SafeIllegalArgumentException;
import com.palantir.logsafe.exceptions.SafeRuntimeException;
+import java.io.IOException;
+import java.math.BigInteger;
+import java.nio.CharBuffer;
+import java.text.CharacterIterator;
+import java.text.StringCharacterIterator;
+import java.util.Iterator;
+import java.util.List;
+import java.util.Locale;
+import java.util.Map;
+import java.util.regex.Matcher;
+import java.util.regex.Pattern;
/**
* Forked version of com.googlecode.protobuf.format.JsonFormat from protobuf-java-format-1.2
diff --git a/atlasdb-service/src/test/java/com/palantir/atlasdb/impl/AtlasDbServiceImplTest.java b/atlasdb-service/src/test/java/com/palantir/atlasdb/impl/AtlasDbServiceImplTest.java
index f9cb014dba9..0ee64ec83c3 100644
--- a/atlasdb-service/src/test/java/com/palantir/atlasdb/impl/AtlasDbServiceImplTest.java
+++ b/atlasdb-service/src/test/java/com/palantir/atlasdb/impl/AtlasDbServiceImplTest.java
@@ -19,12 +19,11 @@
import static org.mockito.Mockito.mock;
import static org.mockito.Mockito.verify;
-import org.junit.Before;
-import org.junit.Test;
-
import com.palantir.atlasdb.keyvalue.api.KeyValueService;
import com.palantir.atlasdb.keyvalue.api.TableReference;
import com.palantir.atlasdb.transaction.api.TransactionManager;
+import org.junit.Before;
+import org.junit.Test;
public class AtlasDbServiceImplTest {
private KeyValueService kvs;
diff --git a/atlasdb-service/src/test/java/com/palantir/atlasdb/jackson/AtlasDeserializersTest.java b/atlasdb-service/src/test/java/com/palantir/atlasdb/jackson/AtlasDeserializersTest.java
index e5c61df1ff4..215db07f357 100644
--- a/atlasdb-service/src/test/java/com/palantir/atlasdb/jackson/AtlasDeserializersTest.java
+++ b/atlasdb-service/src/test/java/com/palantir/atlasdb/jackson/AtlasDeserializersTest.java
@@ -15,9 +15,6 @@
*/
package com.palantir.atlasdb.jackson;
-import org.junit.Assert;
-import org.junit.Test;
-
import com.fasterxml.jackson.databind.JsonNode;
import com.fasterxml.jackson.databind.ObjectMapper;
import com.google.common.collect.ImmutableList;
@@ -25,6 +22,8 @@
import com.palantir.atlasdb.table.description.NameComponentDescription;
import com.palantir.atlasdb.table.description.NameMetadataDescription;
import com.palantir.atlasdb.table.description.ValueType;
+import org.junit.Assert;
+import org.junit.Test;
public class AtlasDeserializersTest {
private static final NameMetadataDescription NAME_METADATA_DESCRIPTION = NameMetadataDescription.create(
diff --git a/atlasdb-tests-shared/src/main/java/com/palantir/atlasdb/ComparingTimestampCache.java b/atlasdb-tests-shared/src/main/java/com/palantir/atlasdb/ComparingTimestampCache.java
index 1cc602d8a0e..501969b6a42 100644
--- a/atlasdb-tests-shared/src/main/java/com/palantir/atlasdb/ComparingTimestampCache.java
+++ b/atlasdb-tests-shared/src/main/java/com/palantir/atlasdb/ComparingTimestampCache.java
@@ -16,11 +16,6 @@
package com.palantir.atlasdb;
-import java.util.Objects;
-import java.util.stream.Stream;
-
-import javax.annotation.Nullable;
-
import com.palantir.atlasdb.cache.DefaultTimestampCache;
import com.palantir.atlasdb.cache.OffHeapTimestampCache;
import com.palantir.atlasdb.cache.TimestampCache;
@@ -28,6 +23,9 @@
import com.palantir.atlasdb.util.MetricsManager;
import com.palantir.logsafe.Preconditions;
import com.palantir.logsafe.SafeArg;
+import java.util.Objects;
+import java.util.stream.Stream;
+import javax.annotation.Nullable;
public final class ComparingTimestampCache implements TimestampCache {
private final TimestampCache first;
diff --git a/atlasdb-tests-shared/src/main/java/com/palantir/atlasdb/keyvalue/impl/AbstractGetCandidateCellsForSweepingTest.java b/atlasdb-tests-shared/src/main/java/com/palantir/atlasdb/keyvalue/impl/AbstractGetCandidateCellsForSweepingTest.java
index f71f9f81d98..681058b43d1 100644
--- a/atlasdb-tests-shared/src/main/java/com/palantir/atlasdb/keyvalue/impl/AbstractGetCandidateCellsForSweepingTest.java
+++ b/atlasdb-tests-shared/src/main/java/com/palantir/atlasdb/keyvalue/impl/AbstractGetCandidateCellsForSweepingTest.java
@@ -18,13 +18,6 @@
import static org.assertj.core.api.Assertions.assertThat;
import static org.junit.Assert.assertEquals;
-import java.util.List;
-import java.util.Map;
-import java.util.stream.Collectors;
-
-import org.junit.Before;
-import org.junit.Test;
-
import com.google.common.collect.ImmutableList;
import com.google.common.collect.Iterables;
import com.google.common.collect.Iterators;
@@ -41,6 +34,11 @@
import com.palantir.atlasdb.keyvalue.api.KeyValueService;
import com.palantir.atlasdb.keyvalue.api.TableReference;
import com.palantir.common.base.ClosableIterator;
+import java.util.List;
+import java.util.Map;
+import java.util.stream.Collectors;
+import org.junit.Before;
+import org.junit.Test;
public abstract class AbstractGetCandidateCellsForSweepingTest {
private final KvsManager kvsManager;
diff --git a/atlasdb-tests-shared/src/main/java/com/palantir/atlasdb/keyvalue/impl/AbstractKeyValueServiceTest.java b/atlasdb-tests-shared/src/main/java/com/palantir/atlasdb/keyvalue/impl/AbstractKeyValueServiceTest.java
index 716ec6d2d58..8199a603803 100644
--- a/atlasdb-tests-shared/src/main/java/com/palantir/atlasdb/keyvalue/impl/AbstractKeyValueServiceTest.java
+++ b/atlasdb-tests-shared/src/main/java/com/palantir/atlasdb/keyvalue/impl/AbstractKeyValueServiceTest.java
@@ -27,30 +27,6 @@
import static org.junit.Assert.assertThat;
import static org.junit.Assert.assertTrue;
-import java.nio.ByteBuffer;
-import java.nio.charset.StandardCharsets;
-import java.util.ArrayList;
-import java.util.Arrays;
-import java.util.Collection;
-import java.util.Collections;
-import java.util.HashMap;
-import java.util.Iterator;
-import java.util.List;
-import java.util.Map;
-import java.util.Map.Entry;
-import java.util.Set;
-import java.util.SortedMap;
-import java.util.function.Function;
-import java.util.function.UnaryOperator;
-import java.util.stream.Collectors;
-
-import org.apache.commons.lang3.ArrayUtils;
-import org.junit.After;
-import org.junit.Assert;
-import org.junit.Assume;
-import org.junit.Before;
-import org.junit.Test;
-
import com.google.common.collect.ImmutableList;
import com.google.common.collect.ImmutableMap;
import com.google.common.collect.ImmutableMultimap;
@@ -91,6 +67,28 @@
import com.palantir.atlasdb.table.description.ValueType;
import com.palantir.common.base.ClosableIterator;
import com.palantir.common.exception.AtlasDbDependencyException;
+import java.nio.ByteBuffer;
+import java.nio.charset.StandardCharsets;
+import java.util.ArrayList;
+import java.util.Arrays;
+import java.util.Collection;
+import java.util.Collections;
+import java.util.HashMap;
+import java.util.Iterator;
+import java.util.List;
+import java.util.Map;
+import java.util.Map.Entry;
+import java.util.Set;
+import java.util.SortedMap;
+import java.util.function.Function;
+import java.util.function.UnaryOperator;
+import java.util.stream.Collectors;
+import org.apache.commons.lang3.ArrayUtils;
+import org.junit.After;
+import org.junit.Assert;
+import org.junit.Assume;
+import org.junit.Before;
+import org.junit.Test;
public abstract class AbstractKeyValueServiceTest {
private final KvsManager kvsManager;
diff --git a/atlasdb-tests-shared/src/main/java/com/palantir/atlasdb/keyvalue/impl/AbstractMultiCasTest.java b/atlasdb-tests-shared/src/main/java/com/palantir/atlasdb/keyvalue/impl/AbstractMultiCasTest.java
index f99c64efc90..936f2f3d161 100644
--- a/atlasdb-tests-shared/src/main/java/com/palantir/atlasdb/keyvalue/impl/AbstractMultiCasTest.java
+++ b/atlasdb-tests-shared/src/main/java/com/palantir/atlasdb/keyvalue/impl/AbstractMultiCasTest.java
@@ -18,14 +18,6 @@
import static org.assertj.core.api.Assertions.assertThat;
import static org.assertj.core.api.Assertions.assertThatThrownBy;
-import java.util.Map;
-import java.util.stream.Collectors;
-import java.util.stream.Stream;
-
-import org.junit.After;
-import org.junit.Before;
-import org.junit.Test;
-
import com.google.common.collect.ImmutableMap;
import com.palantir.atlasdb.AtlasDbConstants;
import com.palantir.atlasdb.encoding.PtBytes;
@@ -33,6 +25,12 @@
import com.palantir.atlasdb.keyvalue.api.KeyAlreadyExistsException;
import com.palantir.atlasdb.keyvalue.api.KeyValueService;
import com.palantir.atlasdb.keyvalue.api.TableReference;
+import java.util.Map;
+import java.util.stream.Collectors;
+import java.util.stream.Stream;
+import org.junit.After;
+import org.junit.Before;
+import org.junit.Test;
public class AbstractMultiCasTest {
private static final byte[] ROW_1 = PtBytes.toBytes("row1");
diff --git a/atlasdb-tests-shared/src/main/java/com/palantir/atlasdb/keyvalue/impl/AssertLockedKeyValueService.java b/atlasdb-tests-shared/src/main/java/com/palantir/atlasdb/keyvalue/impl/AssertLockedKeyValueService.java
index 67848988da5..1db6f0f19fb 100644
--- a/atlasdb-tests-shared/src/main/java/com/palantir/atlasdb/keyvalue/impl/AssertLockedKeyValueService.java
+++ b/atlasdb-tests-shared/src/main/java/com/palantir/atlasdb/keyvalue/impl/AssertLockedKeyValueService.java
@@ -15,10 +15,6 @@
*/
package com.palantir.atlasdb.keyvalue.impl;
-import java.util.Arrays;
-import java.util.Map;
-import java.util.SortedMap;
-
import com.google.common.collect.Maps;
import com.palantir.atlasdb.keyvalue.api.Cell;
import com.palantir.atlasdb.keyvalue.api.KeyValueService;
@@ -33,6 +29,9 @@
import com.palantir.lock.LockRequest;
import com.palantir.lock.LockService;
import com.palantir.logsafe.Preconditions;
+import java.util.Arrays;
+import java.util.Map;
+import java.util.SortedMap;
public class AssertLockedKeyValueService extends ForwardingKeyValueService {
final KeyValueService delegate;
diff --git a/atlasdb-tests-shared/src/main/java/com/palantir/atlasdb/keyvalue/impl/TestDataBuilder.java b/atlasdb-tests-shared/src/main/java/com/palantir/atlasdb/keyvalue/impl/TestDataBuilder.java
index d1dd9073e30..b17024d2f2e 100644
--- a/atlasdb-tests-shared/src/main/java/com/palantir/atlasdb/keyvalue/impl/TestDataBuilder.java
+++ b/atlasdb-tests-shared/src/main/java/com/palantir/atlasdb/keyvalue/impl/TestDataBuilder.java
@@ -15,15 +15,14 @@
*/
package com.palantir.atlasdb.keyvalue.impl;
-import java.nio.charset.StandardCharsets;
-import java.util.Map;
-
import com.google.common.collect.Maps;
import com.google.common.primitives.Ints;
import com.palantir.atlasdb.encoding.PtBytes;
import com.palantir.atlasdb.keyvalue.api.Cell;
import com.palantir.atlasdb.keyvalue.api.KeyValueService;
import com.palantir.atlasdb.keyvalue.api.TableReference;
+import java.nio.charset.StandardCharsets;
+import java.util.Map;
public class TestDataBuilder {
private final KeyValueService kvs;
diff --git a/atlasdb-tests-shared/src/main/java/com/palantir/atlasdb/keyvalue/impl/TestResourceManager.java b/atlasdb-tests-shared/src/main/java/com/palantir/atlasdb/keyvalue/impl/TestResourceManager.java
index 2bbf454072f..7aa5683a314 100644
--- a/atlasdb-tests-shared/src/main/java/com/palantir/atlasdb/keyvalue/impl/TestResourceManager.java
+++ b/atlasdb-tests-shared/src/main/java/com/palantir/atlasdb/keyvalue/impl/TestResourceManager.java
@@ -16,17 +16,15 @@
package com.palantir.atlasdb.keyvalue.impl;
+import com.google.common.base.Suppliers;
+import com.palantir.atlasdb.keyvalue.api.KeyValueService;
+import com.palantir.atlasdb.transaction.api.TransactionManager;
import java.util.ArrayList;
import java.util.List;
import java.util.Optional;
import java.util.function.Supplier;
-
import org.junit.rules.ExternalResource;
-import com.google.common.base.Suppliers;
-import com.palantir.atlasdb.keyvalue.api.KeyValueService;
-import com.palantir.atlasdb.transaction.api.TransactionManager;
-
public class TestResourceManager extends ExternalResource implements KvsManager, TransactionManagerManager {
private final Supplier getKvsSupplier;
private final List closeableResources = new ArrayList<>();
diff --git a/atlasdb-tests-shared/src/main/java/com/palantir/atlasdb/keyvalue/impl/TrackingKeyValueService.java b/atlasdb-tests-shared/src/main/java/com/palantir/atlasdb/keyvalue/impl/TrackingKeyValueService.java
index f2b17a645d2..cdf070ea634 100644
--- a/atlasdb-tests-shared/src/main/java/com/palantir/atlasdb/keyvalue/impl/TrackingKeyValueService.java
+++ b/atlasdb-tests-shared/src/main/java/com/palantir/atlasdb/keyvalue/impl/TrackingKeyValueService.java
@@ -15,9 +15,6 @@
*/
package com.palantir.atlasdb.keyvalue.impl;
-import java.util.Map;
-import java.util.Set;
-
import com.google.common.collect.ImmutableSet;
import com.google.common.collect.Maps;
import com.google.common.collect.Multimap;
@@ -32,6 +29,8 @@
import com.palantir.atlasdb.keyvalue.api.TableReference;
import com.palantir.atlasdb.keyvalue.api.Value;
import com.palantir.common.base.ClosableIterator;
+import java.util.Map;
+import java.util.Set;
public class TrackingKeyValueService extends ForwardingKeyValueService {
private final Set tablesWrittenTo = Sets.newSetFromMap(Maps.newConcurrentMap());
diff --git a/atlasdb-tests-shared/src/main/java/com/palantir/atlasdb/keyvalue/impl/TransactionManagerManager.java b/atlasdb-tests-shared/src/main/java/com/palantir/atlasdb/keyvalue/impl/TransactionManagerManager.java
index 9ba43ac47e1..0a5f31d9aa8 100644
--- a/atlasdb-tests-shared/src/main/java/com/palantir/atlasdb/keyvalue/impl/TransactionManagerManager.java
+++ b/atlasdb-tests-shared/src/main/java/com/palantir/atlasdb/keyvalue/impl/TransactionManagerManager.java
@@ -16,9 +16,8 @@
package com.palantir.atlasdb.keyvalue.impl;
-import java.util.Optional;
-
import com.palantir.atlasdb.transaction.api.TransactionManager;
+import java.util.Optional;
public interface TransactionManagerManager {
/**
diff --git a/atlasdb-tests-shared/src/main/java/com/palantir/atlasdb/sweep/AbstractBackgroundSweeperIntegrationTest.java b/atlasdb-tests-shared/src/main/java/com/palantir/atlasdb/sweep/AbstractBackgroundSweeperIntegrationTest.java
index 6af4e696647..1b8dced0507 100644
--- a/atlasdb-tests-shared/src/main/java/com/palantir/atlasdb/sweep/AbstractBackgroundSweeperIntegrationTest.java
+++ b/atlasdb-tests-shared/src/main/java/com/palantir/atlasdb/sweep/AbstractBackgroundSweeperIntegrationTest.java
@@ -15,17 +15,6 @@
*/
package com.palantir.atlasdb.sweep;
-import java.util.List;
-import java.util.Map;
-import java.util.Set;
-import java.util.concurrent.CountDownLatch;
-import java.util.function.LongSupplier;
-
-import org.junit.After;
-import org.junit.Assert;
-import org.junit.Before;
-import org.junit.Test;
-
import com.google.common.collect.ImmutableList;
import com.google.common.collect.Maps;
import com.google.common.primitives.Ints;
@@ -57,6 +46,15 @@
import com.palantir.common.base.ClosableIterator;
import com.palantir.lock.SingleLockService;
import com.palantir.timestamp.InMemoryTimestampService;
+import java.util.List;
+import java.util.Map;
+import java.util.Set;
+import java.util.concurrent.CountDownLatch;
+import java.util.function.LongSupplier;
+import org.junit.After;
+import org.junit.Assert;
+import org.junit.Before;
+import org.junit.Test;
public abstract class AbstractBackgroundSweeperIntegrationTest {
static final TableReference TABLE_1 = TableReference.createFromFullyQualifiedName("foo.bar");
diff --git a/atlasdb-tests-shared/src/main/java/com/palantir/atlasdb/sweep/AbstractSweepTaskRunnerTest.java b/atlasdb-tests-shared/src/main/java/com/palantir/atlasdb/sweep/AbstractSweepTaskRunnerTest.java
index 6a4cf4c9971..c9558ba2140 100644
--- a/atlasdb-tests-shared/src/main/java/com/palantir/atlasdb/sweep/AbstractSweepTaskRunnerTest.java
+++ b/atlasdb-tests-shared/src/main/java/com/palantir/atlasdb/sweep/AbstractSweepTaskRunnerTest.java
@@ -25,19 +25,6 @@
import static org.mockito.ArgumentMatchers.eq;
import static org.mockito.Mockito.doAnswer;
-import java.util.ArrayList;
-import java.util.Collection;
-import java.util.List;
-import java.util.Optional;
-import java.util.concurrent.atomic.AtomicLong;
-import java.util.function.LongSupplier;
-
-import org.apache.commons.lang3.RandomStringUtils;
-import org.junit.Assert;
-import org.junit.Before;
-import org.junit.Test;
-import org.mockito.Mockito;
-
import com.google.common.collect.HashMultimap;
import com.google.common.collect.ImmutableList;
import com.google.common.collect.ImmutableSet;
@@ -55,6 +42,17 @@
import com.palantir.atlasdb.protos.generated.TableMetadataPersistence.SweepStrategy;
import com.palantir.atlasdb.transaction.impl.TransactionConstants;
import com.palantir.util.Pair;
+import java.util.ArrayList;
+import java.util.Collection;
+import java.util.List;
+import java.util.Optional;
+import java.util.concurrent.atomic.AtomicLong;
+import java.util.function.LongSupplier;
+import org.apache.commons.lang3.RandomStringUtils;
+import org.junit.Assert;
+import org.junit.Before;
+import org.junit.Test;
+import org.mockito.Mockito;
public abstract class AbstractSweepTaskRunnerTest extends AbstractSweepTest {
protected static final int DEFAULT_BATCH_SIZE = 1000;
@@ -409,4 +407,3 @@ private SweepResults partialSweep(long ts) {
return results;
}
}
-
diff --git a/atlasdb-tests-shared/src/main/java/com/palantir/atlasdb/sweep/AbstractSweepTest.java b/atlasdb-tests-shared/src/main/java/com/palantir/atlasdb/sweep/AbstractSweepTest.java
index cd2866f6eaa..9201bf49eda 100644
--- a/atlasdb-tests-shared/src/main/java/com/palantir/atlasdb/sweep/AbstractSweepTest.java
+++ b/atlasdb-tests-shared/src/main/java/com/palantir/atlasdb/sweep/AbstractSweepTest.java
@@ -19,15 +19,6 @@
import static org.junit.Assert.assertEquals;
import static org.junit.Assert.assertNull;
-import java.nio.charset.StandardCharsets;
-import java.util.List;
-import java.util.Map;
-import java.util.Optional;
-import java.util.Set;
-
-import org.junit.Before;
-import org.junit.Test;
-
import com.google.common.collect.ImmutableMap;
import com.google.common.collect.ImmutableSet;
import com.google.common.collect.Lists;
@@ -52,6 +43,13 @@
import com.palantir.atlasdb.transaction.service.TransactionServices;
import com.palantir.atlasdb.util.MetricsManagers;
import com.palantir.timestamp.InMemoryTimestampService;
+import java.nio.charset.StandardCharsets;
+import java.util.List;
+import java.util.Map;
+import java.util.Optional;
+import java.util.Set;
+import org.junit.Before;
+import org.junit.Test;
public abstract class AbstractSweepTest {
protected static final String FULL_TABLE_NAME = "test_table.xyz_atlasdb_sweeper_test";
diff --git a/atlasdb-tests-shared/src/main/java/com/palantir/atlasdb/sweep/AbstractTargetedSweepTest.java b/atlasdb-tests-shared/src/main/java/com/palantir/atlasdb/sweep/AbstractTargetedSweepTest.java
index 89ad874664f..9c86237ba6d 100644
--- a/atlasdb-tests-shared/src/main/java/com/palantir/atlasdb/sweep/AbstractTargetedSweepTest.java
+++ b/atlasdb-tests-shared/src/main/java/com/palantir/atlasdb/sweep/AbstractTargetedSweepTest.java
@@ -19,11 +19,6 @@
import static org.assertj.core.api.Assertions.assertThat;
import static org.mockito.Mockito.mock;
-import java.util.Optional;
-
-import org.junit.Before;
-import org.junit.Test;
-
import com.google.common.collect.ImmutableMap;
import com.palantir.atlasdb.encoding.PtBytes;
import com.palantir.atlasdb.keyvalue.api.Cell;
@@ -42,6 +37,9 @@
import com.palantir.atlasdb.util.MetricsManager;
import com.palantir.atlasdb.util.MetricsManagers;
import com.palantir.lock.v2.TimelockService;
+import java.util.Optional;
+import org.junit.Before;
+import org.junit.Test;
public class AbstractTargetedSweepTest extends AbstractSweepTest {
private static final TableReference TABLE_TO_BE_DROPPED = TableReference.createFromFullyQualifiedName("ts.drop");
diff --git a/atlasdb-tests-shared/src/main/java/com/palantir/atlasdb/sweep/SweepTestUtils.java b/atlasdb-tests-shared/src/main/java/com/palantir/atlasdb/sweep/SweepTestUtils.java
index 570340f0e31..e1e1d5fd13b 100644
--- a/atlasdb-tests-shared/src/main/java/com/palantir/atlasdb/sweep/SweepTestUtils.java
+++ b/atlasdb-tests-shared/src/main/java/com/palantir/atlasdb/sweep/SweepTestUtils.java
@@ -15,11 +15,6 @@
*/
package com.palantir.atlasdb.sweep;
-import java.util.function.Supplier;
-
-import org.awaitility.Awaitility;
-import org.awaitility.Duration;
-
import com.palantir.atlasdb.cleaner.NoOpCleaner;
import com.palantir.atlasdb.cleaner.api.Cleaner;
import com.palantir.atlasdb.keyvalue.api.KeyValueService;
@@ -51,6 +46,9 @@
import com.palantir.timestamp.InMemoryTimestampService;
import com.palantir.timestamp.TimestampManagementService;
import com.palantir.timestamp.TimestampService;
+import java.util.function.Supplier;
+import org.awaitility.Awaitility;
+import org.awaitility.Duration;
public final class SweepTestUtils {
private SweepTestUtils() {}
diff --git a/atlasdb-tests-shared/src/main/java/com/palantir/atlasdb/sweep/SweeperServiceImplIntegrationTest.java b/atlasdb-tests-shared/src/main/java/com/palantir/atlasdb/sweep/SweeperServiceImplIntegrationTest.java
index 9491931490b..e58ec709a8e 100644
--- a/atlasdb-tests-shared/src/main/java/com/palantir/atlasdb/sweep/SweeperServiceImplIntegrationTest.java
+++ b/atlasdb-tests-shared/src/main/java/com/palantir/atlasdb/sweep/SweeperServiceImplIntegrationTest.java
@@ -15,12 +15,11 @@
*/
package com.palantir.atlasdb.sweep;
-import org.junit.Before;
-import org.junit.Test;
-
import com.palantir.atlasdb.keyvalue.api.KeyValueService;
import com.palantir.atlasdb.keyvalue.impl.InMemoryKeyValueService;
import com.palantir.atlasdb.protos.generated.TableMetadataPersistence.SweepStrategy;
+import org.junit.Before;
+import org.junit.Test;
public class SweeperServiceImplIntegrationTest extends AbstractBackgroundSweeperIntegrationTest {
private SweeperService sweeperService;
diff --git a/atlasdb-tests-shared/src/main/java/com/palantir/atlasdb/sweep/progress/AbstractSweepProgressStoreTest.java b/atlasdb-tests-shared/src/main/java/com/palantir/atlasdb/sweep/progress/AbstractSweepProgressStoreTest.java
index e8f866e17dc..0d1438d8267 100644
--- a/atlasdb-tests-shared/src/main/java/com/palantir/atlasdb/sweep/progress/AbstractSweepProgressStoreTest.java
+++ b/atlasdb-tests-shared/src/main/java/com/palantir/atlasdb/sweep/progress/AbstractSweepProgressStoreTest.java
@@ -15,13 +15,6 @@
*/
package com.palantir.atlasdb.sweep.progress;
-import java.util.Optional;
-
-import org.junit.After;
-import org.junit.Assert;
-import org.junit.Before;
-import org.junit.Test;
-
import com.fasterxml.jackson.core.JsonProcessingException;
import com.palantir.atlasdb.AtlasDbConstants;
import com.palantir.atlasdb.encoding.PtBytes;
@@ -29,6 +22,11 @@
import com.palantir.atlasdb.keyvalue.api.KeyValueService;
import com.palantir.atlasdb.keyvalue.api.TableReference;
import com.palantir.atlasdb.keyvalue.impl.KvsManager;
+import java.util.Optional;
+import org.junit.After;
+import org.junit.Assert;
+import org.junit.Before;
+import org.junit.Test;
public abstract class AbstractSweepProgressStoreTest {
private static final TableReference TABLE = TableReference.createFromFullyQualifiedName("foo.bar");
diff --git a/atlasdb-tests-shared/src/main/java/com/palantir/atlasdb/timestamp/AbstractDbTimestampBoundStoreTest.java b/atlasdb-tests-shared/src/main/java/com/palantir/atlasdb/timestamp/AbstractDbTimestampBoundStoreTest.java
index 2abf5ba8344..f0c883453de 100644
--- a/atlasdb-tests-shared/src/main/java/com/palantir/atlasdb/timestamp/AbstractDbTimestampBoundStoreTest.java
+++ b/atlasdb-tests-shared/src/main/java/com/palantir/atlasdb/timestamp/AbstractDbTimestampBoundStoreTest.java
@@ -15,12 +15,11 @@
*/
package com.palantir.atlasdb.timestamp;
+import com.palantir.timestamp.TimestampBoundStore;
import org.junit.Assert;
import org.junit.Before;
import org.junit.Test;
-import com.palantir.timestamp.TimestampBoundStore;
-
public abstract class AbstractDbTimestampBoundStoreTest {
protected TimestampBoundStore store;
diff --git a/atlasdb-tests-shared/src/main/java/com/palantir/atlasdb/timestamp/AbstractTimestampServiceTests.java b/atlasdb-tests-shared/src/main/java/com/palantir/atlasdb/timestamp/AbstractTimestampServiceTests.java
index 91748f3d5d3..997f676de87 100644
--- a/atlasdb-tests-shared/src/main/java/com/palantir/atlasdb/timestamp/AbstractTimestampServiceTests.java
+++ b/atlasdb-tests-shared/src/main/java/com/palantir/atlasdb/timestamp/AbstractTimestampServiceTests.java
@@ -15,20 +15,18 @@
*/
package com.palantir.atlasdb.timestamp;
+import com.palantir.timestamp.TimestampManagementService;
+import com.palantir.timestamp.TimestampRange;
+import com.palantir.timestamp.TimestampService;
import java.util.Set;
import java.util.concurrent.ConcurrentSkipListSet;
import java.util.concurrent.ExecutorService;
import java.util.concurrent.Executors;
import java.util.concurrent.TimeUnit;
import java.util.concurrent.TimeoutException;
-
import org.assertj.core.api.Assertions;
import org.junit.Test;
-import com.palantir.timestamp.TimestampManagementService;
-import com.palantir.timestamp.TimestampRange;
-import com.palantir.timestamp.TimestampService;
-
public abstract class AbstractTimestampServiceTests {
private static final long ONE_MILLION = 1_000_000;
private static final long TWO_MILLION = 2 * ONE_MILLION;
diff --git a/atlasdb-tests-shared/src/main/java/com/palantir/atlasdb/transaction/impl/AbstractSerializableTransactionTest.java b/atlasdb-tests-shared/src/main/java/com/palantir/atlasdb/transaction/impl/AbstractSerializableTransactionTest.java
index 63b55985670..74801b5e947 100644
--- a/atlasdb-tests-shared/src/main/java/com/palantir/atlasdb/transaction/impl/AbstractSerializableTransactionTest.java
+++ b/atlasdb-tests-shared/src/main/java/com/palantir/atlasdb/transaction/impl/AbstractSerializableTransactionTest.java
@@ -20,19 +20,6 @@
import static org.junit.Assert.assertNull;
import static org.junit.Assert.fail;
-import java.util.Iterator;
-import java.util.Map;
-import java.util.Optional;
-import java.util.concurrent.BrokenBarrierException;
-import java.util.concurrent.Callable;
-import java.util.concurrent.CyclicBarrier;
-import java.util.concurrent.ExecutionException;
-import java.util.concurrent.ExecutorService;
-import java.util.concurrent.Future;
-
-import org.junit.Assert;
-import org.junit.Test;
-
import com.google.common.base.Suppliers;
import com.google.common.collect.ImmutableList;
import com.google.common.collect.ImmutableMap;
@@ -70,6 +57,17 @@
import com.palantir.common.base.Throwables;
import com.palantir.common.concurrent.PTExecutors;
import com.palantir.lock.impl.LegacyTimelockService;
+import java.util.Iterator;
+import java.util.Map;
+import java.util.Optional;
+import java.util.concurrent.BrokenBarrierException;
+import java.util.concurrent.Callable;
+import java.util.concurrent.CyclicBarrier;
+import java.util.concurrent.ExecutionException;
+import java.util.concurrent.ExecutorService;
+import java.util.concurrent.Future;
+import org.junit.Assert;
+import org.junit.Test;
@SuppressWarnings("CheckReturnValue")
public abstract class AbstractSerializableTransactionTest extends AbstractTransactionTest {
diff --git a/atlasdb-tests-shared/src/main/java/com/palantir/atlasdb/transaction/impl/AbstractTransactionTest.java b/atlasdb-tests-shared/src/main/java/com/palantir/atlasdb/transaction/impl/AbstractTransactionTest.java
index 844016b7f61..b3d8c9b2b9e 100644
--- a/atlasdb-tests-shared/src/main/java/com/palantir/atlasdb/transaction/impl/AbstractTransactionTest.java
+++ b/atlasdb-tests-shared/src/main/java/com/palantir/atlasdb/transaction/impl/AbstractTransactionTest.java
@@ -24,23 +24,6 @@
import static org.junit.Assert.assertTrue;
import static org.junit.Assert.fail;
-import java.nio.charset.StandardCharsets;
-import java.util.Arrays;
-import java.util.Collections;
-import java.util.Iterator;
-import java.util.List;
-import java.util.Map;
-import java.util.Optional;
-import java.util.Random;
-import java.util.concurrent.CountDownLatch;
-import java.util.concurrent.ExecutionException;
-import java.util.concurrent.ExecutorService;
-import java.util.concurrent.Executors;
-import java.util.concurrent.FutureTask;
-import java.util.stream.Collectors;
-
-import org.junit.Test;
-
import com.google.common.collect.ImmutableList;
import com.google.common.collect.ImmutableMap;
import com.google.common.collect.ImmutableMultimap;
@@ -96,6 +79,21 @@
import com.palantir.lock.impl.LegacyTimelockService;
import com.palantir.util.Pair;
import com.palantir.util.paging.TokenBackedBasicResultsPage;
+import java.nio.charset.StandardCharsets;
+import java.util.Arrays;
+import java.util.Collections;
+import java.util.Iterator;
+import java.util.List;
+import java.util.Map;
+import java.util.Optional;
+import java.util.Random;
+import java.util.concurrent.CountDownLatch;
+import java.util.concurrent.ExecutionException;
+import java.util.concurrent.ExecutorService;
+import java.util.concurrent.Executors;
+import java.util.concurrent.FutureTask;
+import java.util.stream.Collectors;
+import org.junit.Test;
@SuppressWarnings({"checkstyle:all","DefaultCharset"}) // TODO(someonebored): clean this horrible test class up!
public abstract class AbstractTransactionTest extends TransactionTestSetup {
diff --git a/atlasdb-tests-shared/src/main/java/com/palantir/atlasdb/transaction/impl/CommitLockTest.java b/atlasdb-tests-shared/src/main/java/com/palantir/atlasdb/transaction/impl/CommitLockTest.java
index d6e63064c6b..02bffbfc564 100644
--- a/atlasdb-tests-shared/src/main/java/com/palantir/atlasdb/transaction/impl/CommitLockTest.java
+++ b/atlasdb-tests-shared/src/main/java/com/palantir/atlasdb/transaction/impl/CommitLockTest.java
@@ -18,16 +18,6 @@
import static org.junit.Assert.assertFalse;
import static org.junit.Assert.assertTrue;
-import java.util.Map;
-import java.util.Optional;
-
-import org.junit.Assume;
-import org.junit.ClassRule;
-import org.junit.experimental.theories.DataPoints;
-import org.junit.experimental.theories.Theories;
-import org.junit.experimental.theories.Theory;
-import org.junit.runner.RunWith;
-
import com.google.common.base.Suppliers;
import com.google.common.collect.ImmutableMap;
import com.google.common.collect.ImmutableSet;
@@ -53,6 +43,14 @@
import com.palantir.lock.LockDescriptor;
import com.palantir.lock.v2.LockRequest;
import com.palantir.lock.v2.LockResponse;
+import java.util.Map;
+import java.util.Optional;
+import org.junit.Assume;
+import org.junit.ClassRule;
+import org.junit.experimental.theories.DataPoints;
+import org.junit.experimental.theories.Theories;
+import org.junit.experimental.theories.Theory;
+import org.junit.runner.RunWith;
@RunWith(Theories.class)
public class CommitLockTest extends TransactionTestSetup {
diff --git a/atlasdb-tests-shared/src/main/java/com/palantir/atlasdb/transaction/impl/GetAsyncDelegate.java b/atlasdb-tests-shared/src/main/java/com/palantir/atlasdb/transaction/impl/GetAsyncDelegate.java
index 5bfd7c97a26..bdcd95448b9 100644
--- a/atlasdb-tests-shared/src/main/java/com/palantir/atlasdb/transaction/impl/GetAsyncDelegate.java
+++ b/atlasdb-tests-shared/src/main/java/com/palantir/atlasdb/transaction/impl/GetAsyncDelegate.java
@@ -16,13 +16,12 @@
package com.palantir.atlasdb.transaction.impl;
-import java.util.Map;
-import java.util.Set;
-
import com.palantir.atlasdb.futures.AtlasFutures;
import com.palantir.atlasdb.keyvalue.api.Cell;
import com.palantir.atlasdb.keyvalue.api.TableReference;
import com.palantir.atlasdb.transaction.api.Transaction;
+import java.util.Map;
+import java.util.Set;
public class GetAsyncDelegate extends ForwardingTransaction {
private final Transaction delegate;
diff --git a/atlasdb-tests-shared/src/main/java/com/palantir/atlasdb/transaction/impl/TestConflictDetectionManagers.java b/atlasdb-tests-shared/src/main/java/com/palantir/atlasdb/transaction/impl/TestConflictDetectionManagers.java
index 772bd1ee636..47b54a5bd5f 100644
--- a/atlasdb-tests-shared/src/main/java/com/palantir/atlasdb/transaction/impl/TestConflictDetectionManagers.java
+++ b/atlasdb-tests-shared/src/main/java/com/palantir/atlasdb/transaction/impl/TestConflictDetectionManagers.java
@@ -15,12 +15,11 @@
*/
package com.palantir.atlasdb.transaction.impl;
-import java.util.Map;
-
import com.google.common.annotations.VisibleForTesting;
import com.google.common.cache.CacheLoader;
import com.palantir.atlasdb.keyvalue.api.TableReference;
import com.palantir.atlasdb.transaction.api.ConflictHandler;
+import java.util.Map;
public final class TestConflictDetectionManagers {
private TestConflictDetectionManagers() {}
diff --git a/atlasdb-tests-shared/src/main/java/com/palantir/atlasdb/transaction/impl/TestTransactionManagerImpl.java b/atlasdb-tests-shared/src/main/java/com/palantir/atlasdb/transaction/impl/TestTransactionManagerImpl.java
index 22959e2fd70..c07dd8aa8e6 100644
--- a/atlasdb-tests-shared/src/main/java/com/palantir/atlasdb/transaction/impl/TestTransactionManagerImpl.java
+++ b/atlasdb-tests-shared/src/main/java/com/palantir/atlasdb/transaction/impl/TestTransactionManagerImpl.java
@@ -15,12 +15,6 @@
*/
package com.palantir.atlasdb.transaction.impl;
-import java.util.HashMap;
-import java.util.Map;
-import java.util.Optional;
-import java.util.concurrent.ExecutorService;
-import java.util.function.Supplier;
-
import com.google.common.base.Suppliers;
import com.google.common.util.concurrent.MoreExecutors;
import com.palantir.atlasdb.cache.DefaultTimestampCache;
@@ -46,6 +40,11 @@
import com.palantir.lock.v2.LockToken;
import com.palantir.timestamp.TimestampManagementService;
import com.palantir.timestamp.TimestampService;
+import java.util.HashMap;
+import java.util.Map;
+import java.util.Optional;
+import java.util.concurrent.ExecutorService;
+import java.util.function.Supplier;
public class TestTransactionManagerImpl extends SerializableTransactionManager implements TestTransactionManager {
static final TransactionConfig TRANSACTION_CONFIG = ImmutableTransactionConfig.builder().build();
diff --git a/atlasdb-tests-shared/src/main/java/com/palantir/atlasdb/transaction/impl/TransactionTestSetup.java b/atlasdb-tests-shared/src/main/java/com/palantir/atlasdb/transaction/impl/TransactionTestSetup.java
index 73c64058976..c31f77234cc 100644
--- a/atlasdb-tests-shared/src/main/java/com/palantir/atlasdb/transaction/impl/TransactionTestSetup.java
+++ b/atlasdb-tests-shared/src/main/java/com/palantir/atlasdb/transaction/impl/TransactionTestSetup.java
@@ -15,18 +15,6 @@
*/
package com.palantir.atlasdb.transaction.impl;
-import java.io.File;
-import java.io.IOException;
-import java.util.Map;
-
-import org.junit.AfterClass;
-import org.junit.Before;
-import org.junit.BeforeClass;
-import org.junit.ClassRule;
-import org.junit.rules.TemporaryFolder;
-import org.rocksdb.RocksDB;
-import org.rocksdb.RocksDBException;
-
import com.google.common.collect.ImmutableMap;
import com.google.common.collect.ImmutableSet;
import com.google.common.collect.Maps;
@@ -62,6 +50,16 @@
import com.palantir.timestamp.TimestampManagementService;
import com.palantir.timestamp.TimestampService;
import com.palantir.util.Pair;
+import java.io.File;
+import java.io.IOException;
+import java.util.Map;
+import org.junit.AfterClass;
+import org.junit.Before;
+import org.junit.BeforeClass;
+import org.junit.ClassRule;
+import org.junit.rules.TemporaryFolder;
+import org.rocksdb.RocksDB;
+import org.rocksdb.RocksDBException;
public abstract class TransactionTestSetup {
@ClassRule
diff --git a/atlasdb-tests-shared/src/main/java/com/palantir/atlasdb/transaction/impl/metrics/TransactionOutcomeMetricsAssert.java b/atlasdb-tests-shared/src/main/java/com/palantir/atlasdb/transaction/impl/metrics/TransactionOutcomeMetricsAssert.java
index f620f9c0bb3..852381b1351 100644
--- a/atlasdb-tests-shared/src/main/java/com/palantir/atlasdb/transaction/impl/metrics/TransactionOutcomeMetricsAssert.java
+++ b/atlasdb-tests-shared/src/main/java/com/palantir/atlasdb/transaction/impl/metrics/TransactionOutcomeMetricsAssert.java
@@ -15,18 +15,16 @@
*/
package com.palantir.atlasdb.transaction.impl.metrics;
-import java.util.function.LongConsumer;
-
-import org.assertj.core.api.AbstractAssert;
-import org.assertj.core.api.WritableAssertionInfo;
-import org.assertj.core.internal.Objects;
-
import com.codahale.metrics.Metric;
import com.google.common.collect.ImmutableMap;
import com.palantir.atlasdb.keyvalue.api.TableReference;
import com.palantir.atlasdb.logging.LoggingArgs;
import com.palantir.tritium.metrics.registry.MetricName;
import com.palantir.tritium.metrics.registry.TaggedMetricRegistry;
+import java.util.function.LongConsumer;
+import org.assertj.core.api.AbstractAssert;
+import org.assertj.core.api.WritableAssertionInfo;
+import org.assertj.core.internal.Objects;
public class TransactionOutcomeMetricsAssert extends
AbstractAssert {
diff --git a/atlasdb-tests-shared/src/test/java/com/palantir/atlasdb/AllAtlasDbFastTests.java b/atlasdb-tests-shared/src/test/java/com/palantir/atlasdb/AllAtlasDbFastTests.java
index 1a2e815d80c..b3db7a9b965 100644
--- a/atlasdb-tests-shared/src/test/java/com/palantir/atlasdb/AllAtlasDbFastTests.java
+++ b/atlasdb-tests-shared/src/test/java/com/palantir/atlasdb/AllAtlasDbFastTests.java
@@ -15,15 +15,14 @@
*/
package com.palantir.atlasdb;
-import org.junit.runner.RunWith;
-import org.junit.runners.Suite;
-import org.junit.runners.Suite.SuiteClasses;
-
import com.palantir.atlasdb.cleaner.AsyncPuncherTest;
import com.palantir.atlasdb.cleaner.InMemorySweepTaskRunnerTest;
import com.palantir.atlasdb.cleaner.PuncherTest;
import com.palantir.atlasdb.compress.CompressionUtilsTest;
import com.palantir.atlasdb.keyvalue.impl.RangeRequestsTest;
+import org.junit.runner.RunWith;
+import org.junit.runners.Suite;
+import org.junit.runners.Suite.SuiteClasses;
@RunWith(Suite.class)
@SuiteClasses({
diff --git a/atlasdb-tests-shared/src/test/java/com/palantir/atlasdb/AtlasDbTestCase.java b/atlasdb-tests-shared/src/test/java/com/palantir/atlasdb/AtlasDbTestCase.java
index ca2a0b2403e..2890dd255c5 100644
--- a/atlasdb-tests-shared/src/test/java/com/palantir/atlasdb/AtlasDbTestCase.java
+++ b/atlasdb-tests-shared/src/test/java/com/palantir/atlasdb/AtlasDbTestCase.java
@@ -17,12 +17,6 @@
import static org.mockito.Mockito.spy;
-import java.util.Set;
-import java.util.concurrent.ExecutorService;
-
-import org.junit.After;
-import org.junit.Before;
-
import com.google.common.util.concurrent.MoreExecutors;
import com.palantir.atlasdb.cache.DefaultTimestampCache;
import com.palantir.atlasdb.keyvalue.api.KeyValueService;
@@ -54,6 +48,10 @@
import com.palantir.lock.LockService;
import com.palantir.lock.impl.LockServiceImpl;
import com.palantir.timestamp.InMemoryTimestampService;
+import java.util.Set;
+import java.util.concurrent.ExecutorService;
+import org.junit.After;
+import org.junit.Before;
public class AtlasDbTestCase {
protected LockClient lockClient;
diff --git a/atlasdb-tests-shared/src/test/java/com/palantir/atlasdb/cleaner/AsyncPuncherTest.java b/atlasdb-tests-shared/src/test/java/com/palantir/atlasdb/cleaner/AsyncPuncherTest.java
index 1b5c723af8f..2c4ce9ef75a 100644
--- a/atlasdb-tests-shared/src/test/java/com/palantir/atlasdb/cleaner/AsyncPuncherTest.java
+++ b/atlasdb-tests-shared/src/test/java/com/palantir/atlasdb/cleaner/AsyncPuncherTest.java
@@ -19,16 +19,14 @@
import static org.mockito.Mockito.mock;
import static org.mockito.Mockito.when;
-import java.util.function.LongSupplier;
-
-import org.junit.Before;
-import org.junit.Test;
-
import com.google.common.base.Suppliers;
import com.palantir.common.time.Clock;
import com.palantir.common.time.SystemClock;
import com.palantir.timestamp.InMemoryTimestampService;
import com.palantir.timestamp.TimestampService;
+import java.util.function.LongSupplier;
+import org.junit.Before;
+import org.junit.Test;
public class AsyncPuncherTest {
diff --git a/atlasdb-tests-shared/src/test/java/com/palantir/atlasdb/cleaner/InMemorySweepTaskRunnerTest.java b/atlasdb-tests-shared/src/test/java/com/palantir/atlasdb/cleaner/InMemorySweepTaskRunnerTest.java
index 22879a3af58..4ab456fba9b 100644
--- a/atlasdb-tests-shared/src/test/java/com/palantir/atlasdb/cleaner/InMemorySweepTaskRunnerTest.java
+++ b/atlasdb-tests-shared/src/test/java/com/palantir/atlasdb/cleaner/InMemorySweepTaskRunnerTest.java
@@ -15,19 +15,17 @@
*/
package com.palantir.atlasdb.cleaner;
+import com.palantir.atlasdb.keyvalue.api.SweepResults;
+import com.palantir.atlasdb.keyvalue.impl.TestResourceManager;
+import com.palantir.atlasdb.protos.generated.TableMetadataPersistence;
+import com.palantir.atlasdb.sweep.AbstractSweepTaskRunnerTest;
import java.util.Optional;
import java.util.stream.IntStream;
-
import org.apache.commons.lang3.RandomStringUtils;
import org.junit.Assert;
import org.junit.ClassRule;
import org.junit.Test;
-import com.palantir.atlasdb.keyvalue.api.SweepResults;
-import com.palantir.atlasdb.keyvalue.impl.TestResourceManager;
-import com.palantir.atlasdb.protos.generated.TableMetadataPersistence;
-import com.palantir.atlasdb.sweep.AbstractSweepTaskRunnerTest;
-
public class InMemorySweepTaskRunnerTest extends AbstractSweepTaskRunnerTest {
@ClassRule
public static final TestResourceManager TRM = TestResourceManager.inMemory();
diff --git a/atlasdb-tests-shared/src/test/java/com/palantir/atlasdb/cleaner/PuncherTest.java b/atlasdb-tests-shared/src/test/java/com/palantir/atlasdb/cleaner/PuncherTest.java
index 23176baac2a..1644fde644b 100644
--- a/atlasdb-tests-shared/src/test/java/com/palantir/atlasdb/cleaner/PuncherTest.java
+++ b/atlasdb-tests-shared/src/test/java/com/palantir/atlasdb/cleaner/PuncherTest.java
@@ -17,21 +17,19 @@
import static org.junit.Assert.assertEquals;
+import com.google.common.base.Suppliers;
+import com.google.common.collect.ImmutableList;
+import com.palantir.atlasdb.keyvalue.api.KeyValueService;
+import com.palantir.atlasdb.keyvalue.impl.InMemoryKeyValueService;
+import com.palantir.common.time.Clock;
import java.util.Collection;
import java.util.function.Supplier;
-
import org.junit.After;
import org.junit.Test;
import org.junit.runner.RunWith;
import org.junit.runners.Parameterized;
import org.junit.runners.Parameterized.Parameters;
-import com.google.common.base.Suppliers;
-import com.google.common.collect.ImmutableList;
-import com.palantir.atlasdb.keyvalue.api.KeyValueService;
-import com.palantir.atlasdb.keyvalue.impl.InMemoryKeyValueService;
-import com.palantir.common.time.Clock;
-
@RunWith(Parameterized.class)
public class PuncherTest {
private static final long GRANULARITY_MILLIS = 10;
diff --git a/atlasdb-tests-shared/src/test/java/com/palantir/atlasdb/compress/CompressionUtilsTest.java b/atlasdb-tests-shared/src/test/java/com/palantir/atlasdb/compress/CompressionUtilsTest.java
index b537d7cb5cb..5781e734ba7 100644
--- a/atlasdb-tests-shared/src/test/java/com/palantir/atlasdb/compress/CompressionUtilsTest.java
+++ b/atlasdb-tests-shared/src/test/java/com/palantir/atlasdb/compress/CompressionUtilsTest.java
@@ -19,12 +19,10 @@
import static org.junit.Assert.assertFalse;
import static org.junit.Assert.assertTrue;
+import com.palantir.atlasdb.table.description.ColumnValueDescription.Compression;
import java.util.Arrays;
-
import org.junit.Test;
-import com.palantir.atlasdb.table.description.ColumnValueDescription.Compression;
-
public class CompressionUtilsTest {
@Test
public void testCompressAndDecompress() {
diff --git a/atlasdb-tests-shared/src/test/java/com/palantir/atlasdb/health/MetricsBasedTimelockHealthCheckTest.java b/atlasdb-tests-shared/src/test/java/com/palantir/atlasdb/health/MetricsBasedTimelockHealthCheckTest.java
index 8dfd9bc980c..66de6d2f54a 100644
--- a/atlasdb-tests-shared/src/test/java/com/palantir/atlasdb/health/MetricsBasedTimelockHealthCheckTest.java
+++ b/atlasdb-tests-shared/src/test/java/com/palantir/atlasdb/health/MetricsBasedTimelockHealthCheckTest.java
@@ -19,13 +19,11 @@
import static org.mockito.Mockito.mock;
import static org.mockito.Mockito.when;
-import java.util.concurrent.TimeUnit;
-
-import org.junit.Test;
-
import com.codahale.metrics.MetricRegistry;
import com.palantir.atlasdb.transaction.impl.InstrumentedTimelockService;
import com.palantir.lock.v2.TimelockService;
+import java.util.concurrent.TimeUnit;
+import org.junit.Test;
public class MetricsBasedTimelockHealthCheckTest {
private static final long METRICS_TICK_INTERVAL = TimeUnit.SECONDS.toMillis(5);
diff --git a/atlasdb-tests-shared/src/test/java/com/palantir/atlasdb/keyvalue/MemorySerializableTransactionTest.java b/atlasdb-tests-shared/src/test/java/com/palantir/atlasdb/keyvalue/MemorySerializableTransactionTest.java
index 685fa39f9bc..9883feec89d 100644
--- a/atlasdb-tests-shared/src/test/java/com/palantir/atlasdb/keyvalue/MemorySerializableTransactionTest.java
+++ b/atlasdb-tests-shared/src/test/java/com/palantir/atlasdb/keyvalue/MemorySerializableTransactionTest.java
@@ -15,10 +15,9 @@
*/
package com.palantir.atlasdb.keyvalue;
-import org.junit.ClassRule;
-
import com.palantir.atlasdb.keyvalue.impl.TestResourceManager;
import com.palantir.atlasdb.transaction.impl.AbstractSerializableTransactionTest;
+import org.junit.ClassRule;
public class MemorySerializableTransactionTest extends AbstractSerializableTransactionTest {
@ClassRule
diff --git a/atlasdb-tests-shared/src/test/java/com/palantir/atlasdb/keyvalue/MemoryTransactionTest.java b/atlasdb-tests-shared/src/test/java/com/palantir/atlasdb/keyvalue/MemoryTransactionTest.java
index 29141b71ed9..0ecedc5aec5 100644
--- a/atlasdb-tests-shared/src/test/java/com/palantir/atlasdb/keyvalue/MemoryTransactionTest.java
+++ b/atlasdb-tests-shared/src/test/java/com/palantir/atlasdb/keyvalue/MemoryTransactionTest.java
@@ -15,10 +15,9 @@
*/
package com.palantir.atlasdb.keyvalue;
-import org.junit.ClassRule;
-
import com.palantir.atlasdb.keyvalue.impl.TestResourceManager;
import com.palantir.atlasdb.transaction.impl.AbstractTransactionTest;
+import org.junit.ClassRule;
public class MemoryTransactionTest extends AbstractTransactionTest {
@ClassRule
diff --git a/atlasdb-tests-shared/src/test/java/com/palantir/atlasdb/keyvalue/impl/ColumnRangeSelectionTest.java b/atlasdb-tests-shared/src/test/java/com/palantir/atlasdb/keyvalue/impl/ColumnRangeSelectionTest.java
index bbd9abbd587..c043ea1905e 100644
--- a/atlasdb-tests-shared/src/test/java/com/palantir/atlasdb/keyvalue/impl/ColumnRangeSelectionTest.java
+++ b/atlasdb-tests-shared/src/test/java/com/palantir/atlasdb/keyvalue/impl/ColumnRangeSelectionTest.java
@@ -19,10 +19,9 @@
import static org.assertj.core.api.Assertions.assertThatCode;
import static org.assertj.core.api.Assertions.assertThatThrownBy;
-import org.junit.Test;
-
import com.palantir.atlasdb.encoding.PtBytes;
import com.palantir.atlasdb.keyvalue.api.ColumnRangeSelection;
+import org.junit.Test;
public class ColumnRangeSelectionTest {
private static final byte[] BYTES_1 = PtBytes.toBytes("aaaaaaa");
diff --git a/atlasdb-tests-shared/src/test/java/com/palantir/atlasdb/keyvalue/impl/RangeRequestTest.java b/atlasdb-tests-shared/src/test/java/com/palantir/atlasdb/keyvalue/impl/RangeRequestTest.java
index f76134ca7c6..ef4ca4484dd 100644
--- a/atlasdb-tests-shared/src/test/java/com/palantir/atlasdb/keyvalue/impl/RangeRequestTest.java
+++ b/atlasdb-tests-shared/src/test/java/com/palantir/atlasdb/keyvalue/impl/RangeRequestTest.java
@@ -17,11 +17,10 @@
import static org.junit.Assert.assertEquals;
-import org.junit.Assert;
-import org.junit.Test;
-
import com.palantir.atlasdb.keyvalue.api.RangeRequest;
import com.palantir.atlasdb.keyvalue.api.RangeRequests;
+import org.junit.Assert;
+import org.junit.Test;
public class RangeRequestTest {
diff --git a/atlasdb-tests-shared/src/test/java/com/palantir/atlasdb/keyvalue/impl/RangeRequestsTest.java b/atlasdb-tests-shared/src/test/java/com/palantir/atlasdb/keyvalue/impl/RangeRequestsTest.java
index 94109ecb6f7..3e80a50a485 100644
--- a/atlasdb-tests-shared/src/test/java/com/palantir/atlasdb/keyvalue/impl/RangeRequestsTest.java
+++ b/atlasdb-tests-shared/src/test/java/com/palantir/atlasdb/keyvalue/impl/RangeRequestsTest.java
@@ -17,15 +17,13 @@
import static org.assertj.core.api.Assertions.assertThat;
+import com.palantir.atlasdb.encoding.PtBytes;
+import com.palantir.atlasdb.keyvalue.api.RangeRequests;
import java.util.Arrays;
import java.util.Random;
-
import org.junit.Assert;
import org.junit.Test;
-import com.palantir.atlasdb.encoding.PtBytes;
-import com.palantir.atlasdb.keyvalue.api.RangeRequests;
-
public class RangeRequestsTest {
private static final byte[] BYTES_1 = PtBytes.toBytes("apple");
diff --git a/atlasdb-tests-shared/src/test/java/com/palantir/atlasdb/keyvalue/impl/TableSplittingKeyValueServiceTest.java b/atlasdb-tests-shared/src/test/java/com/palantir/atlasdb/keyvalue/impl/TableSplittingKeyValueServiceTest.java
index 9267a0ff866..83bd21668d0 100644
--- a/atlasdb-tests-shared/src/test/java/com/palantir/atlasdb/keyvalue/impl/TableSplittingKeyValueServiceTest.java
+++ b/atlasdb-tests-shared/src/test/java/com/palantir/atlasdb/keyvalue/impl/TableSplittingKeyValueServiceTest.java
@@ -17,12 +17,6 @@
import static org.assertj.core.api.Assertions.assertThat;
-import java.util.Map;
-
-import org.jmock.Expectations;
-import org.jmock.Mockery;
-import org.junit.Test;
-
import com.google.common.collect.ImmutableList;
import com.google.common.collect.ImmutableMap;
import com.palantir.atlasdb.keyvalue.api.Cell;
@@ -30,6 +24,10 @@
import com.palantir.atlasdb.keyvalue.api.KeyValueService;
import com.palantir.atlasdb.keyvalue.api.Namespace;
import com.palantir.atlasdb.keyvalue.api.TableReference;
+import java.util.Map;
+import org.jmock.Expectations;
+import org.jmock.Mockery;
+import org.junit.Test;
@SuppressWarnings({"checkstyle:Indentation", "checkstyle:RightCurly"}) // Expectations syntax
public class TableSplittingKeyValueServiceTest {
diff --git a/atlasdb-tests-shared/src/test/java/com/palantir/atlasdb/keyvalue/impl/TableTasksTest.java b/atlasdb-tests-shared/src/test/java/com/palantir/atlasdb/keyvalue/impl/TableTasksTest.java
index 3d5b77a76e5..dd6d161389e 100644
--- a/atlasdb-tests-shared/src/test/java/com/palantir/atlasdb/keyvalue/impl/TableTasksTest.java
+++ b/atlasdb-tests-shared/src/test/java/com/palantir/atlasdb/keyvalue/impl/TableTasksTest.java
@@ -15,17 +15,6 @@
*/
package com.palantir.atlasdb.keyvalue.impl;
-import java.util.Collections;
-import java.util.Map.Entry;
-import java.util.Random;
-import java.util.concurrent.atomic.AtomicLong;
-import java.util.function.Supplier;
-
-import org.junit.After;
-import org.junit.Assert;
-import org.junit.Before;
-import org.junit.Test;
-
import com.google.common.base.Suppliers;
import com.google.common.collect.HashMultimap;
import com.google.common.collect.ImmutableMap;
@@ -57,6 +46,15 @@
import com.palantir.lock.LockServerOptions;
import com.palantir.lock.impl.LockServiceImpl;
import com.palantir.timestamp.InMemoryTimestampService;
+import java.util.Collections;
+import java.util.Map.Entry;
+import java.util.Random;
+import java.util.concurrent.atomic.AtomicLong;
+import java.util.function.Supplier;
+import org.junit.After;
+import org.junit.Assert;
+import org.junit.Before;
+import org.junit.Test;
public class TableTasksTest {
private MetricsManager metricsManager;
diff --git a/atlasdb-tests-shared/src/test/java/com/palantir/atlasdb/keyvalue/impl/TracingKvsTest.java b/atlasdb-tests-shared/src/test/java/com/palantir/atlasdb/keyvalue/impl/TracingKvsTest.java
index 51bf98015d2..20aacf6bb55 100644
--- a/atlasdb-tests-shared/src/test/java/com/palantir/atlasdb/keyvalue/impl/TracingKvsTest.java
+++ b/atlasdb-tests-shared/src/test/java/com/palantir/atlasdb/keyvalue/impl/TracingKvsTest.java
@@ -17,22 +17,20 @@
import static org.assertj.core.api.Assertions.assertThat;
+import com.palantir.tracing.Tracer;
+import com.palantir.tracing.api.Span;
+import com.palantir.tracing.api.SpanObserver;
+import com.palantir.tracing.api.SpanType;
import java.util.ArrayList;
import java.util.Collections;
import java.util.List;
import java.util.Objects;
import java.util.Optional;
import java.util.stream.Collectors;
-
import org.junit.ClassRule;
import org.slf4j.Logger;
import org.slf4j.LoggerFactory;
-import com.palantir.tracing.Tracer;
-import com.palantir.tracing.api.Span;
-import com.palantir.tracing.api.SpanObserver;
-import com.palantir.tracing.api.SpanType;
-
public class TracingKvsTest extends AbstractKeyValueServiceTest {
@ClassRule
public static final TestResourceManager TRM = new TestResourceManager(() ->
diff --git a/atlasdb-tests-shared/src/test/java/com/palantir/atlasdb/ptobject/EncodingUtilsTest.java b/atlasdb-tests-shared/src/test/java/com/palantir/atlasdb/ptobject/EncodingUtilsTest.java
index d56feb4aac9..28674d0b99f 100644
--- a/atlasdb-tests-shared/src/test/java/com/palantir/atlasdb/ptobject/EncodingUtilsTest.java
+++ b/atlasdb-tests-shared/src/test/java/com/palantir/atlasdb/ptobject/EncodingUtilsTest.java
@@ -19,15 +19,6 @@
import static org.junit.Assert.assertEquals;
import static org.junit.Assert.assertTrue;
-import java.math.BigInteger;
-import java.util.Arrays;
-import java.util.List;
-import java.util.Map;
-import java.util.Random;
-import java.util.SortedMap;
-
-import org.junit.Test;
-
import com.google.common.collect.ImmutableList;
import com.google.common.collect.Lists;
import com.google.common.collect.Maps;
@@ -37,6 +28,13 @@
import com.palantir.atlasdb.ptobject.EncodingUtils.EncodingType;
import com.palantir.atlasdb.table.description.ValueType;
import com.palantir.util.crypto.Sha256Hash;
+import java.math.BigInteger;
+import java.util.Arrays;
+import java.util.List;
+import java.util.Map;
+import java.util.Random;
+import java.util.SortedMap;
+import org.junit.Test;
@SuppressWarnings("checkstyle:all")
public class EncodingUtilsTest {
diff --git a/atlasdb-tests-shared/src/test/java/com/palantir/atlasdb/schema/AbstractTaskCheckpointerTest.java b/atlasdb-tests-shared/src/test/java/com/palantir/atlasdb/schema/AbstractTaskCheckpointerTest.java
index 4abf577273c..49521c95ee2 100644
--- a/atlasdb-tests-shared/src/test/java/com/palantir/atlasdb/schema/AbstractTaskCheckpointerTest.java
+++ b/atlasdb-tests-shared/src/test/java/com/palantir/atlasdb/schema/AbstractTaskCheckpointerTest.java
@@ -15,20 +15,18 @@
*/
package com.palantir.atlasdb.schema;
+import com.google.common.collect.Maps;
+import com.palantir.atlasdb.AtlasDbTestCase;
+import com.palantir.atlasdb.transaction.api.Transaction;
+import com.palantir.atlasdb.transaction.api.TransactionTask;
import java.util.Arrays;
import java.util.Map;
import java.util.Map.Entry;
import java.util.Random;
-
import org.junit.Assert;
import org.junit.Before;
import org.junit.Test;
-import com.google.common.collect.Maps;
-import com.palantir.atlasdb.AtlasDbTestCase;
-import com.palantir.atlasdb.transaction.api.Transaction;
-import com.palantir.atlasdb.transaction.api.TransactionTask;
-
public abstract class AbstractTaskCheckpointerTest extends AtlasDbTestCase {
protected AbstractTaskCheckpointer checkpointer;
diff --git a/atlasdb-tests-shared/src/test/java/com/palantir/atlasdb/schema/NamespaceTest.java b/atlasdb-tests-shared/src/test/java/com/palantir/atlasdb/schema/NamespaceTest.java
index 6418fcd19f9..6e4e990a30d 100644
--- a/atlasdb-tests-shared/src/test/java/com/palantir/atlasdb/schema/NamespaceTest.java
+++ b/atlasdb-tests-shared/src/test/java/com/palantir/atlasdb/schema/NamespaceTest.java
@@ -17,12 +17,10 @@
import static org.junit.Assert.fail;
+import com.palantir.atlasdb.keyvalue.api.Namespace;
import java.util.regex.Pattern;
-
import org.junit.Test;
-import com.palantir.atlasdb.keyvalue.api.Namespace;
-
public class NamespaceTest {
private void expectSuccess(String string, Pattern pattern) {
diff --git a/atlasdb-tests-shared/src/test/java/com/palantir/atlasdb/schema/TableMigratorTest.java b/atlasdb-tests-shared/src/test/java/com/palantir/atlasdb/schema/TableMigratorTest.java
index 47723651248..c58b40977a3 100644
--- a/atlasdb-tests-shared/src/test/java/com/palantir/atlasdb/schema/TableMigratorTest.java
+++ b/atlasdb-tests-shared/src/test/java/com/palantir/atlasdb/schema/TableMigratorTest.java
@@ -15,13 +15,6 @@
*/
package com.palantir.atlasdb.schema;
-import java.util.Map;
-import java.util.Map.Entry;
-
-import org.apache.commons.lang3.mutable.MutableLong;
-import org.junit.Assert;
-import org.junit.Test;
-
import com.google.common.collect.ImmutableMap;
import com.google.common.collect.Iterables;
import com.google.common.collect.Lists;
@@ -51,6 +44,11 @@
import com.palantir.common.base.BatchingVisitable;
import com.palantir.common.concurrent.PTExecutors;
import com.palantir.common.exception.TableMappingNotFoundException;
+import java.util.Map;
+import java.util.Map.Entry;
+import org.apache.commons.lang3.mutable.MutableLong;
+import org.junit.Assert;
+import org.junit.Test;
public class TableMigratorTest extends AtlasDbTestCase {
@Test
diff --git a/atlasdb-tests-shared/src/test/java/com/palantir/atlasdb/schema/indexing/IndexTest.java b/atlasdb-tests-shared/src/test/java/com/palantir/atlasdb/schema/indexing/IndexTest.java
index b70cd3a6ff8..3891193e1ad 100644
--- a/atlasdb-tests-shared/src/test/java/com/palantir/atlasdb/schema/indexing/IndexTest.java
+++ b/atlasdb-tests-shared/src/test/java/com/palantir/atlasdb/schema/indexing/IndexTest.java
@@ -17,13 +17,6 @@
import static org.junit.Assert.assertEquals;
-import java.util.Arrays;
-import java.util.List;
-
-import org.junit.Assert;
-import org.junit.Before;
-import org.junit.Test;
-
import com.google.common.collect.HashMultimap;
import com.google.common.collect.Iterables;
import com.google.common.collect.Multimap;
@@ -44,6 +37,11 @@
import com.palantir.atlasdb.table.description.Schemas;
import com.palantir.atlasdb.table.description.ValueType;
import com.palantir.atlasdb.transaction.api.RuntimeTransactionTask;
+import java.util.Arrays;
+import java.util.List;
+import org.junit.Assert;
+import org.junit.Before;
+import org.junit.Test;
public class IndexTest extends AtlasDbTestCase {
diff --git a/atlasdb-tests-shared/src/test/java/com/palantir/atlasdb/schema/indexing/IndexTestSchema.java b/atlasdb-tests-shared/src/test/java/com/palantir/atlasdb/schema/indexing/IndexTestSchema.java
index 496115b9c6b..8c1113fe228 100644
--- a/atlasdb-tests-shared/src/test/java/com/palantir/atlasdb/schema/indexing/IndexTestSchema.java
+++ b/atlasdb-tests-shared/src/test/java/com/palantir/atlasdb/schema/indexing/IndexTestSchema.java
@@ -15,8 +15,6 @@
*/
package com.palantir.atlasdb.schema.indexing;
-import java.io.File;
-
import com.palantir.atlasdb.keyvalue.api.Namespace;
import com.palantir.atlasdb.protos.generated.TableMetadataPersistence.ValueByteOrder;
import com.palantir.atlasdb.schema.AtlasSchema;
@@ -26,6 +24,7 @@
import com.palantir.atlasdb.table.description.Schema;
import com.palantir.atlasdb.table.description.TableDefinition;
import com.palantir.atlasdb.table.description.ValueType;
+import java.io.File;
public class IndexTestSchema implements AtlasSchema {
public static final AtlasSchema INSTANCE = new IndexTestSchema();
diff --git a/atlasdb-tests-shared/src/test/java/com/palantir/atlasdb/schema/stream/StreamTest.java b/atlasdb-tests-shared/src/test/java/com/palantir/atlasdb/schema/stream/StreamTest.java
index 97282078e89..bdbe7d373bf 100644
--- a/atlasdb-tests-shared/src/test/java/com/palantir/atlasdb/schema/stream/StreamTest.java
+++ b/atlasdb-tests-shared/src/test/java/com/palantir/atlasdb/schema/stream/StreamTest.java
@@ -27,31 +27,6 @@
import static org.junit.Assert.assertTrue;
import static org.junit.Assert.fail;
-import java.io.ByteArrayInputStream;
-import java.io.File;
-import java.io.IOException;
-import java.io.InputStream;
-import java.util.Arrays;
-import java.util.Collection;
-import java.util.Map;
-import java.util.Optional;
-import java.util.Random;
-import java.util.Set;
-import java.util.concurrent.CountDownLatch;
-import java.util.concurrent.ExecutorService;
-import java.util.concurrent.Future;
-import java.util.concurrent.atomic.AtomicBoolean;
-
-import org.apache.commons.io.FileUtils;
-import org.apache.commons.io.IOUtils;
-import org.apache.commons.io.output.ByteArrayOutputStream;
-import org.junit.Before;
-import org.junit.Rule;
-import org.junit.Test;
-import org.junit.rules.TemporaryFolder;
-import org.junit.runner.RunWith;
-import org.junit.runners.Parameterized;
-
import com.google.common.base.Throwables;
import com.google.common.collect.ImmutableList;
import com.google.common.collect.ImmutableMap;
@@ -94,6 +69,29 @@
import com.palantir.common.io.ForwardingInputStream;
import com.palantir.util.Pair;
import com.palantir.util.crypto.Sha256Hash;
+import java.io.ByteArrayInputStream;
+import java.io.File;
+import java.io.IOException;
+import java.io.InputStream;
+import java.util.Arrays;
+import java.util.Collection;
+import java.util.Map;
+import java.util.Optional;
+import java.util.Random;
+import java.util.Set;
+import java.util.concurrent.CountDownLatch;
+import java.util.concurrent.ExecutorService;
+import java.util.concurrent.Future;
+import java.util.concurrent.atomic.AtomicBoolean;
+import org.apache.commons.io.FileUtils;
+import org.apache.commons.io.IOUtils;
+import org.apache.commons.io.output.ByteArrayOutputStream;
+import org.junit.Before;
+import org.junit.Rule;
+import org.junit.Test;
+import org.junit.rules.TemporaryFolder;
+import org.junit.runner.RunWith;
+import org.junit.runners.Parameterized;
@RunWith(Parameterized.class)
public class StreamTest extends AtlasDbTestCase {
diff --git a/atlasdb-tests-shared/src/test/java/com/palantir/atlasdb/schema/stream/StreamTestSchema.java b/atlasdb-tests-shared/src/test/java/com/palantir/atlasdb/schema/stream/StreamTestSchema.java
index e1f98902e49..52f5b380ac7 100644
--- a/atlasdb-tests-shared/src/test/java/com/palantir/atlasdb/schema/stream/StreamTestSchema.java
+++ b/atlasdb-tests-shared/src/test/java/com/palantir/atlasdb/schema/stream/StreamTestSchema.java
@@ -15,14 +15,13 @@
*/
package com.palantir.atlasdb.schema.stream;
-import java.io.File;
-
import com.palantir.atlasdb.keyvalue.api.Namespace;
import com.palantir.atlasdb.schema.AtlasSchema;
import com.palantir.atlasdb.table.description.OptionalType;
import com.palantir.atlasdb.table.description.Schema;
import com.palantir.atlasdb.table.description.TableDefinition;
import com.palantir.atlasdb.table.description.ValueType;
+import java.io.File;
public class StreamTestSchema implements AtlasSchema {
public static final AtlasSchema INSTANCE = new StreamTestSchema();
diff --git a/atlasdb-tests-shared/src/test/java/com/palantir/atlasdb/sweep/priority/SweepPriorityStoreTest.java b/atlasdb-tests-shared/src/test/java/com/palantir/atlasdb/sweep/priority/SweepPriorityStoreTest.java
index f4d267b6421..559bf046dc3 100644
--- a/atlasdb-tests-shared/src/test/java/com/palantir/atlasdb/sweep/priority/SweepPriorityStoreTest.java
+++ b/atlasdb-tests-shared/src/test/java/com/palantir/atlasdb/sweep/priority/SweepPriorityStoreTest.java
@@ -17,14 +17,6 @@
import static org.assertj.core.api.Assertions.assertThat;
-import java.util.OptionalLong;
-import java.util.concurrent.ExecutorService;
-
-import org.junit.After;
-import org.junit.Assert;
-import org.junit.Before;
-import org.junit.Test;
-
import com.google.common.collect.ImmutableList;
import com.google.common.collect.ImmutableSet;
import com.palantir.atlasdb.keyvalue.api.KeyValueService;
@@ -34,6 +26,12 @@
import com.palantir.atlasdb.sweep.SweepTestUtils;
import com.palantir.atlasdb.transaction.api.TransactionManager;
import com.palantir.common.concurrent.PTExecutors;
+import java.util.OptionalLong;
+import java.util.concurrent.ExecutorService;
+import org.junit.After;
+import org.junit.Assert;
+import org.junit.Before;
+import org.junit.Test;
public class SweepPriorityStoreTest {
private ExecutorService exec;
diff --git a/atlasdb-tests-shared/src/test/java/com/palantir/atlasdb/sweep/progress/SweepProgressStoreTest.java b/atlasdb-tests-shared/src/test/java/com/palantir/atlasdb/sweep/progress/SweepProgressStoreTest.java
index 5247da203b3..47fd796c05a 100644
--- a/atlasdb-tests-shared/src/test/java/com/palantir/atlasdb/sweep/progress/SweepProgressStoreTest.java
+++ b/atlasdb-tests-shared/src/test/java/com/palantir/atlasdb/sweep/progress/SweepProgressStoreTest.java
@@ -15,9 +15,8 @@
*/
package com.palantir.atlasdb.sweep.progress;
-import org.junit.ClassRule;
-
import com.palantir.atlasdb.keyvalue.impl.TestResourceManager;
+import org.junit.ClassRule;
public class SweepProgressStoreTest extends AbstractSweepProgressStoreTest {
@ClassRule
diff --git a/atlasdb-tests-shared/src/test/java/com/palantir/atlasdb/sweep/queue/TargetedSweepTest.java b/atlasdb-tests-shared/src/test/java/com/palantir/atlasdb/sweep/queue/TargetedSweepTest.java
index b5ed9bd84e6..5e2e70cb848 100644
--- a/atlasdb-tests-shared/src/test/java/com/palantir/atlasdb/sweep/queue/TargetedSweepTest.java
+++ b/atlasdb-tests-shared/src/test/java/com/palantir/atlasdb/sweep/queue/TargetedSweepTest.java
@@ -22,15 +22,6 @@
import static org.mockito.Mockito.times;
import static org.mockito.Mockito.verify;
-import java.util.List;
-import java.util.concurrent.atomic.AtomicLong;
-import java.util.stream.Collectors;
-import java.util.stream.Stream;
-
-import org.junit.Before;
-import org.junit.Test;
-import org.mockito.ArgumentCaptor;
-
import com.google.common.collect.ImmutableList;
import com.google.common.collect.ImmutableMap;
import com.google.common.collect.ImmutableSet;
@@ -46,6 +37,13 @@
import com.palantir.atlasdb.transaction.api.PreCommitCondition;
import com.palantir.atlasdb.transaction.api.Transaction;
import com.palantir.atlasdb.transaction.api.TransactionConflictException;
+import java.util.List;
+import java.util.concurrent.atomic.AtomicLong;
+import java.util.stream.Collectors;
+import java.util.stream.Stream;
+import org.junit.Before;
+import org.junit.Test;
+import org.mockito.ArgumentCaptor;
public class TargetedSweepTest extends AtlasDbTestCase {
private static final TableReference TABLE_CONS = TableReference.createFromFullyQualifiedName("test.1");
diff --git a/atlasdb-tests-shared/src/test/java/com/palantir/atlasdb/transaction/impl/CachingTransactionTest.java b/atlasdb-tests-shared/src/test/java/com/palantir/atlasdb/transaction/impl/CachingTransactionTest.java
index 0cc5116f542..729c0686de6 100644
--- a/atlasdb-tests-shared/src/test/java/com/palantir/atlasdb/transaction/impl/CachingTransactionTest.java
+++ b/atlasdb-tests-shared/src/test/java/com/palantir/atlasdb/transaction/impl/CachingTransactionTest.java
@@ -18,6 +18,18 @@
import static org.junit.Assert.assertEquals;
+import com.google.common.collect.ImmutableList;
+import com.google.common.collect.ImmutableMap;
+import com.google.common.collect.ImmutableSet;
+import com.google.common.collect.ImmutableSortedMap;
+import com.google.common.collect.ImmutableSortedSet;
+import com.google.common.util.concurrent.Futures;
+import com.palantir.atlasdb.encoding.PtBytes;
+import com.palantir.atlasdb.keyvalue.api.Cell;
+import com.palantir.atlasdb.keyvalue.api.ColumnSelection;
+import com.palantir.atlasdb.keyvalue.api.RowResult;
+import com.palantir.atlasdb.keyvalue.api.TableReference;
+import com.palantir.atlasdb.transaction.api.Transaction;
import java.util.Arrays;
import java.util.Collection;
import java.util.Map;
@@ -26,26 +38,12 @@
import java.util.function.BiFunction;
import java.util.function.Function;
import java.util.function.UnaryOperator;
-
import org.jmock.Expectations;
import org.jmock.Mockery;
import org.junit.Test;
import org.junit.runner.RunWith;
import org.junit.runners.Parameterized;
-import com.google.common.collect.ImmutableList;
-import com.google.common.collect.ImmutableMap;
-import com.google.common.collect.ImmutableSet;
-import com.google.common.collect.ImmutableSortedMap;
-import com.google.common.collect.ImmutableSortedSet;
-import com.google.common.util.concurrent.Futures;
-import com.palantir.atlasdb.encoding.PtBytes;
-import com.palantir.atlasdb.keyvalue.api.Cell;
-import com.palantir.atlasdb.keyvalue.api.ColumnSelection;
-import com.palantir.atlasdb.keyvalue.api.RowResult;
-import com.palantir.atlasdb.keyvalue.api.TableReference;
-import com.palantir.atlasdb.transaction.api.Transaction;
-
@RunWith(Parameterized.class)
public class CachingTransactionTest {
private static final byte[] ROW_BYTES = "row".getBytes();
diff --git a/atlasdb-tests-shared/src/test/java/com/palantir/atlasdb/transaction/impl/SnapshotTransactionTest.java b/atlasdb-tests-shared/src/test/java/com/palantir/atlasdb/transaction/impl/SnapshotTransactionTest.java
index 39490a387b4..70e82ac1fa5 100644
--- a/atlasdb-tests-shared/src/test/java/com/palantir/atlasdb/transaction/impl/SnapshotTransactionTest.java
+++ b/atlasdb-tests-shared/src/test/java/com/palantir/atlasdb/transaction/impl/SnapshotTransactionTest.java
@@ -36,44 +36,6 @@
import static org.mockito.Mockito.verify;
import static org.mockito.Mockito.when;
-import java.math.BigInteger;
-import java.util.Arrays;
-import java.util.Collection;
-import java.util.Collections;
-import java.util.Iterator;
-import java.util.List;
-import java.util.Map;
-import java.util.Optional;
-import java.util.Random;
-import java.util.SortedMap;
-import java.util.concurrent.Callable;
-import java.util.concurrent.CompletionService;
-import java.util.concurrent.ExecutionException;
-import java.util.concurrent.ExecutorCompletionService;
-import java.util.concurrent.ExecutorService;
-import java.util.concurrent.Executors;
-import java.util.concurrent.Future;
-import java.util.concurrent.ScheduledExecutorService;
-import java.util.concurrent.TimeUnit;
-import java.util.function.Supplier;
-import java.util.stream.Collectors;
-
-import org.apache.commons.lang3.mutable.MutableInt;
-import org.apache.commons.lang3.mutable.MutableLong;
-import org.apache.commons.lang3.tuple.Pair;
-import org.hamcrest.Matchers;
-import org.jmock.Expectations;
-import org.jmock.Mockery;
-import org.jmock.Sequence;
-import org.jmock.lib.concurrent.DeterministicScheduler;
-import org.jmock.lib.concurrent.Synchroniser;
-import org.junit.Assert;
-import org.junit.Before;
-import org.junit.Ignore;
-import org.junit.Test;
-import org.junit.runner.RunWith;
-import org.junit.runners.Parameterized;
-
import com.google.common.base.Joiner;
import com.google.common.base.Suppliers;
import com.google.common.collect.ImmutableList;
@@ -147,6 +109,42 @@
import com.palantir.lock.v2.LockImmutableTimestampResponse;
import com.palantir.lock.v2.TimelockService;
import com.palantir.timestamp.TimestampService;
+import java.math.BigInteger;
+import java.util.Arrays;
+import java.util.Collection;
+import java.util.Collections;
+import java.util.Iterator;
+import java.util.List;
+import java.util.Map;
+import java.util.Optional;
+import java.util.Random;
+import java.util.SortedMap;
+import java.util.concurrent.Callable;
+import java.util.concurrent.CompletionService;
+import java.util.concurrent.ExecutionException;
+import java.util.concurrent.ExecutorCompletionService;
+import java.util.concurrent.ExecutorService;
+import java.util.concurrent.Executors;
+import java.util.concurrent.Future;
+import java.util.concurrent.ScheduledExecutorService;
+import java.util.concurrent.TimeUnit;
+import java.util.function.Supplier;
+import java.util.stream.Collectors;
+import org.apache.commons.lang3.mutable.MutableInt;
+import org.apache.commons.lang3.mutable.MutableLong;
+import org.apache.commons.lang3.tuple.Pair;
+import org.hamcrest.Matchers;
+import org.jmock.Expectations;
+import org.jmock.Mockery;
+import org.jmock.Sequence;
+import org.jmock.lib.concurrent.DeterministicScheduler;
+import org.jmock.lib.concurrent.Synchroniser;
+import org.junit.Assert;
+import org.junit.Before;
+import org.junit.Ignore;
+import org.junit.Test;
+import org.junit.runner.RunWith;
+import org.junit.runners.Parameterized;
@SuppressWarnings("checkstyle:all")
@RunWith(Parameterized.class)
diff --git a/atlasdb-tests-shared/src/test/java/com/palantir/atlasdb/transaction/impl/TransactionManagerTest.java b/atlasdb-tests-shared/src/test/java/com/palantir/atlasdb/transaction/impl/TransactionManagerTest.java
index 172356aae50..988d259459d 100644
--- a/atlasdb-tests-shared/src/test/java/com/palantir/atlasdb/transaction/impl/TransactionManagerTest.java
+++ b/atlasdb-tests-shared/src/test/java/com/palantir/atlasdb/transaction/impl/TransactionManagerTest.java
@@ -23,11 +23,6 @@
import static org.mockito.Mockito.verifyNoMoreInteractions;
import static org.mockito.Mockito.when;
-import java.util.UUID;
-
-import org.junit.ClassRule;
-import org.junit.Test;
-
import com.google.common.util.concurrent.MoreExecutors;
import com.palantir.atlasdb.cache.DefaultTimestampCache;
import com.palantir.atlasdb.cleaner.NoOpCleaner;
@@ -51,6 +46,9 @@
import com.palantir.lock.v2.TimestampAndPartition;
import com.palantir.timestamp.TimestampManagementService;
import com.palantir.timestamp.TimestampService;
+import java.util.UUID;
+import org.junit.ClassRule;
+import org.junit.Test;
public class TransactionManagerTest extends TransactionTestSetup {
@ClassRule
diff --git a/atlasdb-tests-shared/src/test/java/com/palantir/atlasdb/transaction/impl/metrics/TransactionOutcomeMetricsTest.java b/atlasdb-tests-shared/src/test/java/com/palantir/atlasdb/transaction/impl/metrics/TransactionOutcomeMetricsTest.java
index 0fb45496429..9aed911b837 100644
--- a/atlasdb-tests-shared/src/test/java/com/palantir/atlasdb/transaction/impl/metrics/TransactionOutcomeMetricsTest.java
+++ b/atlasdb-tests-shared/src/test/java/com/palantir/atlasdb/transaction/impl/metrics/TransactionOutcomeMetricsTest.java
@@ -17,12 +17,6 @@
import static com.palantir.atlasdb.transaction.impl.metrics.TransactionOutcomeMetricsAssert.assertThat;
-import java.util.Map;
-import java.util.Set;
-import java.util.stream.IntStream;
-
-import org.junit.Test;
-
import com.codahale.metrics.MetricRegistry;
import com.google.common.collect.ImmutableMap;
import com.google.common.collect.ImmutableSet;
@@ -31,6 +25,10 @@
import com.palantir.atlasdb.util.MetricsManager;
import com.palantir.tritium.metrics.registry.DefaultTaggedMetricRegistry;
import com.palantir.tritium.metrics.registry.TaggedMetricRegistry;
+import java.util.Map;
+import java.util.Set;
+import java.util.stream.IntStream;
+import org.junit.Test;
public class TransactionOutcomeMetricsTest {
private static final Namespace NAMESPACE = Namespace.DEFAULT_NAMESPACE;
diff --git a/build.gradle b/build.gradle
index e5aef39d40a..7fee585eaaa 100644
--- a/build.gradle
+++ b/build.gradle
@@ -16,10 +16,9 @@ buildscript {
classpath 'com.jfrog.bintray.gradle:gradle-bintray-plugin:1.8.4'
classpath 'org.jfrog.buildinfo:build-info-extractor-gradle:4.9.8'
classpath 'gradle.plugin.com.palantir.gradle.docker:gradle-docker:0.22.1'
- classpath 'com.palantir.baseline:gradle-baseline-java:1.8.2'
+ classpath 'com.palantir.baseline:gradle-baseline-java:2.49.1'
classpath 'gradle.plugin.com.palantir.gradle.gitversion:gradle-git-version:0.11.0'
classpath 'com.palantir.sls-packaging:gradle-sls-packaging:3.8.1'
- classpath 'com.netflix.nebula:nebula-dependency-recommender:8.0.0'
classpath 'com.netflix.nebula:nebula-publishing-plugin:14.1.1'
classpath 'gradle.plugin.com.hierynomus.gradle.plugins:license-gradle-plugin:0.15.0'
classpath 'gradle.plugin.org.inferred:gradle-processors:3.2.0'
@@ -28,7 +27,6 @@ buildscript {
plugins {
id 'com.github.johnrengelman.shadow' version '5.1.0'
- id 'com.palantir.configuration-resolver' version '0.4.0'
id 'org.unbroken-dome.test-sets' version '2.1.1'
id 'com.google.protobuf' version '0.8.10'
}
@@ -39,6 +37,8 @@ repositories {
url 'https://dl.bintray.com/palantir/releases/'
}
}
+apply plugin: 'com.palantir.baseline'
+apply plugin: 'com.palantir.baseline'
apply plugin: 'com.palantir.baseline-config'
apply plugin: 'com.palantir.baseline-circleci'
apply plugin: 'com.palantir.git-version'
@@ -58,7 +58,6 @@ task printLastVersion {
allprojects {
apply plugin: 'java-library'
- apply plugin: 'nebula.dependency-recommender'
apply plugin: 'org.inferred.processors' // installs the "processor" configuration needed for baseline-error-prone
// temporary until this is merged/fixed inside gradle-processors
@@ -88,7 +87,6 @@ allprojects {
}
subprojects {
- apply plugin: 'com.palantir.configuration-resolver'
task allDeps(type: DependencyReportTask) {}
}
diff --git a/cassandra-partitioner/src/main/java/com/palantir/atlasdb/keyvalue/cassandra/dht/AtlasDbOrderedPartitioner.java b/cassandra-partitioner/src/main/java/com/palantir/atlasdb/keyvalue/cassandra/dht/AtlasDbOrderedPartitioner.java
index 30af0d390f3..c5a1a59ed01 100644
--- a/cassandra-partitioner/src/main/java/com/palantir/atlasdb/keyvalue/cassandra/dht/AtlasDbOrderedPartitioner.java
+++ b/cassandra-partitioner/src/main/java/com/palantir/atlasdb/keyvalue/cassandra/dht/AtlasDbOrderedPartitioner.java
@@ -15,6 +15,8 @@
*/
package com.palantir.atlasdb.keyvalue.cassandra.dht;
+import com.google.common.collect.ImmutableSet;
+import com.google.common.collect.Maps;
import java.math.BigInteger;
import java.nio.ByteBuffer;
import java.security.SecureRandom;
@@ -22,7 +24,6 @@
import java.util.Map;
import java.util.Random;
import java.util.concurrent.atomic.AtomicInteger;
-
import org.apache.cassandra.dht.ByteOrderedPartitioner;
import org.apache.cassandra.dht.IPartitioner;
import org.apache.cassandra.dht.Token;
@@ -33,9 +34,6 @@
import org.apache.cassandra.utils.Pair;
import org.apache.commons.lang3.ArrayUtils;
-import com.google.common.collect.ImmutableSet;
-import com.google.common.collect.Maps;
-
public class AtlasDbOrderedPartitioner extends ByteOrderedPartitioner {
private final Random r = new SecureRandom();
private final AtomicInteger indexCounter = new AtomicInteger(r.nextInt(6));
diff --git a/commons-annotations/src/main/java/com/palantir/atlasdb/metrics/Timed.java b/commons-annotations/src/main/java/com/palantir/atlasdb/metrics/Timed.java
index 5bdb4bd882f..1e9ef8199a5 100644
--- a/commons-annotations/src/main/java/com/palantir/atlasdb/metrics/Timed.java
+++ b/commons-annotations/src/main/java/com/palantir/atlasdb/metrics/Timed.java
@@ -30,4 +30,3 @@
@Target(ElementType.METHOD)
public @interface Timed {
}
-
diff --git a/commons-annotations/src/main/java/com/palantir/common/annotation/Immutable.java b/commons-annotations/src/main/java/com/palantir/common/annotation/Immutable.java
index 440e7523b40..52276252b0b 100644
--- a/commons-annotations/src/main/java/com/palantir/common/annotation/Immutable.java
+++ b/commons-annotations/src/main/java/com/palantir/common/annotation/Immutable.java
@@ -31,4 +31,3 @@
public @interface Immutable {
// marker annotation
}
-
diff --git a/commons-api/src/main/java/com/palantir/util/VersionStrings.java b/commons-api/src/main/java/com/palantir/util/VersionStrings.java
index 9e4fd876ad9..398ad3674a6 100644
--- a/commons-api/src/main/java/com/palantir/util/VersionStrings.java
+++ b/commons-api/src/main/java/com/palantir/util/VersionStrings.java
@@ -15,9 +15,8 @@
*/
package com.palantir.util;
-import java.util.Iterator;
-
import com.google.common.base.Splitter;
+import java.util.Iterator;
public final class VersionStrings {
private VersionStrings() {
diff --git a/commons-api/src/main/java/com/palantir/util/jmx/AbstractOperationStats.java b/commons-api/src/main/java/com/palantir/util/jmx/AbstractOperationStats.java
index 2daf25f193c..54dc4a21b17 100644
--- a/commons-api/src/main/java/com/palantir/util/jmx/AbstractOperationStats.java
+++ b/commons-api/src/main/java/com/palantir/util/jmx/AbstractOperationStats.java
@@ -15,15 +15,13 @@
*/
package com.palantir.util.jmx;
+import com.palantir.annotations.PgNotExtendableApi;
+import com.palantir.annotations.PgPublicApi;
import java.util.List;
import java.util.concurrent.CopyOnWriteArrayList;
import java.util.concurrent.atomic.AtomicLong;
-
import javax.management.MXBean;
-import com.palantir.annotations.PgNotExtendableApi;
-import com.palantir.annotations.PgPublicApi;
-
/*
* TODO (carrino) consider abstracting this a little bit further so that this
* isn't time-specific (e.g., this class could be useful for tracking statistics
diff --git a/commons-api/src/main/java/com/palantir/util/sql/SqlStats.java b/commons-api/src/main/java/com/palantir/util/sql/SqlStats.java
index 7af0bf29e8b..430471c9a2c 100644
--- a/commons-api/src/main/java/com/palantir/util/sql/SqlStats.java
+++ b/commons-api/src/main/java/com/palantir/util/sql/SqlStats.java
@@ -15,6 +15,8 @@
*/
package com.palantir.util.sql;
+import com.palantir.util.JMXUtils;
+import com.palantir.util.Pair;
import java.util.ArrayList;
import java.util.Collections;
import java.util.HashMap;
@@ -22,9 +24,6 @@
import java.util.Map;
import java.util.concurrent.atomic.AtomicLong;
-import com.palantir.util.JMXUtils;
-import com.palantir.util.Pair;
-
/**
* MXBean which tracks statistics for all SQL queries made by the server.
*/
@@ -173,4 +172,3 @@ public void incrementClearTempTableByTruncate() {
clearTempTableByTruncateCount.incrementAndGet();
}
}
-
diff --git a/commons-db/src/main/java/com/palantir/db/oracle/JdbcHandler.java b/commons-db/src/main/java/com/palantir/db/oracle/JdbcHandler.java
index 8e5d6955fe2..69787a39387 100644
--- a/commons-db/src/main/java/com/palantir/db/oracle/JdbcHandler.java
+++ b/commons-db/src/main/java/com/palantir/db/oracle/JdbcHandler.java
@@ -15,14 +15,13 @@
*/
package com.palantir.db.oracle;
+import com.fasterxml.jackson.annotation.JsonTypeInfo;
import java.io.OutputStream;
import java.sql.Blob;
import java.sql.Connection;
import java.sql.SQLException;
import java.util.List;
-import com.fasterxml.jackson.annotation.JsonTypeInfo;
-
/**
* JdbcHandler allows Oracle dependent logic to be injected into the SQL
* dependent classes that support both Legacy DB and AtlasDB's Dbkvs
diff --git a/commons-db/src/main/java/com/palantir/nexus/db/ConnectionSupplier.java b/commons-db/src/main/java/com/palantir/nexus/db/ConnectionSupplier.java
index a103b0694c6..c655fa87f34 100644
--- a/commons-db/src/main/java/com/palantir/nexus/db/ConnectionSupplier.java
+++ b/commons-db/src/main/java/com/palantir/nexus/db/ConnectionSupplier.java
@@ -15,12 +15,11 @@
*/
package com.palantir.nexus.db;
+import com.palantir.exception.PalantirSqlException;
import java.io.Closeable;
import java.sql.Connection;
import java.util.function.Supplier;
-import com.palantir.exception.PalantirSqlException;
-
public interface ConnectionSupplier extends Supplier, Closeable {
/**
diff --git a/commons-db/src/main/java/com/palantir/nexus/db/DBType.java b/commons-db/src/main/java/com/palantir/nexus/db/DBType.java
index 405fc216a23..2e7c9dac2f5 100644
--- a/commons-db/src/main/java/com/palantir/nexus/db/DBType.java
+++ b/commons-db/src/main/java/com/palantir/nexus/db/DBType.java
@@ -15,13 +15,11 @@
*/
package com.palantir.nexus.db;
-import java.sql.Connection;
-
-import javax.annotation.Nullable;
-
import com.palantir.exception.PalantirSqlException;
import com.palantir.logsafe.exceptions.SafeRuntimeException;
import com.palantir.sql.Connections;
+import java.sql.Connection;
+import javax.annotation.Nullable;
/**
* Extensibility point for adding more type of databases.
diff --git a/commons-db/src/main/java/com/palantir/nexus/db/ReentrantConnectionSupplier.java b/commons-db/src/main/java/com/palantir/nexus/db/ReentrantConnectionSupplier.java
index 79b54b29a1a..e597df7dc56 100644
--- a/commons-db/src/main/java/com/palantir/nexus/db/ReentrantConnectionSupplier.java
+++ b/commons-db/src/main/java/com/palantir/nexus/db/ReentrantConnectionSupplier.java
@@ -15,11 +15,10 @@
*/
package com.palantir.nexus.db;
+import com.palantir.exception.PalantirSqlException;
import java.io.Closeable;
import java.sql.Connection;
-import com.palantir.exception.PalantirSqlException;
-
public interface ReentrantConnectionSupplier extends ConnectionSupplier, Closeable {
/**
diff --git a/commons-db/src/main/java/com/palantir/nexus/db/ResourceCreationLocation.java b/commons-db/src/main/java/com/palantir/nexus/db/ResourceCreationLocation.java
index 92fd1dc9845..5f84a820dbd 100644
--- a/commons-db/src/main/java/com/palantir/nexus/db/ResourceCreationLocation.java
+++ b/commons-db/src/main/java/com/palantir/nexus/db/ResourceCreationLocation.java
@@ -15,12 +15,10 @@
*/
package com.palantir.nexus.db;
+import com.google.common.base.MoreObjects;
import java.io.Serializable;
-
import org.joda.time.format.ISODateTimeFormat;
-import com.google.common.base.MoreObjects;
-
/**
* A RuntimeException that represents the creation location of a resource (e.g. JDBC Connection or
* ResultSet) to help identify potential resource leaks. This instance tracks the stack trace of
diff --git a/commons-db/src/main/java/com/palantir/nexus/db/ThreadConfinedProxy.java b/commons-db/src/main/java/com/palantir/nexus/db/ThreadConfinedProxy.java
index b2a9b5a0ef6..960d258a737 100644
--- a/commons-db/src/main/java/com/palantir/nexus/db/ThreadConfinedProxy.java
+++ b/commons-db/src/main/java/com/palantir/nexus/db/ThreadConfinedProxy.java
@@ -15,24 +15,21 @@
*/
package com.palantir.nexus.db;
+import com.google.common.base.Throwables;
+import com.google.common.reflect.AbstractInvocationHandler;
+import com.palantir.common.proxy.DelegatingInvocationHandler;
+import com.palantir.logsafe.Preconditions;
+import com.palantir.util.AssertUtils;
import java.lang.reflect.InvocationHandler;
import java.lang.reflect.InvocationTargetException;
import java.lang.reflect.Method;
import java.lang.reflect.Proxy;
import java.util.concurrent.Callable;
-
import javax.annotation.concurrent.GuardedBy;
-
import org.apache.commons.lang3.Validate;
import org.slf4j.Logger;
import org.slf4j.LoggerFactory;
-import com.google.common.base.Throwables;
-import com.google.common.reflect.AbstractInvocationHandler;
-import com.palantir.common.proxy.DelegatingInvocationHandler;
-import com.palantir.logsafe.Preconditions;
-import com.palantir.util.AssertUtils;
-
/**
* Dynamic Proxy for confining an object to a particular thread, but allowing explicit handoff.
*
diff --git a/commons-db/src/main/java/com/palantir/nexus/db/monitoring/profiler/SimpleSqlProfiler.java b/commons-db/src/main/java/com/palantir/nexus/db/monitoring/profiler/SimpleSqlProfiler.java
index 4db7f655891..d46d942c419 100644
--- a/commons-db/src/main/java/com/palantir/nexus/db/monitoring/profiler/SimpleSqlProfiler.java
+++ b/commons-db/src/main/java/com/palantir/nexus/db/monitoring/profiler/SimpleSqlProfiler.java
@@ -15,21 +15,18 @@
*/
package com.palantir.nexus.db.monitoring.profiler;
-import java.util.Collection;
-import java.util.concurrent.ConcurrentMap;
-import java.util.concurrent.CopyOnWriteArrayList;
-
-import javax.annotation.Nullable;
-
-import org.slf4j.Logger;
-import org.slf4j.LoggerFactory;
-
import com.google.common.collect.ImmutableList;
import com.google.common.collect.Lists;
import com.google.common.collect.Maps;
import com.palantir.common.concurrent.ExecutorInheritableThreadLocal;
import com.palantir.util.AssertUtils;
import com.palantir.util.sql.SqlCallStats;
+import java.util.Collection;
+import java.util.concurrent.ConcurrentMap;
+import java.util.concurrent.CopyOnWriteArrayList;
+import javax.annotation.Nullable;
+import org.slf4j.Logger;
+import org.slf4j.LoggerFactory;
enum SimpleSqlProfiler implements SqlProfiler {
INSTANCE;
diff --git a/commons-db/src/main/java/com/palantir/nexus/db/monitoring/profiler/SqlProfiler.java b/commons-db/src/main/java/com/palantir/nexus/db/monitoring/profiler/SqlProfiler.java
index de390afa84a..b0cab90624f 100644
--- a/commons-db/src/main/java/com/palantir/nexus/db/monitoring/profiler/SqlProfiler.java
+++ b/commons-db/src/main/java/com/palantir/nexus/db/monitoring/profiler/SqlProfiler.java
@@ -15,10 +15,9 @@
*/
package com.palantir.nexus.db.monitoring.profiler;
-import java.util.Collection;
-
import com.palantir.nexus.db.sql.BasicSQL;
import com.palantir.util.sql.SqlCallStats;
+import java.util.Collection;
/**
* Code extracted from {@link BasicSQL}. This provides one of several competing and overlapping
diff --git a/commons-db/src/main/java/com/palantir/nexus/db/monitoring/timer/CombinedSqlTimer.java b/commons-db/src/main/java/com/palantir/nexus/db/monitoring/timer/CombinedSqlTimer.java
index 3758a486c82..341df7dee6e 100644
--- a/commons-db/src/main/java/com/palantir/nexus/db/monitoring/timer/CombinedSqlTimer.java
+++ b/commons-db/src/main/java/com/palantir/nexus/db/monitoring/timer/CombinedSqlTimer.java
@@ -15,9 +15,8 @@
*/
package com.palantir.nexus.db.monitoring.timer;
-import java.util.List;
-
import com.google.common.collect.Lists;
+import java.util.List;
final class CombinedSqlTimer implements SqlTimer {
private final List sqlTimers;
diff --git a/commons-db/src/main/java/com/palantir/nexus/db/monitoring/timer/SqlStatsSqlTimer.java b/commons-db/src/main/java/com/palantir/nexus/db/monitoring/timer/SqlStatsSqlTimer.java
index 163c83563bf..4dd70a60e05 100644
--- a/commons-db/src/main/java/com/palantir/nexus/db/monitoring/timer/SqlStatsSqlTimer.java
+++ b/commons-db/src/main/java/com/palantir/nexus/db/monitoring/timer/SqlStatsSqlTimer.java
@@ -15,13 +15,12 @@
*/
package com.palantir.nexus.db.monitoring.timer;
-import org.slf4j.Logger;
-import org.slf4j.LoggerFactory;
-
import com.palantir.nexus.db.sql.monitoring.logger.SqlLoggers;
import com.palantir.util.jmx.OperationTimer;
import com.palantir.util.jmx.OperationTimer.TimingState;
import com.palantir.util.timer.LoggingOperationTimer;
+import org.slf4j.Logger;
+import org.slf4j.LoggerFactory;
final class SqlStatsSqlTimer implements SqlTimer {
private final Logger logger = SqlLoggers.LOGGER;
diff --git a/commons-db/src/main/java/com/palantir/nexus/db/pool/BaseConnectionManager.java b/commons-db/src/main/java/com/palantir/nexus/db/pool/BaseConnectionManager.java
index 9045e977a17..3fffac77387 100644
--- a/commons-db/src/main/java/com/palantir/nexus/db/pool/BaseConnectionManager.java
+++ b/commons-db/src/main/java/com/palantir/nexus/db/pool/BaseConnectionManager.java
@@ -15,11 +15,10 @@
*/
package com.palantir.nexus.db.pool;
+import com.palantir.exception.PalantirSqlException;
import java.sql.Connection;
import java.sql.SQLException;
-import com.palantir.exception.PalantirSqlException;
-
public abstract class BaseConnectionManager implements ConnectionManager {
@Override
public Connection getConnectionUnchecked() {
diff --git a/commons-db/src/main/java/com/palantir/nexus/db/pool/CloseTracking.java b/commons-db/src/main/java/com/palantir/nexus/db/pool/CloseTracking.java
index b66c2f42d7d..123006a0379 100644
--- a/commons-db/src/main/java/com/palantir/nexus/db/pool/CloseTracking.java
+++ b/commons-db/src/main/java/com/palantir/nexus/db/pool/CloseTracking.java
@@ -15,18 +15,16 @@
*/
package com.palantir.nexus.db.pool;
-import java.sql.Connection;
-import java.util.Arrays;
-import java.util.Set;
-
-import org.slf4j.Logger;
-import org.slf4j.LoggerFactory;
-
import com.google.common.base.FinalizableReferenceQueue;
import com.google.common.base.FinalizableWeakReference;
import com.google.common.collect.Maps;
import com.google.common.collect.Sets;
import com.palantir.nexus.db.ResourceCreationLocation;
+import java.sql.Connection;
+import java.util.Arrays;
+import java.util.Set;
+import org.slf4j.Logger;
+import org.slf4j.LoggerFactory;
public class CloseTracking {
private static final Logger log = LoggerFactory.getLogger(CloseTracking.class);
diff --git a/commons-db/src/main/java/com/palantir/nexus/db/pool/ConnectionManager.java b/commons-db/src/main/java/com/palantir/nexus/db/pool/ConnectionManager.java
index d7134fdc3e8..e7aa071c939 100644
--- a/commons-db/src/main/java/com/palantir/nexus/db/pool/ConnectionManager.java
+++ b/commons-db/src/main/java/com/palantir/nexus/db/pool/ConnectionManager.java
@@ -15,11 +15,10 @@
*/
package com.palantir.nexus.db.pool;
+import com.palantir.nexus.db.DBType;
import java.sql.Connection;
import java.sql.SQLException;
-import com.palantir.nexus.db.DBType;
-
/**
* A SQL connection pool that can be flushed.
*
diff --git a/commons-db/src/main/java/com/palantir/nexus/db/pool/ReentrantManagedConnectionSupplier.java b/commons-db/src/main/java/com/palantir/nexus/db/pool/ReentrantManagedConnectionSupplier.java
index 1b1b6574d7a..7eaeb8d3b46 100644
--- a/commons-db/src/main/java/com/palantir/nexus/db/pool/ReentrantManagedConnectionSupplier.java
+++ b/commons-db/src/main/java/com/palantir/nexus/db/pool/ReentrantManagedConnectionSupplier.java
@@ -15,14 +15,12 @@
*/
package com.palantir.nexus.db.pool;
-import java.sql.Connection;
-import java.sql.SQLException;
-
-import javax.annotation.concurrent.ThreadSafe;
-
import com.palantir.exception.PalantirSqlException;
import com.palantir.logsafe.Preconditions;
import com.palantir.nexus.db.ReentrantConnectionSupplier;
+import java.sql.Connection;
+import java.sql.SQLException;
+import javax.annotation.concurrent.ThreadSafe;
@ThreadSafe
public class ReentrantManagedConnectionSupplier implements ReentrantConnectionSupplier {
diff --git a/commons-db/src/main/java/com/palantir/nexus/db/pool/ResourceTypes.java b/commons-db/src/main/java/com/palantir/nexus/db/pool/ResourceTypes.java
index 8cd61a32d52..7c18ef0d044 100644
--- a/commons-db/src/main/java/com/palantir/nexus/db/pool/ResourceTypes.java
+++ b/commons-db/src/main/java/com/palantir/nexus/db/pool/ResourceTypes.java
@@ -15,16 +15,15 @@
*/
package com.palantir.nexus.db.pool;
+import com.google.common.reflect.AbstractInvocationHandler;
+import com.palantir.nexus.db.DBType;
+import com.palantir.proxy.util.ProxyUtils;
import java.lang.reflect.InvocationHandler;
import java.lang.reflect.InvocationTargetException;
import java.lang.reflect.Method;
import java.sql.Connection;
import java.sql.SQLException;
-import com.google.common.reflect.AbstractInvocationHandler;
-import com.palantir.nexus.db.DBType;
-import com.palantir.proxy.util.ProxyUtils;
-
public final class ResourceTypes {
private ResourceTypes() {
// no
diff --git a/commons-db/src/main/java/com/palantir/nexus/db/pool/RetriableTransactions.java b/commons-db/src/main/java/com/palantir/nexus/db/pool/RetriableTransactions.java
index b4586835233..4bb87419666 100644
--- a/commons-db/src/main/java/com/palantir/nexus/db/pool/RetriableTransactions.java
+++ b/commons-db/src/main/java/com/palantir/nexus/db/pool/RetriableTransactions.java
@@ -15,6 +15,10 @@
*/
package com.palantir.nexus.db.pool;
+import com.google.common.cache.CacheBuilder;
+import com.google.common.cache.CacheLoader;
+import com.google.common.cache.LoadingCache;
+import com.palantir.logsafe.Preconditions;
import java.sql.Connection;
import java.sql.PreparedStatement;
import java.sql.ResultSet;
@@ -22,17 +26,10 @@
import java.util.Optional;
import java.util.UUID;
import java.util.concurrent.atomic.AtomicBoolean;
-
import javax.annotation.Nullable;
-
import org.slf4j.Logger;
import org.slf4j.LoggerFactory;
-import com.google.common.cache.CacheBuilder;
-import com.google.common.cache.CacheLoader;
-import com.google.common.cache.LoadingCache;
-import com.palantir.logsafe.Preconditions;
-
public final class RetriableTransactions {
private static final Logger log = LoggerFactory.getLogger(RetriableTransactions.class);
diff --git a/commons-db/src/main/java/com/palantir/nexus/db/pool/RetriableWriteTransaction.java b/commons-db/src/main/java/com/palantir/nexus/db/pool/RetriableWriteTransaction.java
index d7c2363d7e7..fc62e9a9e90 100644
--- a/commons-db/src/main/java/com/palantir/nexus/db/pool/RetriableWriteTransaction.java
+++ b/commons-db/src/main/java/com/palantir/nexus/db/pool/RetriableWriteTransaction.java
@@ -17,7 +17,6 @@
import java.sql.Connection;
import java.sql.SQLException;
-
import javax.annotation.Nullable;
/**
diff --git a/commons-db/src/main/java/com/palantir/nexus/db/sql/AbstractAgnosticResultRow.java b/commons-db/src/main/java/com/palantir/nexus/db/sql/AbstractAgnosticResultRow.java
index b3247b11be3..bc39dd78bd7 100644
--- a/commons-db/src/main/java/com/palantir/nexus/db/sql/AbstractAgnosticResultRow.java
+++ b/commons-db/src/main/java/com/palantir/nexus/db/sql/AbstractAgnosticResultRow.java
@@ -15,20 +15,18 @@
*/
package com.palantir.nexus.db.sql;
+import com.palantir.common.base.Throwables;
+import com.palantir.exception.PalantirSqlException;
+import com.palantir.nexus.db.DBType;
import java.io.ByteArrayOutputStream;
import java.io.IOException;
import java.io.InputStream;
import java.io.InputStreamReader;
import java.sql.SQLException;
import java.util.Map;
-
import org.slf4j.Logger;
import org.slf4j.LoggerFactory;
-import com.palantir.common.base.Throwables;
-import com.palantir.exception.PalantirSqlException;
-import com.palantir.nexus.db.DBType;
-
public abstract class AbstractAgnosticResultRow implements AgnosticResultRow {
private static final Logger log = LoggerFactory.getLogger(AbstractAgnosticResultRow.class);
diff --git a/commons-db/src/main/java/com/palantir/nexus/db/sql/AgnosticLightResultRow.java b/commons-db/src/main/java/com/palantir/nexus/db/sql/AgnosticLightResultRow.java
index fa34bfa49f1..bef604b38ae 100644
--- a/commons-db/src/main/java/com/palantir/nexus/db/sql/AgnosticLightResultRow.java
+++ b/commons-db/src/main/java/com/palantir/nexus/db/sql/AgnosticLightResultRow.java
@@ -16,12 +16,10 @@
package com.palantir.nexus.db.sql;
+import com.palantir.exception.PalantirSqlException;
import java.io.InputStream;
-
import org.joda.time.DateTime;
-import com.palantir.exception.PalantirSqlException;
-
public interface AgnosticLightResultRow extends AgnosticResultRow {
InputStream getBinaryInputStream(final String colname) throws PalantirSqlException;
diff --git a/commons-db/src/main/java/com/palantir/nexus/db/sql/AgnosticLightResultRowImpl.java b/commons-db/src/main/java/com/palantir/nexus/db/sql/AgnosticLightResultRowImpl.java
index b19356dc3bf..2fe31869be7 100644
--- a/commons-db/src/main/java/com/palantir/nexus/db/sql/AgnosticLightResultRowImpl.java
+++ b/commons-db/src/main/java/com/palantir/nexus/db/sql/AgnosticLightResultRowImpl.java
@@ -16,6 +16,11 @@
package com.palantir.nexus.db.sql;
+import com.palantir.exception.PalantirSqlException;
+import com.palantir.nexus.db.DBType;
+import com.palantir.sql.Blobs;
+import com.palantir.sql.Clobs;
+import com.palantir.sql.ResultSets;
import java.io.InputStream;
import java.math.BigDecimal;
import java.sql.Blob;
@@ -23,15 +28,8 @@
import java.sql.ResultSet;
import java.sql.Timestamp;
import java.util.Map;
-
import org.joda.time.DateTime;
-import com.palantir.exception.PalantirSqlException;
-import com.palantir.nexus.db.DBType;
-import com.palantir.sql.Blobs;
-import com.palantir.sql.Clobs;
-import com.palantir.sql.ResultSets;
-
public class AgnosticLightResultRowImpl extends AbstractAgnosticResultRow implements AgnosticLightResultRow {
private ResultSet results;
diff --git a/commons-db/src/main/java/com/palantir/nexus/db/sql/AgnosticLightResultSet.java b/commons-db/src/main/java/com/palantir/nexus/db/sql/AgnosticLightResultSet.java
index 57723e027b6..88afd5b9a53 100644
--- a/commons-db/src/main/java/com/palantir/nexus/db/sql/AgnosticLightResultSet.java
+++ b/commons-db/src/main/java/com/palantir/nexus/db/sql/AgnosticLightResultSet.java
@@ -15,11 +15,10 @@
*/
package com.palantir.nexus.db.sql;
+import com.palantir.common.visitor.Visitor;
import java.io.Closeable;
import java.util.Iterator;
-import com.palantir.common.visitor.Visitor;
-
/**
* This result set only loads one row at a time, and thus provides a
* low-overhead solution for large queries. Read the comments about the iterator before
diff --git a/commons-db/src/main/java/com/palantir/nexus/db/sql/AgnosticLightResultSetImpl.java b/commons-db/src/main/java/com/palantir/nexus/db/sql/AgnosticLightResultSetImpl.java
index c9fde2df020..15577e84230 100644
--- a/commons-db/src/main/java/com/palantir/nexus/db/sql/AgnosticLightResultSetImpl.java
+++ b/commons-db/src/main/java/com/palantir/nexus/db/sql/AgnosticLightResultSetImpl.java
@@ -16,16 +16,6 @@
package com.palantir.nexus.db.sql;
-import java.sql.PreparedStatement;
-import java.sql.ResultSet;
-import java.sql.ResultSetMetaData;
-import java.sql.SQLException;
-import java.util.Iterator;
-import java.util.Map;
-
-import org.slf4j.Logger;
-import org.slf4j.LoggerFactory;
-
import com.palantir.common.base.Throwables;
import com.palantir.common.visitor.Visitor;
import com.palantir.exception.PalantirInterruptedException;
@@ -36,6 +26,14 @@
import com.palantir.nexus.db.sql.BasicSQLString.FinalSQLString;
import com.palantir.sql.ResultSets;
import com.palantir.util.AssertUtils;
+import java.sql.PreparedStatement;
+import java.sql.ResultSet;
+import java.sql.ResultSetMetaData;
+import java.sql.SQLException;
+import java.util.Iterator;
+import java.util.Map;
+import org.slf4j.Logger;
+import org.slf4j.LoggerFactory;
/**
* This result set only loads one row at a time, and thus provides a
@@ -233,4 +231,3 @@ public void visitAndClose(Visitor super AgnosticLightResultRow> visitor) throw
}
}
}
-
diff --git a/commons-db/src/main/java/com/palantir/nexus/db/sql/AgnosticResultRowImpl.java b/commons-db/src/main/java/com/palantir/nexus/db/sql/AgnosticResultRowImpl.java
index faf9ed62d74..4f85f69c9f5 100644
--- a/commons-db/src/main/java/com/palantir/nexus/db/sql/AgnosticResultRowImpl.java
+++ b/commons-db/src/main/java/com/palantir/nexus/db/sql/AgnosticResultRowImpl.java
@@ -16,6 +16,10 @@
package com.palantir.nexus.db.sql;
+import com.palantir.exception.PalantirSqlException;
+import com.palantir.nexus.db.DBType;
+import com.palantir.sql.Blobs;
+import com.palantir.sql.Clobs;
import java.math.BigDecimal;
import java.sql.Blob;
import java.sql.Clob;
@@ -23,11 +27,6 @@
import java.util.List;
import java.util.Map;
-import com.palantir.exception.PalantirSqlException;
-import com.palantir.nexus.db.DBType;
-import com.palantir.sql.Blobs;
-import com.palantir.sql.Clobs;
-
public class AgnosticResultRowImpl extends AbstractAgnosticResultRow {
private List