Repository: impala Updated Branches: refs/heads/master af7476d55 -> 96f976534
IMPALA-7823: Clean up unused Java imports Cleans up unused Java imports. Testing: This is only a source-level change. Verified that the FE builds correctly. Ran FE tests. Change-Id: Ic1820a7f9e55dc168510987520949026169f445e Reviewed-on: http://gerrit.cloudera.org:8080/11987 Reviewed-by: Impala Public Jenkins <[email protected]> Tested-by: Impala Public Jenkins <[email protected]> Project: http://git-wip-us.apache.org/repos/asf/impala/repo Commit: http://git-wip-us.apache.org/repos/asf/impala/commit/a39c8472 Tree: http://git-wip-us.apache.org/repos/asf/impala/tree/a39c8472 Diff: http://git-wip-us.apache.org/repos/asf/impala/diff/a39c8472 Branch: refs/heads/master Commit: a39c8472adc5bd832b0fdd0469f8152356708974 Parents: af7476d Author: Paul Rogers <[email protected]> Authored: Mon Nov 26 12:13:00 2018 -0800 Committer: Impala Public Jenkins <[email protected]> Committed: Wed Nov 28 01:17:36 2018 +0000 ---------------------------------------------------------------------- .../org/apache/impala/analysis/AlterTableStmt.java | 1 - .../apache/impala/analysis/ColumnLineageGraph.java | 1 - .../impala/analysis/CreateFunctionStmtBase.java | 1 - .../org/apache/impala/analysis/ParquetHelper.java | 1 - .../java/org/apache/impala/analysis/PlanHint.java | 2 -- .../org/apache/impala/analysis/PrivilegeSpec.java | 1 - .../impala/authorization/SentryAuthProvider.java | 10 +++------- .../org/apache/impala/catalog/CatalogDeltaLog.java | 1 - .../org/apache/impala/catalog/CatalogObjectCache.java | 2 -- .../java/org/apache/impala/catalog/FeFsTable.java | 1 - .../java/org/apache/impala/catalog/HdfsPartition.java | 1 - .../catalog/HdfsPartitionLocationCompressor.java | 2 -- .../org/apache/impala/catalog/ImpaladCatalog.java | 1 - .../org/apache/impala/catalog/PartitionStatsUtil.java | 1 - .../apache/impala/catalog/local/LocalFsPartition.java | 3 --- .../java/org/apache/impala/common/FileSystemUtil.java | 1 - .../main/java/org/apache/impala/common/JniUtil.java | 1 - .../java/org/apache/impala/planner/DataPartition.java | 3 --- .../java/org/apache/impala/planner/HdfsScanNode.java | 1 - .../java/org/apache/impala/planner/JoinBuildSink.java | 1 - .../java/org/apache/impala/planner/KuduScanNode.java | 1 - .../java/org/apache/impala/planner/ValueRange.java | 3 --- .../org/apache/impala/rewrite/FoldConstantsRule.java | 1 - .../impala/rewrite/RemoveRedundantStringCast.java | 1 - .../java/org/apache/impala/service/FeSupport.java | 2 -- .../main/java/org/apache/impala/service/Frontend.java | 2 -- .../java/org/apache/impala/service/JniCatalog.java | 4 ---- .../java/org/apache/impala/service/JniFrontend.java | 1 - .../java/org/apache/impala/util/NativeLogger.java | 3 --- .../impala/analysis/StmtMetadataLoaderTest.java | 1 - .../java/org/apache/impala/catalog/CatalogTest.java | 8 -------- .../datagenerator/HBaseTestDataRegionAssignment.java | 8 -------- .../apache/impala/hive/executor/UdfExecutorTest.java | 1 - .../java/org/apache/impala/planner/S3PlannerTest.java | 2 -- .../impala/testutil/CatalogServiceTestCatalog.java | 2 -- .../apache/impala/testutil/ImpaladTestCatalog.java | 1 - .../org/apache/impala/testutil/TestFileParser.java | 1 - .../test/java/org/apache/impala/util/JniUtilTest.java | 14 +++----------- .../java/org/apache/impala/util/TestTopNCache.java | 1 - 39 files changed, 6 insertions(+), 87 deletions(-) ---------------------------------------------------------------------- http://git-wip-us.apache.org/repos/asf/impala/blob/a39c8472/fe/src/main/java/org/apache/impala/analysis/AlterTableStmt.java ---------------------------------------------------------------------- diff --git a/fe/src/main/java/org/apache/impala/analysis/AlterTableStmt.java b/fe/src/main/java/org/apache/impala/analysis/AlterTableStmt.java index 0089f0d..6304678 100644 --- a/fe/src/main/java/org/apache/impala/analysis/AlterTableStmt.java +++ b/fe/src/main/java/org/apache/impala/analysis/AlterTableStmt.java @@ -20,7 +20,6 @@ package org.apache.impala.analysis; import java.util.List; import org.apache.impala.authorization.Privilege; -import org.apache.impala.catalog.DataSourceTable; import org.apache.impala.catalog.FeDataSourceTable; import org.apache.impala.catalog.FeTable; import org.apache.impala.common.AnalysisException; http://git-wip-us.apache.org/repos/asf/impala/blob/a39c8472/fe/src/main/java/org/apache/impala/analysis/ColumnLineageGraph.java ---------------------------------------------------------------------- diff --git a/fe/src/main/java/org/apache/impala/analysis/ColumnLineageGraph.java b/fe/src/main/java/org/apache/impala/analysis/ColumnLineageGraph.java index 7df6e29..28d7674 100644 --- a/fe/src/main/java/org/apache/impala/analysis/ColumnLineageGraph.java +++ b/fe/src/main/java/org/apache/impala/analysis/ColumnLineageGraph.java @@ -17,7 +17,6 @@ package org.apache.impala.analysis; -import java.util.Arrays; import java.util.Collection; import java.util.LinkedHashMap; import java.util.List; http://git-wip-us.apache.org/repos/asf/impala/blob/a39c8472/fe/src/main/java/org/apache/impala/analysis/CreateFunctionStmtBase.java ---------------------------------------------------------------------- diff --git a/fe/src/main/java/org/apache/impala/analysis/CreateFunctionStmtBase.java b/fe/src/main/java/org/apache/impala/analysis/CreateFunctionStmtBase.java index ea48b3b..0d4d9bb 100644 --- a/fe/src/main/java/org/apache/impala/analysis/CreateFunctionStmtBase.java +++ b/fe/src/main/java/org/apache/impala/analysis/CreateFunctionStmtBase.java @@ -25,7 +25,6 @@ import org.apache.hadoop.fs.permission.FsAction; import org.apache.impala.authorization.AuthorizeableFn; import org.apache.impala.authorization.Privilege; import org.apache.impala.authorization.PrivilegeRequest; -import org.apache.impala.catalog.ImpaladCatalog; import org.apache.impala.catalog.FeDb; import org.apache.impala.catalog.Function; import org.apache.impala.catalog.Type; http://git-wip-us.apache.org/repos/asf/impala/blob/a39c8472/fe/src/main/java/org/apache/impala/analysis/ParquetHelper.java ---------------------------------------------------------------------- diff --git a/fe/src/main/java/org/apache/impala/analysis/ParquetHelper.java b/fe/src/main/java/org/apache/impala/analysis/ParquetHelper.java index e3d5554..dad2420 100644 --- a/fe/src/main/java/org/apache/impala/analysis/ParquetHelper.java +++ b/fe/src/main/java/org/apache/impala/analysis/ParquetHelper.java @@ -27,7 +27,6 @@ import com.google.common.base.Preconditions; import com.google.common.collect.Maps; import org.apache.hadoop.fs.FileSystem; import org.apache.hadoop.fs.Path; -import org.apache.hadoop.fs.permission.FsAction; import org.apache.parquet.hadoop.metadata.ParquetMetadata; import org.apache.parquet.hadoop.ParquetFileReader; import org.apache.parquet.schema.OriginalType; http://git-wip-us.apache.org/repos/asf/impala/blob/a39c8472/fe/src/main/java/org/apache/impala/analysis/PlanHint.java ---------------------------------------------------------------------- diff --git a/fe/src/main/java/org/apache/impala/analysis/PlanHint.java b/fe/src/main/java/org/apache/impala/analysis/PlanHint.java index d16919f..595bc47 100644 --- a/fe/src/main/java/org/apache/impala/analysis/PlanHint.java +++ b/fe/src/main/java/org/apache/impala/analysis/PlanHint.java @@ -18,11 +18,9 @@ package org.apache.impala.analysis; -import java.util.ArrayList; import java.util.List; import com.google.common.base.Joiner; -import com.google.common.base.Preconditions; import com.google.common.collect.Lists; /** http://git-wip-us.apache.org/repos/asf/impala/blob/a39c8472/fe/src/main/java/org/apache/impala/analysis/PrivilegeSpec.java ---------------------------------------------------------------------- diff --git a/fe/src/main/java/org/apache/impala/analysis/PrivilegeSpec.java b/fe/src/main/java/org/apache/impala/analysis/PrivilegeSpec.java index f14b304..2984c4c 100644 --- a/fe/src/main/java/org/apache/impala/analysis/PrivilegeSpec.java +++ b/fe/src/main/java/org/apache/impala/analysis/PrivilegeSpec.java @@ -23,7 +23,6 @@ import org.apache.impala.authorization.Privilege; import org.apache.impala.catalog.FeDataSourceTable; import org.apache.impala.catalog.FeTable; import org.apache.impala.catalog.FeView; -import org.apache.impala.catalog.PrincipalPrivilege; import org.apache.impala.catalog.TableLoadingException; import org.apache.impala.common.AnalysisException; import org.apache.impala.thrift.TPrivilege; http://git-wip-us.apache.org/repos/asf/impala/blob/a39c8472/fe/src/main/java/org/apache/impala/authorization/SentryAuthProvider.java ---------------------------------------------------------------------- diff --git a/fe/src/main/java/org/apache/impala/authorization/SentryAuthProvider.java b/fe/src/main/java/org/apache/impala/authorization/SentryAuthProvider.java index a4f0743..dc30dff 100644 --- a/fe/src/main/java/org/apache/impala/authorization/SentryAuthProvider.java +++ b/fe/src/main/java/org/apache/impala/authorization/SentryAuthProvider.java @@ -17,14 +17,8 @@ package org.apache.impala.authorization; -import com.google.common.base.Preconditions; - -import org.apache.impala.catalog.AuthorizationPolicy; - import org.apache.commons.lang.reflect.ConstructorUtils; -import org.apache.sentry.core.common.Model; -import org.apache.sentry.core.model.db.HivePrivilegeModel; -import org.apache.sentry.policy.common.PolicyEngine; +import org.apache.impala.catalog.AuthorizationPolicy; import org.apache.sentry.policy.engine.common.CommonPolicyEngine; import org.apache.sentry.provider.cache.SimpleCacheProviderBackend; import org.apache.sentry.provider.common.ProviderBackend; @@ -32,6 +26,8 @@ import org.apache.sentry.provider.common.ProviderBackendContext; import org.apache.sentry.provider.common.ResourceAuthorizationProvider; import org.apache.sentry.provider.file.SimpleFileProviderBackend; +import com.google.common.base.Preconditions; + /** * Wrapper to facilitate differences in Sentry APIs across Sentry versions. */ http://git-wip-us.apache.org/repos/asf/impala/blob/a39c8472/fe/src/main/java/org/apache/impala/catalog/CatalogDeltaLog.java ---------------------------------------------------------------------- diff --git a/fe/src/main/java/org/apache/impala/catalog/CatalogDeltaLog.java b/fe/src/main/java/org/apache/impala/catalog/CatalogDeltaLog.java index 1a5e2ec..b4267c0 100644 --- a/fe/src/main/java/org/apache/impala/catalog/CatalogDeltaLog.java +++ b/fe/src/main/java/org/apache/impala/catalog/CatalogDeltaLog.java @@ -23,7 +23,6 @@ import java.util.SortedMap; import java.util.TreeMap; import org.apache.impala.thrift.TCatalogObject; -import org.apache.impala.thrift.TTable; import com.google.common.base.Preconditions; import com.google.common.collect.ImmutableList; http://git-wip-us.apache.org/repos/asf/impala/blob/a39c8472/fe/src/main/java/org/apache/impala/catalog/CatalogObjectCache.java ---------------------------------------------------------------------- diff --git a/fe/src/main/java/org/apache/impala/catalog/CatalogObjectCache.java b/fe/src/main/java/org/apache/impala/catalog/CatalogObjectCache.java index 38698c5..e92c070 100644 --- a/fe/src/main/java/org/apache/impala/catalog/CatalogObjectCache.java +++ b/fe/src/main/java/org/apache/impala/catalog/CatalogObjectCache.java @@ -22,8 +22,6 @@ import java.util.List; import java.util.Set; import java.util.concurrent.ConcurrentHashMap; -import org.apache.log4j.Logger; - import com.google.common.base.Preconditions; import com.google.common.collect.Lists; http://git-wip-us.apache.org/repos/asf/impala/blob/a39c8472/fe/src/main/java/org/apache/impala/catalog/FeFsTable.java ---------------------------------------------------------------------- diff --git a/fe/src/main/java/org/apache/impala/catalog/FeFsTable.java b/fe/src/main/java/org/apache/impala/catalog/FeFsTable.java index 0c861af..5c619a2 100644 --- a/fe/src/main/java/org/apache/impala/catalog/FeFsTable.java +++ b/fe/src/main/java/org/apache/impala/catalog/FeFsTable.java @@ -28,7 +28,6 @@ import java.util.TreeMap; import org.apache.hadoop.hive.metastore.api.FieldSchema; import org.apache.impala.analysis.Expr; import org.apache.impala.analysis.LiteralExpr; -import org.apache.impala.analysis.NullLiteral; import org.apache.impala.analysis.PartitionKeyValue; import org.apache.impala.catalog.HdfsPartition.FileDescriptor; import org.apache.impala.common.AnalysisException; http://git-wip-us.apache.org/repos/asf/impala/blob/a39c8472/fe/src/main/java/org/apache/impala/catalog/HdfsPartition.java ---------------------------------------------------------------------- diff --git a/fe/src/main/java/org/apache/impala/catalog/HdfsPartition.java b/fe/src/main/java/org/apache/impala/catalog/HdfsPartition.java index 0a356ae..cd52b5b 100644 --- a/fe/src/main/java/org/apache/impala/catalog/HdfsPartition.java +++ b/fe/src/main/java/org/apache/impala/catalog/HdfsPartition.java @@ -21,7 +21,6 @@ import java.io.IOException; import java.nio.ByteBuffer; import java.util.Collection; import java.util.Comparator; -import java.util.Iterator; import java.util.List; import java.util.Map; import java.util.Set; http://git-wip-us.apache.org/repos/asf/impala/blob/a39c8472/fe/src/main/java/org/apache/impala/catalog/HdfsPartitionLocationCompressor.java ---------------------------------------------------------------------- diff --git a/fe/src/main/java/org/apache/impala/catalog/HdfsPartitionLocationCompressor.java b/fe/src/main/java/org/apache/impala/catalog/HdfsPartitionLocationCompressor.java index 94b9e4d..939bcf6 100644 --- a/fe/src/main/java/org/apache/impala/catalog/HdfsPartitionLocationCompressor.java +++ b/fe/src/main/java/org/apache/impala/catalog/HdfsPartitionLocationCompressor.java @@ -17,8 +17,6 @@ package org.apache.impala.catalog; -import java.util.ArrayList; -import java.util.HashMap; import java.util.List; import org.apache.impala.common.Pair; http://git-wip-us.apache.org/repos/asf/impala/blob/a39c8472/fe/src/main/java/org/apache/impala/catalog/ImpaladCatalog.java ---------------------------------------------------------------------- diff --git a/fe/src/main/java/org/apache/impala/catalog/ImpaladCatalog.java b/fe/src/main/java/org/apache/impala/catalog/ImpaladCatalog.java index cd31279..8f97486 100644 --- a/fe/src/main/java/org/apache/impala/catalog/ImpaladCatalog.java +++ b/fe/src/main/java/org/apache/impala/catalog/ImpaladCatalog.java @@ -19,7 +19,6 @@ package org.apache.impala.catalog; import java.nio.ByteBuffer; import java.util.ArrayDeque; -import java.util.List; import java.util.Set; import java.util.concurrent.atomic.AtomicLong; http://git-wip-us.apache.org/repos/asf/impala/blob/a39c8472/fe/src/main/java/org/apache/impala/catalog/PartitionStatsUtil.java ---------------------------------------------------------------------- diff --git a/fe/src/main/java/org/apache/impala/catalog/PartitionStatsUtil.java b/fe/src/main/java/org/apache/impala/catalog/PartitionStatsUtil.java index d3299f5..c93b245 100644 --- a/fe/src/main/java/org/apache/impala/catalog/PartitionStatsUtil.java +++ b/fe/src/main/java/org/apache/impala/catalog/PartitionStatsUtil.java @@ -25,7 +25,6 @@ import org.apache.impala.common.ImpalaRuntimeException; import org.apache.impala.util.CompressionUtil; import org.apache.impala.util.MetaStoreUtil; -import java.util.Iterator; import java.util.List; import java.util.Map; import org.apache.commons.codec.binary.Base64; http://git-wip-us.apache.org/repos/asf/impala/blob/a39c8472/fe/src/main/java/org/apache/impala/catalog/local/LocalFsPartition.java ---------------------------------------------------------------------- diff --git a/fe/src/main/java/org/apache/impala/catalog/local/LocalFsPartition.java b/fe/src/main/java/org/apache/impala/catalog/local/LocalFsPartition.java index c594873..522e984 100644 --- a/fe/src/main/java/org/apache/impala/catalog/local/LocalFsPartition.java +++ b/fe/src/main/java/org/apache/impala/catalog/local/LocalFsPartition.java @@ -17,7 +17,6 @@ package org.apache.impala.catalog.local; -import java.io.IOException; import java.util.List; import java.util.Map; @@ -30,7 +29,6 @@ import org.apache.impala.catalog.FeCatalogUtils; import org.apache.impala.catalog.FeFsPartition; import org.apache.impala.catalog.FeFsTable; import org.apache.impala.catalog.HdfsFileFormat; -import org.apache.impala.catalog.HdfsPartition; import org.apache.impala.catalog.HdfsPartition.FileDescriptor; import org.apache.impala.catalog.HdfsStorageDescriptor; import org.apache.impala.catalog.HdfsStorageDescriptor.InvalidStorageDescriptorException; @@ -41,7 +39,6 @@ import org.apache.impala.thrift.TPartitionStats; import com.google.common.base.Preconditions; import com.google.common.collect.ImmutableList; -import com.google.common.collect.Maps; public class LocalFsPartition implements FeFsPartition { private final LocalFsTable table_; http://git-wip-us.apache.org/repos/asf/impala/blob/a39c8472/fe/src/main/java/org/apache/impala/common/FileSystemUtil.java ---------------------------------------------------------------------- diff --git a/fe/src/main/java/org/apache/impala/common/FileSystemUtil.java b/fe/src/main/java/org/apache/impala/common/FileSystemUtil.java index 43d164b..21e30ac 100644 --- a/fe/src/main/java/org/apache/impala/common/FileSystemUtil.java +++ b/fe/src/main/java/org/apache/impala/common/FileSystemUtil.java @@ -43,7 +43,6 @@ import org.apache.impala.catalog.HdfsCompression; import org.slf4j.Logger; import org.slf4j.LoggerFactory; -import com.google.common.base.Objects; import com.google.common.base.Preconditions; /** http://git-wip-us.apache.org/repos/asf/impala/blob/a39c8472/fe/src/main/java/org/apache/impala/common/JniUtil.java ---------------------------------------------------------------------- diff --git a/fe/src/main/java/org/apache/impala/common/JniUtil.java b/fe/src/main/java/org/apache/impala/common/JniUtil.java index 37e2769..edfcd8c 100644 --- a/fe/src/main/java/org/apache/impala/common/JniUtil.java +++ b/fe/src/main/java/org/apache/impala/common/JniUtil.java @@ -41,7 +41,6 @@ import org.apache.thrift.protocol.TBinaryProtocol; import org.apache.thrift.protocol.TProtocolFactory; import com.google.common.base.Joiner; -import com.google.common.base.Strings; import org.apache.impala.thrift.TGetJvmMemoryMetricsResponse; import org.apache.impala.thrift.TGetJvmThreadsInfoRequest; http://git-wip-us.apache.org/repos/asf/impala/blob/a39c8472/fe/src/main/java/org/apache/impala/planner/DataPartition.java ---------------------------------------------------------------------- diff --git a/fe/src/main/java/org/apache/impala/planner/DataPartition.java b/fe/src/main/java/org/apache/impala/planner/DataPartition.java index 67a411a..62019ae 100644 --- a/fe/src/main/java/org/apache/impala/planner/DataPartition.java +++ b/fe/src/main/java/org/apache/impala/planner/DataPartition.java @@ -19,9 +19,6 @@ package org.apache.impala.planner; import java.util.List; -import org.slf4j.Logger; -import org.slf4j.LoggerFactory; - import org.apache.impala.analysis.Analyzer; import org.apache.impala.analysis.Expr; import org.apache.impala.analysis.ExprSubstitutionMap; http://git-wip-us.apache.org/repos/asf/impala/blob/a39c8472/fe/src/main/java/org/apache/impala/planner/HdfsScanNode.java ---------------------------------------------------------------------- diff --git a/fe/src/main/java/org/apache/impala/planner/HdfsScanNode.java b/fe/src/main/java/org/apache/impala/planner/HdfsScanNode.java index 1f4c2bd..4cdfbeb 100644 --- a/fe/src/main/java/org/apache/impala/planner/HdfsScanNode.java +++ b/fe/src/main/java/org/apache/impala/planner/HdfsScanNode.java @@ -39,7 +39,6 @@ import org.apache.impala.analysis.FunctionParams; import org.apache.impala.analysis.InPredicate; import org.apache.impala.analysis.IsNotEmptyPredicate; import org.apache.impala.analysis.LiteralExpr; -import org.apache.impala.analysis.NullLiteral; import org.apache.impala.analysis.SlotDescriptor; import org.apache.impala.analysis.SlotId; import org.apache.impala.analysis.SlotRef; http://git-wip-us.apache.org/repos/asf/impala/blob/a39c8472/fe/src/main/java/org/apache/impala/planner/JoinBuildSink.java ---------------------------------------------------------------------- diff --git a/fe/src/main/java/org/apache/impala/planner/JoinBuildSink.java b/fe/src/main/java/org/apache/impala/planner/JoinBuildSink.java index 192602a..a9bea59 100644 --- a/fe/src/main/java/org/apache/impala/planner/JoinBuildSink.java +++ b/fe/src/main/java/org/apache/impala/planner/JoinBuildSink.java @@ -21,7 +21,6 @@ import java.util.List; import org.apache.impala.analysis.BinaryPredicate; import org.apache.impala.analysis.Expr; -import org.apache.impala.analysis.ToSqlOptions; import org.apache.impala.thrift.TDataSink; import org.apache.impala.thrift.TDataSinkType; import org.apache.impala.thrift.TExplainLevel; http://git-wip-us.apache.org/repos/asf/impala/blob/a39c8472/fe/src/main/java/org/apache/impala/planner/KuduScanNode.java ---------------------------------------------------------------------- diff --git a/fe/src/main/java/org/apache/impala/planner/KuduScanNode.java b/fe/src/main/java/org/apache/impala/planner/KuduScanNode.java index 3bb7b6f..aa18f15 100644 --- a/fe/src/main/java/org/apache/impala/planner/KuduScanNode.java +++ b/fe/src/main/java/org/apache/impala/planner/KuduScanNode.java @@ -29,7 +29,6 @@ import org.apache.impala.analysis.Expr; import org.apache.impala.analysis.InPredicate; import org.apache.impala.analysis.IsNullPredicate; import org.apache.impala.analysis.LiteralExpr; -import org.apache.impala.analysis.NullLiteral; import org.apache.impala.analysis.NumericLiteral; import org.apache.impala.analysis.SlotDescriptor; import org.apache.impala.analysis.SlotRef; http://git-wip-us.apache.org/repos/asf/impala/blob/a39c8472/fe/src/main/java/org/apache/impala/planner/ValueRange.java ---------------------------------------------------------------------- diff --git a/fe/src/main/java/org/apache/impala/planner/ValueRange.java b/fe/src/main/java/org/apache/impala/planner/ValueRange.java index 4b8f252..97edfb8 100644 --- a/fe/src/main/java/org/apache/impala/planner/ValueRange.java +++ b/fe/src/main/java/org/apache/impala/planner/ValueRange.java @@ -17,9 +17,6 @@ package org.apache.impala.planner; -import org.slf4j.Logger; -import org.slf4j.LoggerFactory; - import org.apache.impala.analysis.Analyzer; import org.apache.impala.analysis.BinaryPredicate; import org.apache.impala.analysis.CompoundPredicate; http://git-wip-us.apache.org/repos/asf/impala/blob/a39c8472/fe/src/main/java/org/apache/impala/rewrite/FoldConstantsRule.java ---------------------------------------------------------------------- diff --git a/fe/src/main/java/org/apache/impala/rewrite/FoldConstantsRule.java b/fe/src/main/java/org/apache/impala/rewrite/FoldConstantsRule.java index dd0052f..9a0e714 100644 --- a/fe/src/main/java/org/apache/impala/rewrite/FoldConstantsRule.java +++ b/fe/src/main/java/org/apache/impala/rewrite/FoldConstantsRule.java @@ -20,7 +20,6 @@ package org.apache.impala.rewrite; import org.apache.impala.analysis.Analyzer; import org.apache.impala.analysis.Expr; import org.apache.impala.analysis.LiteralExpr; -import org.apache.impala.analysis.NullLiteral; import org.apache.impala.analysis.CastExpr; import org.apache.impala.common.AnalysisException; http://git-wip-us.apache.org/repos/asf/impala/blob/a39c8472/fe/src/main/java/org/apache/impala/rewrite/RemoveRedundantStringCast.java ---------------------------------------------------------------------- diff --git a/fe/src/main/java/org/apache/impala/rewrite/RemoveRedundantStringCast.java b/fe/src/main/java/org/apache/impala/rewrite/RemoveRedundantStringCast.java index 97dcdbb..24f34ba 100644 --- a/fe/src/main/java/org/apache/impala/rewrite/RemoveRedundantStringCast.java +++ b/fe/src/main/java/org/apache/impala/rewrite/RemoveRedundantStringCast.java @@ -23,7 +23,6 @@ import org.apache.impala.analysis.CastExpr; import org.apache.impala.analysis.Expr; import org.apache.impala.analysis.LiteralExpr; import org.apache.impala.analysis.TypeDef; -import org.apache.impala.catalog.Type; import org.apache.impala.common.AnalysisException; /** http://git-wip-us.apache.org/repos/asf/impala/blob/a39c8472/fe/src/main/java/org/apache/impala/service/FeSupport.java ---------------------------------------------------------------------- diff --git a/fe/src/main/java/org/apache/impala/service/FeSupport.java b/fe/src/main/java/org/apache/impala/service/FeSupport.java index e67bf9d..261f0bd 100644 --- a/fe/src/main/java/org/apache/impala/service/FeSupport.java +++ b/fe/src/main/java/org/apache/impala/service/FeSupport.java @@ -24,7 +24,6 @@ import java.util.Set; import org.apache.impala.analysis.BoolLiteral; import org.apache.impala.analysis.Expr; -import org.apache.impala.analysis.NullLiteral; import org.apache.impala.analysis.SlotRef; import org.apache.impala.analysis.TableName; import org.apache.impala.common.InternalException; @@ -47,7 +46,6 @@ import org.apache.impala.thrift.TResultRow; import org.apache.impala.thrift.TSymbolLookupParams; import org.apache.impala.thrift.TSymbolLookupResult; import org.apache.impala.thrift.TTable; -import org.apache.impala.thrift.TUniqueId; import org.apache.impala.util.NativeLibUtil; import org.apache.thrift.TDeserializer; import org.apache.thrift.TException; http://git-wip-us.apache.org/repos/asf/impala/blob/a39c8472/fe/src/main/java/org/apache/impala/service/Frontend.java ---------------------------------------------------------------------- diff --git a/fe/src/main/java/org/apache/impala/service/Frontend.java b/fe/src/main/java/org/apache/impala/service/Frontend.java index f6aa123..d67d1e0 100644 --- a/fe/src/main/java/org/apache/impala/service/Frontend.java +++ b/fe/src/main/java/org/apache/impala/service/Frontend.java @@ -100,7 +100,6 @@ import org.apache.impala.planner.HdfsScanNode; import org.apache.impala.planner.PlanFragment; import org.apache.impala.planner.Planner; import org.apache.impala.planner.ScanNode; -import org.apache.impala.thrift.TAdminRequest; import org.apache.impala.thrift.TAlterDbParams; import org.apache.impala.thrift.TCatalogOpRequest; import org.apache.impala.thrift.TCatalogOpType; @@ -136,7 +135,6 @@ import org.apache.impala.thrift.TResultSet; import org.apache.impala.thrift.TResultSetMetadata; import org.apache.impala.thrift.TShowFilesParams; import org.apache.impala.thrift.TShowStatsOp; -import org.apache.impala.thrift.TShutdownParams; import org.apache.impala.thrift.TStmtType; import org.apache.impala.thrift.TTableName; import org.apache.impala.thrift.TUpdateCatalogCacheRequest; http://git-wip-us.apache.org/repos/asf/impala/blob/a39c8472/fe/src/main/java/org/apache/impala/service/JniCatalog.java ---------------------------------------------------------------------- diff --git a/fe/src/main/java/org/apache/impala/service/JniCatalog.java b/fe/src/main/java/org/apache/impala/service/JniCatalog.java index 2a709e8..457aa78 100644 --- a/fe/src/main/java/org/apache/impala/service/JniCatalog.java +++ b/fe/src/main/java/org/apache/impala/service/JniCatalog.java @@ -23,10 +23,7 @@ import com.google.common.collect.ImmutableList; import com.google.common.collect.Lists; import java.util.ArrayList; -import java.util.Collections; -import java.util.HashMap; import java.util.List; -import java.util.Map; import java.util.UUID; import org.apache.impala.authorization.SentryConfig; @@ -58,7 +55,6 @@ import org.apache.impala.thrift.TGetTablesParams; import org.apache.impala.thrift.TGetTableMetricsParams; import org.apache.impala.thrift.TGetTablesResult; import org.apache.impala.thrift.TLogLevel; -import org.apache.impala.thrift.TPartitionStats; import org.apache.impala.thrift.TPrioritizeLoadRequest; import org.apache.impala.thrift.TResetMetadataRequest; import org.apache.impala.thrift.TSentryAdminCheckRequest; http://git-wip-us.apache.org/repos/asf/impala/blob/a39c8472/fe/src/main/java/org/apache/impala/service/JniFrontend.java ---------------------------------------------------------------------- diff --git a/fe/src/main/java/org/apache/impala/service/JniFrontend.java b/fe/src/main/java/org/apache/impala/service/JniFrontend.java index db2ad29..6a1d7e9 100644 --- a/fe/src/main/java/org/apache/impala/service/JniFrontend.java +++ b/fe/src/main/java/org/apache/impala/service/JniFrontend.java @@ -87,7 +87,6 @@ import org.apache.impala.thrift.TLoadDataReq; import org.apache.impala.thrift.TLoadDataResp; import org.apache.impala.thrift.TLogLevel; import org.apache.impala.thrift.TMetadataOpRequest; -import org.apache.impala.thrift.TPrincipalType; import org.apache.impala.thrift.TQueryCtx; import org.apache.impala.thrift.TResultSet; import org.apache.impala.thrift.TShowFilesParams; http://git-wip-us.apache.org/repos/asf/impala/blob/a39c8472/fe/src/main/java/org/apache/impala/util/NativeLogger.java ---------------------------------------------------------------------- diff --git a/fe/src/main/java/org/apache/impala/util/NativeLogger.java b/fe/src/main/java/org/apache/impala/util/NativeLogger.java index fc5b169..fd95d4b 100644 --- a/fe/src/main/java/org/apache/impala/util/NativeLogger.java +++ b/fe/src/main/java/org/apache/impala/util/NativeLogger.java @@ -17,9 +17,6 @@ package org.apache.impala.util; -import org.slf4j.Logger; -import org.slf4j.LoggerFactory; - /** * Class that manages loading and calling the native logging library to forward * log4j log messages to be logged by glog. http://git-wip-us.apache.org/repos/asf/impala/blob/a39c8472/fe/src/test/java/org/apache/impala/analysis/StmtMetadataLoaderTest.java ---------------------------------------------------------------------- diff --git a/fe/src/test/java/org/apache/impala/analysis/StmtMetadataLoaderTest.java b/fe/src/test/java/org/apache/impala/analysis/StmtMetadataLoaderTest.java index 29eddfe..91aa72f 100644 --- a/fe/src/test/java/org/apache/impala/analysis/StmtMetadataLoaderTest.java +++ b/fe/src/test/java/org/apache/impala/analysis/StmtMetadataLoaderTest.java @@ -23,7 +23,6 @@ import org.apache.impala.analysis.StmtMetadataLoader.StmtTableCache; import org.apache.impala.authorization.AuthorizationConfig; import org.apache.impala.catalog.Catalog; import org.apache.impala.catalog.FeTable; -import org.apache.impala.catalog.Table; import org.apache.impala.common.ImpalaException; import org.apache.impala.common.InternalException; import org.apache.impala.service.Frontend; http://git-wip-us.apache.org/repos/asf/impala/blob/a39c8472/fe/src/test/java/org/apache/impala/catalog/CatalogTest.java ---------------------------------------------------------------------- diff --git a/fe/src/test/java/org/apache/impala/catalog/CatalogTest.java b/fe/src/test/java/org/apache/impala/catalog/CatalogTest.java index b726af7..6b47bda 100644 --- a/fe/src/test/java/org/apache/impala/catalog/CatalogTest.java +++ b/fe/src/test/java/org/apache/impala/catalog/CatalogTest.java @@ -48,36 +48,28 @@ import org.apache.hadoop.hive.metastore.api.ColumnStatisticsData; import org.apache.impala.analysis.FunctionName; import org.apache.impala.analysis.LiteralExpr; import org.apache.impala.analysis.NumericLiteral; -import org.apache.impala.analysis.TableName; import org.apache.impala.catalog.MetaStoreClientPool.MetaStoreClient; -import org.apache.impala.common.FrontendTestBase; import org.apache.impala.common.ImpalaException; import org.apache.impala.common.Reference; import org.apache.impala.service.BackendConfig; import org.apache.impala.testutil.CatalogServiceTestCatalog; -import org.apache.impala.thrift.CatalogObjectsConstants; import org.apache.impala.thrift.TBackendGflags; import org.apache.impala.thrift.TFunctionBinaryType; import org.apache.impala.thrift.TGetPartitionStatsRequest; -import org.apache.impala.thrift.TGetPartitionStatsResponse; import org.apache.impala.thrift.TPartitionStats; import org.apache.impala.thrift.TPrincipalType; import org.apache.impala.thrift.TPrivilege; import org.apache.impala.thrift.TPrivilegeLevel; import org.apache.impala.thrift.TPrivilegeScope; import org.apache.impala.thrift.TTableName; -import org.apache.impala.thrift.TUniqueId; import org.junit.Before; import org.junit.Test; -import com.google.common.base.Preconditions; import com.google.common.base.Strings; import com.google.common.collect.Lists; import com.google.common.collect.Maps; import com.google.common.collect.Sets; -import junit.framework.AssertionFailedError; - public class CatalogTest { private CatalogServiceCatalog catalog_; http://git-wip-us.apache.org/repos/asf/impala/blob/a39c8472/fe/src/test/java/org/apache/impala/datagenerator/HBaseTestDataRegionAssignment.java ---------------------------------------------------------------------- diff --git a/fe/src/test/java/org/apache/impala/datagenerator/HBaseTestDataRegionAssignment.java b/fe/src/test/java/org/apache/impala/datagenerator/HBaseTestDataRegionAssignment.java index 85f8510..0b13566 100644 --- a/fe/src/test/java/org/apache/impala/datagenerator/HBaseTestDataRegionAssignment.java +++ b/fe/src/test/java/org/apache/impala/datagenerator/HBaseTestDataRegionAssignment.java @@ -25,23 +25,15 @@ import java.util.List; import org.apache.hadoop.conf.Configuration; import org.apache.hadoop.hbase.ClusterStatus; -import org.apache.hadoop.hbase.HRegionLocation; import org.apache.hadoop.hbase.MetaTableAccessor; import org.apache.hadoop.hbase.ServerName; import org.apache.hadoop.hbase.TableName; import org.apache.hadoop.hbase.client.Admin; import org.apache.hadoop.hbase.client.Connection; import org.apache.hadoop.hbase.client.ConnectionFactory; -import org.apache.hadoop.hbase.client.Get; import org.apache.hadoop.hbase.client.RegionInfo; -import org.apache.hadoop.hbase.client.Result; -import org.apache.hadoop.hbase.client.Table; -import org.apache.hadoop.hbase.util.Bytes; import org.apache.hadoop.hbase.util.Pair; -import org.apache.hadoop.hbase.util.PairOfSameType; -import org.apache.hadoop.hbase.util.Threads; import org.apache.impala.planner.HBaseScanNode; - import org.slf4j.Logger; import org.slf4j.LoggerFactory; http://git-wip-us.apache.org/repos/asf/impala/blob/a39c8472/fe/src/test/java/org/apache/impala/hive/executor/UdfExecutorTest.java ---------------------------------------------------------------------- diff --git a/fe/src/test/java/org/apache/impala/hive/executor/UdfExecutorTest.java b/fe/src/test/java/org/apache/impala/hive/executor/UdfExecutorTest.java index dad04a6..7ca7f4b 100644 --- a/fe/src/test/java/org/apache/impala/hive/executor/UdfExecutorTest.java +++ b/fe/src/test/java/org/apache/impala/hive/executor/UdfExecutorTest.java @@ -71,7 +71,6 @@ import org.apache.thrift.protocol.TBinaryProtocol; import org.junit.Assert; import org.junit.Test; -import com.google.common.base.Joiner; import com.google.common.base.Preconditions; import com.google.common.base.Strings; import com.google.common.collect.Lists; http://git-wip-us.apache.org/repos/asf/impala/blob/a39c8472/fe/src/test/java/org/apache/impala/planner/S3PlannerTest.java ---------------------------------------------------------------------- diff --git a/fe/src/test/java/org/apache/impala/planner/S3PlannerTest.java b/fe/src/test/java/org/apache/impala/planner/S3PlannerTest.java index 49f0188..364eea3 100644 --- a/fe/src/test/java/org/apache/impala/planner/S3PlannerTest.java +++ b/fe/src/test/java/org/apache/impala/planner/S3PlannerTest.java @@ -21,8 +21,6 @@ import static org.junit.Assume.assumeTrue; import java.net.URI; -import org.apache.commons.lang.StringUtils; -import org.apache.hadoop.conf.Configuration; import org.apache.hadoop.fs.Path; import org.junit.Assert; import org.junit.Before; http://git-wip-us.apache.org/repos/asf/impala/blob/a39c8472/fe/src/test/java/org/apache/impala/testutil/CatalogServiceTestCatalog.java ---------------------------------------------------------------------- diff --git a/fe/src/test/java/org/apache/impala/testutil/CatalogServiceTestCatalog.java b/fe/src/test/java/org/apache/impala/testutil/CatalogServiceTestCatalog.java index 3b61028..3310ecf 100644 --- a/fe/src/test/java/org/apache/impala/testutil/CatalogServiceTestCatalog.java +++ b/fe/src/test/java/org/apache/impala/testutil/CatalogServiceTestCatalog.java @@ -21,9 +21,7 @@ import org.apache.impala.authorization.SentryConfig; import org.apache.impala.catalog.AuthorizationPolicy; import org.apache.impala.catalog.CatalogServiceCatalog; import org.apache.impala.common.ImpalaException; -import org.apache.impala.service.BackendConfig; import org.apache.impala.service.FeSupport; -import org.apache.impala.thrift.TBackendGflags; import org.apache.impala.thrift.TUniqueId; /** http://git-wip-us.apache.org/repos/asf/impala/blob/a39c8472/fe/src/test/java/org/apache/impala/testutil/ImpaladTestCatalog.java ---------------------------------------------------------------------- diff --git a/fe/src/test/java/org/apache/impala/testutil/ImpaladTestCatalog.java b/fe/src/test/java/org/apache/impala/testutil/ImpaladTestCatalog.java index 0734ee0..60bbaeb 100644 --- a/fe/src/test/java/org/apache/impala/testutil/ImpaladTestCatalog.java +++ b/fe/src/test/java/org/apache/impala/testutil/ImpaladTestCatalog.java @@ -27,7 +27,6 @@ import org.apache.impala.catalog.FeDb; import org.apache.impala.catalog.HdfsCachePool; import org.apache.impala.catalog.HdfsTable; import org.apache.impala.catalog.ImpaladCatalog; -import org.apache.impala.catalog.Principal; import org.apache.impala.catalog.PrincipalPrivilege; import org.apache.impala.catalog.Role; import org.apache.impala.catalog.Table; http://git-wip-us.apache.org/repos/asf/impala/blob/a39c8472/fe/src/test/java/org/apache/impala/testutil/TestFileParser.java ---------------------------------------------------------------------- diff --git a/fe/src/test/java/org/apache/impala/testutil/TestFileParser.java b/fe/src/test/java/org/apache/impala/testutil/TestFileParser.java index 0c62666..fb04704 100644 --- a/fe/src/test/java/org/apache/impala/testutil/TestFileParser.java +++ b/fe/src/test/java/org/apache/impala/testutil/TestFileParser.java @@ -30,7 +30,6 @@ import java.util.Scanner; import org.apache.impala.common.InternalException; import org.apache.impala.service.FeSupport; -import org.apache.impala.thrift.TErrorCode; import org.apache.impala.thrift.TQueryOptions; import org.apache.log4j.Logger; http://git-wip-us.apache.org/repos/asf/impala/blob/a39c8472/fe/src/test/java/org/apache/impala/util/JniUtilTest.java ---------------------------------------------------------------------- diff --git a/fe/src/test/java/org/apache/impala/util/JniUtilTest.java b/fe/src/test/java/org/apache/impala/util/JniUtilTest.java index 6166179..a11b57e 100644 --- a/fe/src/test/java/org/apache/impala/util/JniUtilTest.java +++ b/fe/src/test/java/org/apache/impala/util/JniUtilTest.java @@ -17,21 +17,13 @@ package org.apache.impala.util; -import static org.junit.Assert.*; - -import com.fasterxml.jackson.databind.JsonNode; -import com.fasterxml.jackson.databind.ObjectMapper; -import com.google.common.base.Preconditions; -import org.apache.impala.thrift.TGetJMXJsonResponse; -import org.junit.Test; +import static org.junit.Assert.assertEquals; import org.apache.impala.common.ImpalaException; import org.apache.impala.common.JniUtil; -import org.apache.thrift.protocol.TBinaryProtocol; -import org.apache.thrift.protocol.TProtocolFactory; import org.apache.impala.thrift.TCacheJarParams; - -import java.io.IOException; +import org.apache.thrift.protocol.TBinaryProtocol; +import org.junit.Test; /** * Unit tests for JniUtil functions. http://git-wip-us.apache.org/repos/asf/impala/blob/a39c8472/fe/src/test/java/org/apache/impala/util/TestTopNCache.java ---------------------------------------------------------------------- diff --git a/fe/src/test/java/org/apache/impala/util/TestTopNCache.java b/fe/src/test/java/org/apache/impala/util/TestTopNCache.java index 1b34599..159a14f 100644 --- a/fe/src/test/java/org/apache/impala/util/TestTopNCache.java +++ b/fe/src/test/java/org/apache/impala/util/TestTopNCache.java @@ -23,7 +23,6 @@ import org.junit.Test; import com.google.common.base.Function; -import java.util.Arrays; import java.util.List; /**
