Standardize on Guava for VisibleForTesting annotation

prestodb · Sep 12, 2024 · 7b68207 · 7b68207
1 parent 7727df8
commit 7b68207
Show file tree

Hide file tree

Showing 4 changed files with 4 additions and 3 deletions.
diff --git a/presto-main/src/main/java/com/facebook/presto/execution/SqlQueryManager.java b/presto-main/src/main/java/com/facebook/presto/execution/SqlQueryManager.java
@@ -34,11 +34,11 @@
 import com.facebook.presto.spi.resourceGroups.ResourceGroupQueryLimits;
 import com.facebook.presto.sql.planner.Plan;
 import com.facebook.presto.version.EmbedVersion;
+import com.google.common.annotations.VisibleForTesting;
 import com.google.common.collect.Ordering;
 import com.google.common.util.concurrent.ListenableFuture;
 import io.airlift.units.DataSize;
 import io.airlift.units.Duration;
-import org.jheaps.annotations.VisibleForTesting;
 import org.weakref.jmx.Flatten;
 import org.weakref.jmx.Managed;
 import org.weakref.jmx.Nested;

diff --git a/presto-main/src/main/java/com/facebook/presto/memory/HighMemoryTaskKiller.java b/presto-main/src/main/java/com/facebook/presto/memory/HighMemoryTaskKiller.java
@@ -22,11 +22,11 @@
 import com.facebook.presto.operator.TaskStats;
 import com.facebook.presto.spi.PrestoException;
 import com.facebook.presto.spi.QueryId;
+import com.google.common.annotations.VisibleForTesting;
 import com.google.common.base.Ticker;
 import com.google.common.collect.ListMultimap;
 import io.airlift.units.DataSize;
 import io.airlift.units.Duration;
-import org.jheaps.annotations.VisibleForTesting;
 
 import javax.annotation.PostConstruct;
 import javax.annotation.PreDestroy;

diff --git a/...rc/main/java/com/facebook/presto/spark/execution/nativeprocess/NativeExecutionModule.java b/...rc/main/java/com/facebook/presto/spark/execution/nativeprocess/NativeExecutionModule.java
@@ -23,12 +23,12 @@
 import com.facebook.presto.spark.execution.shuffle.PrestoSparkShuffleInfoTranslator;
 import com.facebook.presto.spark.execution.task.ForNativeExecutionTask;
 import com.facebook.presto.spark.execution.task.NativeExecutionTaskFactory;
+import com.google.common.annotations.VisibleForTesting;
 import com.google.inject.Binder;
 import com.google.inject.Module;
 import com.google.inject.Scopes;
 import com.google.inject.TypeLiteral;
 import io.airlift.units.Duration;
-import org.jheaps.annotations.VisibleForTesting;
 
 import java.util.Optional;
 

diff --git a/src/checkstyle/presto-checks.xml b/src/checkstyle/presto-checks.xml
@@ -206,6 +206,7 @@
 
         <module name="IllegalImport">
             <property name="illegalPkgs" value="org.weakref.jmx.internal" />
+            <property name="illegalPkgs" value="org.jheaps.annotations" />
             <property name="illegalPkgs" value="jersey.repackaged" />
             <property name="illegalPkgs" value="jdk.nashorn.internal" />
             <property name="illegalPkgs" value="jdk.internal" />