Skip to content

Commit 2cb4552

Browse files
committed
Revert some changes
1 parent 9e5cfe0 commit 2cb4552

File tree

4 files changed

+31
-9
lines changed

4 files changed

+31
-9
lines changed

quarkus/service/src/test/java/org/apache/polaris/service/quarkus/catalog/BasePolarisCatalogTest.java

Lines changed: 18 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -20,8 +20,11 @@
2020

2121
import static java.nio.charset.StandardCharsets.UTF_8;
2222
import static org.apache.iceberg.types.Types.NestedField.required;
23-
import static org.mockito.ArgumentMatchers.*;
24-
import static org.mockito.Mockito.*;
23+
import static org.mockito.ArgumentMatchers.any;
24+
import static org.mockito.ArgumentMatchers.isA;
25+
import static org.mockito.Mockito.doThrow;
26+
import static org.mockito.Mockito.spy;
27+
import static org.mockito.Mockito.when;
2528

2629
import com.google.common.collect.ImmutableMap;
2730
import com.google.common.collect.Iterators;
@@ -41,7 +44,14 @@
4144
import java.util.UUID;
4245
import java.util.function.Supplier;
4346
import org.apache.commons.lang3.NotImplementedException;
44-
import org.apache.iceberg.*;
47+
import org.apache.iceberg.BaseTable;
48+
import org.apache.iceberg.CatalogProperties;
49+
import org.apache.iceberg.PartitionSpec;
50+
import org.apache.iceberg.Schema;
51+
import org.apache.iceberg.SortOrder;
52+
import org.apache.iceberg.Table;
53+
import org.apache.iceberg.TableMetadata;
54+
import org.apache.iceberg.TableMetadataParser;
4555
import org.apache.iceberg.catalog.CatalogTests;
4656
import org.apache.iceberg.catalog.Namespace;
4757
import org.apache.iceberg.catalog.SupportsNamespaces;
@@ -69,7 +79,11 @@
6979
import org.apache.polaris.core.entity.PolarisEntityType;
7080
import org.apache.polaris.core.entity.PrincipalEntity;
7181
import org.apache.polaris.core.entity.TaskEntity;
72-
import org.apache.polaris.core.persistence.*;
82+
import org.apache.polaris.core.persistence.MetaStoreManagerFactory;
83+
import org.apache.polaris.core.persistence.PolarisCredentialsBootstrap;
84+
import org.apache.polaris.core.persistence.PolarisEntityManager;
85+
import org.apache.polaris.core.persistence.PolarisMetaStoreManager;
86+
import org.apache.polaris.core.persistence.PolarisMetaStoreSession;
7387
import org.apache.polaris.core.persistence.cache.EntityCache;
7488
import org.apache.polaris.core.storage.PolarisCredentialProperty;
7589
import org.apache.polaris.core.storage.PolarisStorageIntegration;

quarkus/service/src/test/java/org/apache/polaris/service/quarkus/catalog/io/TestFileIOFactory.java

Lines changed: 7 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -21,13 +21,18 @@
2121
import jakarta.annotation.Nonnull;
2222
import jakarta.enterprise.inject.Vetoed;
2323
import jakarta.inject.Inject;
24-
import java.util.*;
24+
import java.util.ArrayList;
25+
import java.util.List;
26+
import java.util.Map;
27+
import java.util.Optional;
28+
import java.util.Set;
2529
import java.util.function.Supplier;
2630
import org.apache.iceberg.catalog.TableIdentifier;
2731
import org.apache.iceberg.io.FileIO;
2832
import org.apache.polaris.core.PolarisConfigurationStore;
2933
import org.apache.polaris.core.context.RealmId;
30-
import org.apache.polaris.core.persistence.*;
34+
import org.apache.polaris.core.persistence.MetaStoreManagerFactory;
35+
import org.apache.polaris.core.persistence.PolarisResolvedPathWrapper;
3136
import org.apache.polaris.core.storage.PolarisStorageActions;
3237
import org.apache.polaris.service.catalog.io.DefaultFileIOFactory;
3338
import org.apache.polaris.service.catalog.io.FileIOFactory;

service/common/src/main/java/org/apache/polaris/service/catalog/io/DefaultFileIOFactory.java

Lines changed: 4 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -34,7 +34,10 @@
3434
import org.apache.polaris.core.PolarisConfigurationStore;
3535
import org.apache.polaris.core.context.RealmId;
3636
import org.apache.polaris.core.entity.PolarisEntity;
37-
import org.apache.polaris.core.persistence.*;
37+
import org.apache.polaris.core.persistence.MetaStoreManagerFactory;
38+
import org.apache.polaris.core.persistence.PolarisEntityManager;
39+
import org.apache.polaris.core.persistence.PolarisMetaStoreSession;
40+
import org.apache.polaris.core.persistence.PolarisResolvedPathWrapper;
3841
import org.apache.polaris.core.storage.PolarisCredentialVendor;
3942
import org.apache.polaris.core.storage.PolarisStorageActions;
4043
import org.apache.polaris.service.config.RealmEntityManagerFactory;

service/common/src/main/java/org/apache/polaris/service/catalog/io/WasbTranslatingFileIOFactory.java

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -20,7 +20,7 @@
2020

2121
import io.smallrye.common.annotation.Identifier;
2222
import jakarta.annotation.Nonnull;
23-
import jakarta.enterprise.context.RequestScoped;
23+
import jakarta.enterprise.context.ApplicationScoped;
2424
import jakarta.inject.Inject;
2525
import java.util.Map;
2626
import java.util.Set;
@@ -34,7 +34,7 @@
3434
import org.apache.polaris.service.config.RealmEntityManagerFactory;
3535

3636
/** A {@link FileIOFactory} that translates WASB paths to ABFS ones */
37-
@RequestScoped
37+
@ApplicationScoped
3838
@Identifier("wasb")
3939
public class WasbTranslatingFileIOFactory implements FileIOFactory {
4040

0 commit comments

Comments
 (0)