This is an automated email from the ASF dual-hosted git repository. tkalkirill pushed a commit to branch main in repository https://gitbox.apache.org/repos/asf/ignite-3.git
The following commit(s) were added to refs/heads/main by this push: new 90ec6df3e5 IGNITE-20706 Rename CatalogSchemaManager to SchemaManager (#2733) 90ec6df3e5 is described below commit 90ec6df3e5618240d1e7099ea25126beb4e21121 Author: Roman Puchkovskiy <roman.puchkovs...@gmail.com> AuthorDate: Fri Oct 20 17:33:35 2023 +0400 IGNITE-20706 Rename CatalogSchemaManager to SchemaManager (#2733) --- .../apache/ignite/internal/index/IndexManager.java | 6 +- .../ignite/internal/index/IndexManagerTest.java | 4 +- .../rebalance/ItRebalanceDistributedTest.java | 6 +- .../runner/app/ItIgniteNodeRestartTest.java | 4 +- .../org/apache/ignite/internal/app/IgniteImpl.java | 6 +- ...atalogSchemaManager.java => SchemaManager.java} | 4 +- ...hemaManagerTest.java => SchemaManagerTest.java} | 8 +- .../internal/sql/engine/SqlQueryProcessor.java | 6 +- .../engine/exec/ExecutableTableRegistryImpl.java | 6 +- .../internal/sql/engine/exec/exp/RexImpTable.java | 217 +++++++++------------ .../sql/engine/exec/exp/RexToLixTranslator.java | 50 +++-- .../exec/ExecutableTableRegistrySelfTest.java | 4 +- .../internal/table/distributed/TableManager.java | 6 +- .../distributed/schema/NonHistoricSchemas.java | 8 +- .../table/distributed/TableManagerTest.java | 6 +- 15 files changed, 151 insertions(+), 190 deletions(-) diff --git a/modules/index/src/main/java/org/apache/ignite/internal/index/IndexManager.java b/modules/index/src/main/java/org/apache/ignite/internal/index/IndexManager.java index d00df0cc45..6d61eeb2f1 100644 --- a/modules/index/src/main/java/org/apache/ignite/internal/index/IndexManager.java +++ b/modules/index/src/main/java/org/apache/ignite/internal/index/IndexManager.java @@ -49,10 +49,10 @@ import org.apache.ignite.internal.metastorage.MetaStorageManager; import org.apache.ignite.internal.schema.BinaryRow; import org.apache.ignite.internal.schema.BinaryRowConverter; import org.apache.ignite.internal.schema.BinaryTuple; -import org.apache.ignite.internal.schema.CatalogSchemaManager; import org.apache.ignite.internal.schema.Column; import org.apache.ignite.internal.schema.ColumnsExtractor; import org.apache.ignite.internal.schema.SchemaDescriptor; +import org.apache.ignite.internal.schema.SchemaManager; import org.apache.ignite.internal.schema.SchemaRegistry; import org.apache.ignite.internal.storage.MvPartitionStorage; import org.apache.ignite.internal.storage.engine.MvTableStorage; @@ -75,7 +75,7 @@ public class IndexManager implements IgniteComponent { private static final IgniteLogger LOG = Loggers.forClass(IndexManager.class); /** Schema manager. */ - private final CatalogSchemaManager schemaManager; + private final SchemaManager schemaManager; /** Table manager. */ private final TableManager tableManager; @@ -106,7 +106,7 @@ public class IndexManager implements IgniteComponent { * @param catalogService Catalog manager. */ public IndexManager( - CatalogSchemaManager schemaManager, + SchemaManager schemaManager, TableManager tableManager, CatalogService catalogService, MetaStorageManager metaStorageManager, diff --git a/modules/index/src/test/java/org/apache/ignite/internal/index/IndexManagerTest.java b/modules/index/src/test/java/org/apache/ignite/internal/index/IndexManagerTest.java index f8e1797965..9c5c440446 100644 --- a/modules/index/src/test/java/org/apache/ignite/internal/index/IndexManagerTest.java +++ b/modules/index/src/test/java/org/apache/ignite/internal/index/IndexManagerTest.java @@ -54,7 +54,7 @@ import org.apache.ignite.internal.manager.IgniteComponent; import org.apache.ignite.internal.metastorage.impl.MetaStorageManagerImpl; import org.apache.ignite.internal.metastorage.impl.StandaloneMetaStorageManager; import org.apache.ignite.internal.metastorage.server.SimpleInMemoryKeyValueStorage; -import org.apache.ignite.internal.schema.CatalogSchemaManager; +import org.apache.ignite.internal.schema.SchemaManager; import org.apache.ignite.internal.storage.engine.MvTableStorage; import org.apache.ignite.internal.storage.engine.StorageTableDescriptor; import org.apache.ignite.internal.table.InternalTable; @@ -101,7 +101,7 @@ public class IndexManagerTest extends BaseIgniteAbstractTest { when(tableManagerMock.localPartitionSetAsync(anyLong(), anyInt())).thenReturn(completedFuture(PartitionSet.EMPTY_SET)); - CatalogSchemaManager schManager = mock(CatalogSchemaManager.class); + SchemaManager schManager = mock(SchemaManager.class); when(schManager.schemaRegistry(anyLong(), anyInt())).thenReturn(completedFuture(null)); diff --git a/modules/runner/src/integrationTest/java/org/apache/ignite/internal/rebalance/ItRebalanceDistributedTest.java b/modules/runner/src/integrationTest/java/org/apache/ignite/internal/rebalance/ItRebalanceDistributedTest.java index 7c4395cf7f..5f46a14ceb 100644 --- a/modules/runner/src/integrationTest/java/org/apache/ignite/internal/rebalance/ItRebalanceDistributedTest.java +++ b/modules/runner/src/integrationTest/java/org/apache/ignite/internal/rebalance/ItRebalanceDistributedTest.java @@ -140,7 +140,7 @@ import org.apache.ignite.internal.replicator.Replica; import org.apache.ignite.internal.replicator.ReplicaManager; import org.apache.ignite.internal.replicator.ReplicaService; import org.apache.ignite.internal.rest.configuration.RestConfiguration; -import org.apache.ignite.internal.schema.CatalogSchemaManager; +import org.apache.ignite.internal.schema.SchemaManager; import org.apache.ignite.internal.schema.configuration.GcConfiguration; import org.apache.ignite.internal.storage.DataStorageManager; import org.apache.ignite.internal.storage.DataStorageModules; @@ -735,7 +735,7 @@ public class ItRebalanceDistributedTest extends BaseIgniteAbstractTest { private final ClusterManagementGroupManager cmgManager; - private final CatalogSchemaManager schemaManager; + private final SchemaManager schemaManager; private final CatalogManager catalogManager; @@ -931,7 +931,7 @@ public class ItRebalanceDistributedTest extends BaseIgniteAbstractTest { partitionIdleSafeTimePropagationPeriodMsSupplier ); - schemaManager = new CatalogSchemaManager(registry, catalogManager, metaStorageManager); + schemaManager = new SchemaManager(registry, catalogManager, metaStorageManager); var schemaSyncService = new SchemaSyncServiceImpl(metaStorageManager.clusterTime(), delayDurationMsSupplier); diff --git a/modules/runner/src/integrationTest/java/org/apache/ignite/internal/runner/app/ItIgniteNodeRestartTest.java b/modules/runner/src/integrationTest/java/org/apache/ignite/internal/runner/app/ItIgniteNodeRestartTest.java index 06ff12f8f4..751baf57f3 100644 --- a/modules/runner/src/integrationTest/java/org/apache/ignite/internal/runner/app/ItIgniteNodeRestartTest.java +++ b/modules/runner/src/integrationTest/java/org/apache/ignite/internal/runner/app/ItIgniteNodeRestartTest.java @@ -109,7 +109,7 @@ import org.apache.ignite.internal.raft.server.impl.JraftServerImpl; import org.apache.ignite.internal.raft.storage.impl.LocalLogStorageFactory; import org.apache.ignite.internal.replicator.ReplicaManager; import org.apache.ignite.internal.replicator.ReplicaService; -import org.apache.ignite.internal.schema.CatalogSchemaManager; +import org.apache.ignite.internal.schema.SchemaManager; import org.apache.ignite.internal.schema.configuration.GcConfiguration; import org.apache.ignite.internal.sql.engine.SqlQueryProcessor; import org.apache.ignite.internal.storage.DataStorageManager; @@ -370,7 +370,7 @@ public class ItIgniteNodeRestartTest extends BaseIgniteRestartTest { partitionIdleSafeTimePropagationPeriodMsSupplier ); - CatalogSchemaManager schemaManager = new CatalogSchemaManager(registry, catalogManager, metaStorageMgr); + SchemaManager schemaManager = new SchemaManager(registry, catalogManager, metaStorageMgr); DistributionZoneManager distributionZoneManager = new DistributionZoneManager( name, diff --git a/modules/runner/src/main/java/org/apache/ignite/internal/app/IgniteImpl.java b/modules/runner/src/main/java/org/apache/ignite/internal/app/IgniteImpl.java index 200816d19e..289dd9ec1b 100644 --- a/modules/runner/src/main/java/org/apache/ignite/internal/app/IgniteImpl.java +++ b/modules/runner/src/main/java/org/apache/ignite/internal/app/IgniteImpl.java @@ -118,7 +118,7 @@ import org.apache.ignite.internal.rest.configuration.RestConfiguration; import org.apache.ignite.internal.rest.deployment.CodeDeploymentRestFactory; import org.apache.ignite.internal.rest.metrics.MetricRestFactory; import org.apache.ignite.internal.rest.node.NodeManagementRestFactory; -import org.apache.ignite.internal.schema.CatalogSchemaManager; +import org.apache.ignite.internal.schema.SchemaManager; import org.apache.ignite.internal.schema.configuration.GcConfiguration; import org.apache.ignite.internal.security.authentication.AuthenticationManager; import org.apache.ignite.internal.security.authentication.AuthenticationManagerImpl; @@ -279,7 +279,7 @@ public class IgniteImpl implements Ignite { private final DataStorageManager dataStorageMgr; /** Schema manager. */ - private final CatalogSchemaManager schemaManager; + private final SchemaManager schemaManager; /** Metric manager. */ private final MetricManager metricManager; @@ -548,7 +548,7 @@ public class IgniteImpl implements Ignite { SchemaSyncService schemaSyncService = new SchemaSyncServiceImpl(metaStorageMgr.clusterTime(), delayDurationMsSupplier); - schemaManager = new CatalogSchemaManager(registry, catalogManager, metaStorageMgr); + schemaManager = new SchemaManager(registry, catalogManager, metaStorageMgr); distributionZoneManager = new DistributionZoneManager( name, diff --git a/modules/schema/src/main/java/org/apache/ignite/internal/schema/CatalogSchemaManager.java b/modules/schema/src/main/java/org/apache/ignite/internal/schema/SchemaManager.java similarity index 99% rename from modules/schema/src/main/java/org/apache/ignite/internal/schema/CatalogSchemaManager.java rename to modules/schema/src/main/java/org/apache/ignite/internal/schema/SchemaManager.java index a20d4ff6ac..260dc9fa1e 100644 --- a/modules/schema/src/main/java/org/apache/ignite/internal/schema/CatalogSchemaManager.java +++ b/modules/schema/src/main/java/org/apache/ignite/internal/schema/SchemaManager.java @@ -62,7 +62,7 @@ import org.jetbrains.annotations.Nullable; /** * This class services management of table schemas. */ -public class CatalogSchemaManager implements IgniteComponent { +public class SchemaManager implements IgniteComponent { /** Schema history key predicate part. */ private static final String SCHEMA_STORE_PREFIX = ".sch-hist."; private static final String LATEST_SCHEMA_VERSION_STORE_SUFFIX = ".sch-hist-latest"; @@ -82,7 +82,7 @@ public class CatalogSchemaManager implements IgniteComponent { private final MetaStorageManager metastorageMgr; /** Constructor. */ - public CatalogSchemaManager( + public SchemaManager( Consumer<LongFunction<CompletableFuture<?>>> registry, CatalogService catalogService, MetaStorageManager metastorageMgr diff --git a/modules/schema/src/test/java/org/apache/ignite/internal/schema/CatalogSchemaManagerTest.java b/modules/schema/src/test/java/org/apache/ignite/internal/schema/SchemaManagerTest.java similarity index 98% rename from modules/schema/src/test/java/org/apache/ignite/internal/schema/CatalogSchemaManagerTest.java rename to modules/schema/src/test/java/org/apache/ignite/internal/schema/SchemaManagerTest.java index f543fb2db1..81ceddab4b 100644 --- a/modules/schema/src/test/java/org/apache/ignite/internal/schema/CatalogSchemaManagerTest.java +++ b/modules/schema/src/test/java/org/apache/ignite/internal/schema/SchemaManagerTest.java @@ -81,7 +81,7 @@ import org.mockito.quality.Strictness; @ExtendWith(MockitoExtension.class) @MockitoSettings(strictness = Strictness.LENIENT) -class CatalogSchemaManagerTest extends BaseIgniteAbstractTest { +class SchemaManagerTest extends BaseIgniteAbstractTest { private static final String SCHEMA_STORE_PREFIX = ".sch-hist."; private static final String LATEST_SCHEMA_VERSION_STORE_SUFFIX = ".sch-hist-latest"; @@ -107,7 +107,7 @@ class CatalogSchemaManagerTest extends BaseIgniteAbstractTest { private final SimpleInMemoryKeyValueStorage metaStorageKvStorage = new SimpleInMemoryKeyValueStorage("test"); - private CatalogSchemaManager schemaManager; + private SchemaManager schemaManager; private EventListener<CatalogEventParameters> tableCreatedListener; private EventListener<CatalogEventParameters> tableAlteredListener; @@ -132,7 +132,7 @@ class CatalogSchemaManagerTest extends BaseIgniteAbstractTest { return null; }).when(catalogService).listen(eq(CatalogEvent.TABLE_ALTER), any()); - schemaManager = new CatalogSchemaManager(registry, catalogService, metaStorageManager); + schemaManager = new SchemaManager(registry, catalogService, metaStorageManager); schemaManager.start(); assertThat("Watches were not deployed", metaStorageManager.deployWatches(), willCompleteSuccessfully()); @@ -490,7 +490,7 @@ class CatalogSchemaManagerTest extends BaseIgniteAbstractTest { when(catalogService.tables(anyInt())).thenReturn(List.of(tableDescriptorAfterColumnAddition())); doReturn(45L).when(metaStorageManager).appliedRevision(); - schemaManager = new CatalogSchemaManager(registry, catalogService, metaStorageManager); + schemaManager = new SchemaManager(registry, catalogService, metaStorageManager); schemaManager.start(); SchemaRegistry schemaRegistry = schemaManager.schemaRegistry(TABLE_ID); diff --git a/modules/sql-engine/src/main/java/org/apache/ignite/internal/sql/engine/SqlQueryProcessor.java b/modules/sql-engine/src/main/java/org/apache/ignite/internal/sql/engine/SqlQueryProcessor.java index c6f506e0f2..cacad79fb3 100644 --- a/modules/sql-engine/src/main/java/org/apache/ignite/internal/sql/engine/SqlQueryProcessor.java +++ b/modules/sql-engine/src/main/java/org/apache/ignite/internal/sql/engine/SqlQueryProcessor.java @@ -49,7 +49,7 @@ import org.apache.ignite.internal.logger.IgniteLogger; import org.apache.ignite.internal.logger.Loggers; import org.apache.ignite.internal.metrics.MetricManager; import org.apache.ignite.internal.replicator.ReplicaService; -import org.apache.ignite.internal.schema.CatalogSchemaManager; +import org.apache.ignite.internal.schema.SchemaManager; import org.apache.ignite.internal.sql.engine.exec.ExchangeServiceImpl; import org.apache.ignite.internal.sql.engine.exec.ExecutableTableRegistryImpl; import org.apache.ignite.internal.sql.engine.exec.ExecutionDependencyResolverImpl; @@ -154,7 +154,7 @@ public class SqlQueryProcessor implements QueryProcessor { private final TableManager tableManager; - private final CatalogSchemaManager schemaManager; + private final SchemaManager schemaManager; private final DataStorageManager dataStorageManager; @@ -197,7 +197,7 @@ public class SqlQueryProcessor implements QueryProcessor { ClusterService clusterSrvc, LogicalTopologyService logicalTopologyService, TableManager tableManager, - CatalogSchemaManager schemaManager, + SchemaManager schemaManager, DataStorageManager dataStorageManager, Supplier<Map<String, Map<String, Class<?>>>> dataStorageFieldsSupplier, ReplicaService replicaService, diff --git a/modules/sql-engine/src/main/java/org/apache/ignite/internal/sql/engine/exec/ExecutableTableRegistryImpl.java b/modules/sql-engine/src/main/java/org/apache/ignite/internal/sql/engine/exec/ExecutableTableRegistryImpl.java index 819f67f7a9..e8c2c35421 100644 --- a/modules/sql-engine/src/main/java/org/apache/ignite/internal/sql/engine/exec/ExecutableTableRegistryImpl.java +++ b/modules/sql-engine/src/main/java/org/apache/ignite/internal/sql/engine/exec/ExecutableTableRegistryImpl.java @@ -23,8 +23,8 @@ import java.util.concurrent.CompletableFuture; import java.util.concurrent.ConcurrentMap; import org.apache.ignite.internal.hlc.HybridClock; import org.apache.ignite.internal.replicator.ReplicaService; -import org.apache.ignite.internal.schema.CatalogSchemaManager; import org.apache.ignite.internal.schema.SchemaDescriptor; +import org.apache.ignite.internal.schema.SchemaManager; import org.apache.ignite.internal.schema.SchemaRegistry; import org.apache.ignite.internal.sql.engine.schema.TableDescriptor; import org.apache.ignite.internal.table.InternalTable; @@ -37,7 +37,7 @@ public class ExecutableTableRegistryImpl implements ExecutableTableRegistry { private final TableManager tableManager; - private final CatalogSchemaManager schemaManager; + private final SchemaManager schemaManager; private final ReplicaService replicaService; @@ -47,7 +47,7 @@ public class ExecutableTableRegistryImpl implements ExecutableTableRegistry { final ConcurrentMap<CacheKey, CompletableFuture<ExecutableTable>> tableCache; /** Constructor. */ - public ExecutableTableRegistryImpl(TableManager tableManager, CatalogSchemaManager schemaManager, + public ExecutableTableRegistryImpl(TableManager tableManager, SchemaManager schemaManager, ReplicaService replicaService, HybridClock clock, int cacheSize) { this.tableManager = tableManager; diff --git a/modules/sql-engine/src/main/java/org/apache/ignite/internal/sql/engine/exec/exp/RexImpTable.java b/modules/sql-engine/src/main/java/org/apache/ignite/internal/sql/engine/exec/exp/RexImpTable.java index d23de90c24..6a82dd4ee1 100644 --- a/modules/sql-engine/src/main/java/org/apache/ignite/internal/sql/engine/exec/exp/RexImpTable.java +++ b/modules/sql-engine/src/main/java/org/apache/ignite/internal/sql/engine/exec/exp/RexImpTable.java @@ -18,108 +18,8 @@ package org.apache.ignite.internal.sql.engine.exec.exp; //CHECKSTYLE:OFF -import org.apache.calcite.DataContext; -import org.apache.calcite.adapter.enumerable.AggAddContext; -import org.apache.calcite.adapter.enumerable.AggContext; -import org.apache.calcite.adapter.enumerable.AggImplementor; -import org.apache.calcite.adapter.enumerable.AggResetContext; -import org.apache.calcite.adapter.enumerable.AggResultContext; -import org.apache.calcite.adapter.enumerable.EnumUtils; -import org.apache.calcite.adapter.enumerable.EnumerableMatch; -import org.apache.calcite.adapter.enumerable.NullPolicy; -import org.apache.calcite.adapter.enumerable.StrictAggImplementor; -import org.apache.calcite.adapter.enumerable.StrictWinAggImplementor; -import org.apache.calcite.adapter.enumerable.TableFunctionCallImplementor; -import org.apache.calcite.adapter.enumerable.WinAggAddContext; -import org.apache.calcite.adapter.enumerable.WinAggContext; -import org.apache.calcite.adapter.enumerable.WinAggImplementor; -import org.apache.calcite.adapter.enumerable.WinAggResultContext; -import org.apache.calcite.adapter.java.JavaTypeFactory; -import org.apache.calcite.avatica.util.ByteString; -import org.apache.calcite.avatica.util.DateTimeUtils; -import org.apache.calcite.avatica.util.TimeUnit; -import org.apache.calcite.avatica.util.TimeUnitRange; -import org.apache.calcite.linq4j.tree.BinaryExpression; -import org.apache.calcite.linq4j.tree.BlockBuilder; -import org.apache.calcite.linq4j.tree.BlockStatement; -import org.apache.calcite.linq4j.tree.ConstantExpression; -import org.apache.calcite.linq4j.tree.Expression; -import org.apache.calcite.linq4j.tree.ExpressionType; -import org.apache.calcite.linq4j.tree.Expressions; -import org.apache.calcite.linq4j.tree.MemberExpression; -import org.apache.calcite.linq4j.tree.MethodCallExpression; -import org.apache.calcite.linq4j.tree.NewExpression; -import org.apache.calcite.linq4j.tree.OptimizeShuttle; -import org.apache.calcite.linq4j.tree.ParameterExpression; -import org.apache.calcite.linq4j.tree.Primitive; -import org.apache.calcite.rel.type.RelDataType; -import org.apache.calcite.rel.type.RelDataTypeFactory; -import org.apache.calcite.rel.type.RelDataTypeFactoryImpl; -import org.apache.calcite.rex.RexCall; -import org.apache.calcite.rex.RexInputRef; -import org.apache.calcite.rex.RexLiteral; -import org.apache.calcite.rex.RexNode; -import org.apache.calcite.rex.RexPatternFieldRef; -import org.apache.calcite.runtime.SqlFunctions; -import org.apache.calcite.schema.FunctionContext; -import org.apache.calcite.schema.ImplementableAggFunction; -import org.apache.calcite.schema.QueryableTable; -import org.apache.calcite.schema.SchemaPlus; -import org.apache.calcite.schema.impl.AggregateFunctionImpl; -import org.apache.calcite.sql.SqlAggFunction; -import org.apache.calcite.sql.SqlBinaryOperator; -import org.apache.calcite.sql.SqlJsonConstructorNullClause; -import org.apache.calcite.sql.SqlJsonEmptyOrError; -import org.apache.calcite.sql.SqlJsonValueEmptyOrErrorBehavior; -import org.apache.calcite.sql.SqlKind; -import org.apache.calcite.sql.SqlMatchFunction; -import org.apache.calcite.sql.SqlOperator; -import org.apache.calcite.sql.SqlTableFunction; -import org.apache.calcite.sql.SqlTypeConstructorFunction; -import org.apache.calcite.sql.SqlWindowTableFunction; -import org.apache.calcite.sql.fun.SqlItemOperator; -import org.apache.calcite.sql.fun.SqlJsonArrayAggAggFunction; -import org.apache.calcite.sql.fun.SqlJsonObjectAggAggFunction; -import org.apache.calcite.sql.fun.SqlQuantifyOperator; -import org.apache.calcite.sql.fun.SqlStdOperatorTable; -import org.apache.calcite.sql.fun.SqlTrimFunction; -import org.apache.calcite.sql.type.SqlTypeName; -import org.apache.calcite.sql.type.SqlTypeUtil; -import org.apache.calcite.sql.validate.SqlUserDefinedAggFunction; -import org.apache.calcite.sql.validate.SqlUserDefinedFunction; -import org.apache.calcite.sql.validate.SqlUserDefinedTableFunction; -import org.apache.calcite.sql.validate.SqlUserDefinedTableMacro; -import org.apache.calcite.util.BuiltInMethod; -import org.apache.calcite.util.Util; - -import com.google.common.collect.ImmutableList; -import com.google.common.collect.ImmutableMap; -import com.google.common.collect.Iterables; - -import org.apache.ignite.internal.sql.engine.sql.fun.IgniteSqlOperatorTable; -import org.apache.ignite.internal.sql.engine.util.IgniteMethod; -import org.checkerframework.checker.nullness.qual.Nullable; - -import java.lang.reflect.Constructor; -import java.lang.reflect.InvocationTargetException; -import java.lang.reflect.Method; -import java.lang.reflect.Modifier; -import java.lang.reflect.Type; -import java.math.BigDecimal; -import java.math.RoundingMode; -import java.util.ArrayList; -import java.util.Arrays; -import java.util.Collections; -import java.util.HashMap; -import java.util.LinkedHashMap; -import java.util.List; -import java.util.Map; -import java.util.Objects; -import java.util.function.Supplier; -import java.util.stream.Collectors; - -import static com.google.common.base.Preconditions.checkArgument; +import static java.util.Objects.requireNonNull; import static org.apache.calcite.adapter.enumerable.EnumUtils.generateCollatorExpression; import static org.apache.calcite.linq4j.tree.ExpressionType.Add; import static org.apache.calcite.linq4j.tree.ExpressionType.Divide; @@ -275,12 +175,6 @@ import static org.apache.calcite.sql.fun.SqlLibraryOperators.UNIX_SECONDS; import static org.apache.calcite.sql.fun.SqlLibraryOperators.XML_TRANSFORM; import static org.apache.calcite.sql.fun.SqlStdOperatorTable.ABS; import static org.apache.calcite.sql.fun.SqlStdOperatorTable.ACOS; -import static org.apache.calcite.sql.fun.SqlStdOperatorTable.ALL_EQ; -import static org.apache.calcite.sql.fun.SqlStdOperatorTable.ALL_GE; -import static org.apache.calcite.sql.fun.SqlStdOperatorTable.ALL_GT; -import static org.apache.calcite.sql.fun.SqlStdOperatorTable.ALL_LE; -import static org.apache.calcite.sql.fun.SqlStdOperatorTable.ALL_LT; -import static org.apache.calcite.sql.fun.SqlStdOperatorTable.ALL_NE; import static org.apache.calcite.sql.fun.SqlStdOperatorTable.AND; import static org.apache.calcite.sql.fun.SqlStdOperatorTable.ANY_VALUE; import static org.apache.calcite.sql.fun.SqlStdOperatorTable.ARG_MAX; @@ -326,14 +220,12 @@ import static org.apache.calcite.sql.fun.SqlStdOperatorTable.EQUALS; import static org.apache.calcite.sql.fun.SqlStdOperatorTable.EVERY; import static org.apache.calcite.sql.fun.SqlStdOperatorTable.EXP; import static org.apache.calcite.sql.fun.SqlStdOperatorTable.EXTRACT; -import static org.apache.calcite.sql.fun.SqlStdOperatorTable.FIRST_VALUE; import static org.apache.calcite.sql.fun.SqlStdOperatorTable.FLOOR; import static org.apache.calcite.sql.fun.SqlStdOperatorTable.FUSION; import static org.apache.calcite.sql.fun.SqlStdOperatorTable.GREATER_THAN; import static org.apache.calcite.sql.fun.SqlStdOperatorTable.GREATER_THAN_OR_EQUAL; import static org.apache.calcite.sql.fun.SqlStdOperatorTable.GROUPING; import static org.apache.calcite.sql.fun.SqlStdOperatorTable.GROUPING_ID; -import static org.apache.calcite.sql.fun.SqlStdOperatorTable.HOP; import static org.apache.calcite.sql.fun.SqlStdOperatorTable.INITCAP; import static org.apache.calcite.sql.fun.SqlStdOperatorTable.INTERSECTION; import static org.apache.calcite.sql.fun.SqlStdOperatorTable.IS_A_SET; @@ -365,11 +257,7 @@ import static org.apache.calcite.sql.fun.SqlStdOperatorTable.JSON_QUERY; import static org.apache.calcite.sql.fun.SqlStdOperatorTable.JSON_TYPE_OPERATOR; import static org.apache.calcite.sql.fun.SqlStdOperatorTable.JSON_VALUE; import static org.apache.calcite.sql.fun.SqlStdOperatorTable.JSON_VALUE_EXPRESSION; -import static org.apache.calcite.sql.fun.SqlStdOperatorTable.LAG; -import static org.apache.calcite.sql.fun.SqlStdOperatorTable.LAST; import static org.apache.calcite.sql.fun.SqlStdOperatorTable.LAST_DAY; -import static org.apache.calcite.sql.fun.SqlStdOperatorTable.LAST_VALUE; -import static org.apache.calcite.sql.fun.SqlStdOperatorTable.LEAD; import static org.apache.calcite.sql.fun.SqlStdOperatorTable.LESS_THAN; import static org.apache.calcite.sql.fun.SqlStdOperatorTable.LESS_THAN_OR_EQUAL; import static org.apache.calcite.sql.fun.SqlStdOperatorTable.LIKE; @@ -398,8 +286,6 @@ import static org.apache.calcite.sql.fun.SqlStdOperatorTable.NEXT_VALUE; import static org.apache.calcite.sql.fun.SqlStdOperatorTable.NOT; import static org.apache.calcite.sql.fun.SqlStdOperatorTable.NOT_EQUALS; import static org.apache.calcite.sql.fun.SqlStdOperatorTable.NOT_SUBMULTISET_OF; -import static org.apache.calcite.sql.fun.SqlStdOperatorTable.NTH_VALUE; -import static org.apache.calcite.sql.fun.SqlStdOperatorTable.NTILE; import static org.apache.calcite.sql.fun.SqlStdOperatorTable.OCTET_LENGTH; import static org.apache.calcite.sql.fun.SqlStdOperatorTable.OR; import static org.apache.calcite.sql.fun.SqlStdOperatorTable.OVERLAY; @@ -417,7 +303,6 @@ import static org.apache.calcite.sql.fun.SqlStdOperatorTable.REPLACE; import static org.apache.calcite.sql.fun.SqlStdOperatorTable.ROUND; import static org.apache.calcite.sql.fun.SqlStdOperatorTable.ROW; import static org.apache.calcite.sql.fun.SqlStdOperatorTable.ROW_NUMBER; -import static org.apache.calcite.sql.fun.SqlStdOperatorTable.SESSION; import static org.apache.calcite.sql.fun.SqlStdOperatorTable.SESSION_USER; import static org.apache.calcite.sql.fun.SqlStdOperatorTable.SIGN; import static org.apache.calcite.sql.fun.SqlStdOperatorTable.SIMILAR_TO; @@ -425,12 +310,6 @@ import static org.apache.calcite.sql.fun.SqlStdOperatorTable.SIN; import static org.apache.calcite.sql.fun.SqlStdOperatorTable.SINGLE_VALUE; import static org.apache.calcite.sql.fun.SqlStdOperatorTable.SLICE; import static org.apache.calcite.sql.fun.SqlStdOperatorTable.SOME; -import static org.apache.calcite.sql.fun.SqlStdOperatorTable.SOME_EQ; -import static org.apache.calcite.sql.fun.SqlStdOperatorTable.SOME_GE; -import static org.apache.calcite.sql.fun.SqlStdOperatorTable.SOME_GT; -import static org.apache.calcite.sql.fun.SqlStdOperatorTable.SOME_LE; -import static org.apache.calcite.sql.fun.SqlStdOperatorTable.SOME_LT; -import static org.apache.calcite.sql.fun.SqlStdOperatorTable.SOME_NE; import static org.apache.calcite.sql.fun.SqlStdOperatorTable.STRUCT_ACCESS; import static org.apache.calcite.sql.fun.SqlStdOperatorTable.SUBMULTISET_OF; import static org.apache.calcite.sql.fun.SqlStdOperatorTable.SUBSTRING; @@ -443,13 +322,10 @@ import static org.apache.calcite.sql.fun.SqlStdOperatorTable.TIMESTAMP_DIFF; import static org.apache.calcite.sql.fun.SqlStdOperatorTable.TRANSLATE; import static org.apache.calcite.sql.fun.SqlStdOperatorTable.TRIM; import static org.apache.calcite.sql.fun.SqlStdOperatorTable.TRUNCATE; -import static org.apache.calcite.sql.fun.SqlStdOperatorTable.TUMBLE; import static org.apache.calcite.sql.fun.SqlStdOperatorTable.UNARY_MINUS; import static org.apache.calcite.sql.fun.SqlStdOperatorTable.UNARY_PLUS; import static org.apache.calcite.sql.fun.SqlStdOperatorTable.UPPER; import static org.apache.calcite.sql.fun.SqlStdOperatorTable.USER; - -import static java.util.Objects.requireNonNull; import static org.apache.ignite.internal.sql.engine.sql.fun.IgniteSqlOperatorTable.NULL_BOUND; import static org.apache.ignite.internal.sql.engine.sql.fun.IgniteSqlOperatorTable.SUBSTR; import static org.apache.ignite.internal.sql.engine.sql.fun.IgniteSqlOperatorTable.SYSTEM_RANGE; @@ -460,6 +336,97 @@ import static org.apache.ignite.internal.sql.engine.util.IgniteMethod.LEAST2; import static org.apache.ignite.internal.sql.engine.util.IgniteMethod.LENGTH; import static org.apache.ignite.internal.sql.engine.util.IgniteMethod.RAND_UUID; +import com.google.common.collect.ImmutableList; +import com.google.common.collect.ImmutableMap; +import com.google.common.collect.Iterables; +import java.lang.reflect.Constructor; +import java.lang.reflect.InvocationTargetException; +import java.lang.reflect.Method; +import java.lang.reflect.Modifier; +import java.lang.reflect.Type; +import java.math.BigDecimal; +import java.math.RoundingMode; +import java.util.ArrayList; +import java.util.Arrays; +import java.util.Collections; +import java.util.HashMap; +import java.util.LinkedHashMap; +import java.util.List; +import java.util.Map; +import java.util.Objects; +import java.util.function.Supplier; +import java.util.stream.Collectors; +import org.apache.calcite.DataContext; +import org.apache.calcite.adapter.enumerable.AggAddContext; +import org.apache.calcite.adapter.enumerable.AggContext; +import org.apache.calcite.adapter.enumerable.AggImplementor; +import org.apache.calcite.adapter.enumerable.AggResetContext; +import org.apache.calcite.adapter.enumerable.AggResultContext; +import org.apache.calcite.adapter.enumerable.EnumUtils; +import org.apache.calcite.adapter.enumerable.NullPolicy; +import org.apache.calcite.adapter.enumerable.StrictAggImplementor; +import org.apache.calcite.adapter.enumerable.StrictWinAggImplementor; +import org.apache.calcite.adapter.enumerable.TableFunctionCallImplementor; +import org.apache.calcite.adapter.enumerable.WinAggAddContext; +import org.apache.calcite.adapter.enumerable.WinAggContext; +import org.apache.calcite.adapter.enumerable.WinAggImplementor; +import org.apache.calcite.adapter.enumerable.WinAggResultContext; +import org.apache.calcite.adapter.java.JavaTypeFactory; +import org.apache.calcite.avatica.util.ByteString; +import org.apache.calcite.avatica.util.DateTimeUtils; +import org.apache.calcite.avatica.util.TimeUnit; +import org.apache.calcite.avatica.util.TimeUnitRange; +import org.apache.calcite.linq4j.tree.BlockBuilder; +import org.apache.calcite.linq4j.tree.ConstantExpression; +import org.apache.calcite.linq4j.tree.Expression; +import org.apache.calcite.linq4j.tree.ExpressionType; +import org.apache.calcite.linq4j.tree.Expressions; +import org.apache.calcite.linq4j.tree.MemberExpression; +import org.apache.calcite.linq4j.tree.MethodCallExpression; +import org.apache.calcite.linq4j.tree.NewExpression; +import org.apache.calcite.linq4j.tree.OptimizeShuttle; +import org.apache.calcite.linq4j.tree.ParameterExpression; +import org.apache.calcite.linq4j.tree.Primitive; +import org.apache.calcite.rel.type.RelDataType; +import org.apache.calcite.rel.type.RelDataTypeFactory; +import org.apache.calcite.rel.type.RelDataTypeFactoryImpl; +import org.apache.calcite.rex.RexCall; +import org.apache.calcite.rex.RexLiteral; +import org.apache.calcite.rex.RexNode; +import org.apache.calcite.runtime.SqlFunctions; +import org.apache.calcite.schema.FunctionContext; +import org.apache.calcite.schema.ImplementableAggFunction; +import org.apache.calcite.schema.QueryableTable; +import org.apache.calcite.schema.SchemaPlus; +import org.apache.calcite.schema.impl.AggregateFunctionImpl; +import org.apache.calcite.sql.SqlAggFunction; +import org.apache.calcite.sql.SqlBinaryOperator; +import org.apache.calcite.sql.SqlJsonConstructorNullClause; +import org.apache.calcite.sql.SqlJsonEmptyOrError; +import org.apache.calcite.sql.SqlJsonValueEmptyOrErrorBehavior; +import org.apache.calcite.sql.SqlKind; +import org.apache.calcite.sql.SqlMatchFunction; +import org.apache.calcite.sql.SqlOperator; +import org.apache.calcite.sql.SqlTableFunction; +import org.apache.calcite.sql.SqlTypeConstructorFunction; +import org.apache.calcite.sql.SqlWindowTableFunction; +import org.apache.calcite.sql.fun.SqlItemOperator; +import org.apache.calcite.sql.fun.SqlJsonArrayAggAggFunction; +import org.apache.calcite.sql.fun.SqlJsonObjectAggAggFunction; +import org.apache.calcite.sql.fun.SqlStdOperatorTable; +import org.apache.calcite.sql.fun.SqlTrimFunction; +import org.apache.calcite.sql.type.SqlTypeName; +import org.apache.calcite.sql.type.SqlTypeUtil; +import org.apache.calcite.sql.validate.SqlUserDefinedAggFunction; +import org.apache.calcite.sql.validate.SqlUserDefinedFunction; +import org.apache.calcite.sql.validate.SqlUserDefinedTableFunction; +import org.apache.calcite.sql.validate.SqlUserDefinedTableMacro; +import org.apache.calcite.util.BuiltInMethod; +import org.apache.calcite.util.Util; +import org.apache.ignite.internal.sql.engine.sql.fun.IgniteSqlOperatorTable; +import org.apache.ignite.internal.sql.engine.util.IgniteMethod; +import org.checkerframework.checker.nullness.qual.Nullable; + /** * Contains implementations of Rex operators as Java code. * Changes in comparison with original code: diff --git a/modules/sql-engine/src/main/java/org/apache/ignite/internal/sql/engine/exec/exp/RexToLixTranslator.java b/modules/sql-engine/src/main/java/org/apache/ignite/internal/sql/engine/exec/exp/RexToLixTranslator.java index 7fcb144f22..59e7c2bb95 100644 --- a/modules/sql-engine/src/main/java/org/apache/ignite/internal/sql/engine/exec/exp/RexToLixTranslator.java +++ b/modules/sql-engine/src/main/java/org/apache/ignite/internal/sql/engine/exec/exp/RexToLixTranslator.java @@ -18,6 +18,28 @@ package org.apache.ignite.internal.sql.engine.exec.exp; //CHECKSTYLE:OFF + +import static java.util.Objects.requireNonNull; +import static org.apache.calcite.sql.fun.SqlLibraryOperators.TRANSLATE3; +import static org.apache.calcite.sql.fun.SqlStdOperatorTable.CASE; +import static org.apache.calcite.sql.fun.SqlStdOperatorTable.CHAR_LENGTH; +import static org.apache.calcite.sql.fun.SqlStdOperatorTable.OCTET_LENGTH; +import static org.apache.calcite.sql.fun.SqlStdOperatorTable.SEARCH; +import static org.apache.calcite.sql.fun.SqlStdOperatorTable.SUBSTRING; +import static org.apache.calcite.sql.fun.SqlStdOperatorTable.UPPER; + +import com.google.common.base.CaseFormat; +import com.google.common.collect.ImmutableList; +import com.google.common.collect.ImmutableMap; +import java.lang.reflect.Constructor; +import java.lang.reflect.Method; +import java.lang.reflect.Modifier; +import java.lang.reflect.Type; +import java.math.BigDecimal; +import java.util.ArrayList; +import java.util.HashMap; +import java.util.List; +import java.util.Map; import java.util.function.Supplier; import org.apache.calcite.DataContext; import org.apache.calcite.adapter.enumerable.EnumUtils; @@ -59,7 +81,6 @@ import org.apache.calcite.runtime.SpatialTypeFunctions; import org.apache.calcite.schema.FunctionContext; import org.apache.calcite.sql.SqlIntervalQualifier; import org.apache.calcite.sql.SqlOperator; -import org.apache.calcite.sql.SqlWindowTableFunction; import org.apache.calcite.sql.fun.SqlStdOperatorTable; import org.apache.calcite.sql.parser.SqlParserPos; import org.apache.calcite.sql.type.SqlTypeName; @@ -68,38 +89,11 @@ import org.apache.calcite.sql.validate.SqlConformance; import org.apache.calcite.util.BuiltInMethod; import org.apache.calcite.util.ControlFlowException; import org.apache.calcite.util.Pair; -import org.apache.calcite.util.Util; - -import com.google.common.base.CaseFormat; -import com.google.common.collect.ImmutableList; -import com.google.common.collect.ImmutableMap; - import org.apache.ignite.internal.sql.engine.type.IgniteTypeFactory; import org.apache.ignite.internal.sql.engine.util.IgniteMethod; import org.checkerframework.checker.nullness.qual.Nullable; import org.locationtech.jts.geom.Geometry; -import java.lang.reflect.Constructor; -import java.lang.reflect.Method; -import java.lang.reflect.Modifier; -import java.lang.reflect.Type; -import java.math.BigDecimal; -import java.util.ArrayList; -import java.util.HashMap; -import java.util.List; -import java.util.Map; - -import static org.apache.calcite.sql.fun.SqlLibraryOperators.TRANSLATE3; -import static org.apache.calcite.sql.fun.SqlStdOperatorTable.CASE; -import static org.apache.calcite.sql.fun.SqlStdOperatorTable.CHAR_LENGTH; -import static org.apache.calcite.sql.fun.SqlStdOperatorTable.OCTET_LENGTH; -import static org.apache.calcite.sql.fun.SqlStdOperatorTable.PREV; -import static org.apache.calcite.sql.fun.SqlStdOperatorTable.SEARCH; -import static org.apache.calcite.sql.fun.SqlStdOperatorTable.SUBSTRING; -import static org.apache.calcite.sql.fun.SqlStdOperatorTable.UPPER; - -import static java.util.Objects.requireNonNull; - /** * Translates {@link org.apache.calcite.rex.RexNode REX expressions} to * {@link Expression linq4j expressions}. diff --git a/modules/sql-engine/src/test/java/org/apache/ignite/internal/sql/engine/exec/ExecutableTableRegistrySelfTest.java b/modules/sql-engine/src/test/java/org/apache/ignite/internal/sql/engine/exec/ExecutableTableRegistrySelfTest.java index e9c2ba77cb..cb04a37fb8 100644 --- a/modules/sql-engine/src/test/java/org/apache/ignite/internal/sql/engine/exec/ExecutableTableRegistrySelfTest.java +++ b/modules/sql-engine/src/test/java/org/apache/ignite/internal/sql/engine/exec/ExecutableTableRegistrySelfTest.java @@ -26,9 +26,9 @@ import java.util.concurrent.CompletableFuture; import org.apache.ignite.internal.TestHybridClock; import org.apache.ignite.internal.hlc.HybridClock; import org.apache.ignite.internal.replicator.ReplicaService; -import org.apache.ignite.internal.schema.CatalogSchemaManager; import org.apache.ignite.internal.schema.Column; import org.apache.ignite.internal.schema.SchemaDescriptor; +import org.apache.ignite.internal.schema.SchemaManager; import org.apache.ignite.internal.schema.SchemaRegistry; import org.apache.ignite.internal.sql.engine.schema.TableDescriptor; import org.apache.ignite.internal.table.InternalTable; @@ -57,7 +57,7 @@ public class ExecutableTableRegistrySelfTest extends BaseIgniteAbstractTest { private TableManager tableManager; @Mock - private CatalogSchemaManager schemaManager; + private SchemaManager schemaManager; @Mock private TableDescriptor descriptor; diff --git a/modules/table/src/main/java/org/apache/ignite/internal/table/distributed/TableManager.java b/modules/table/src/main/java/org/apache/ignite/internal/table/distributed/TableManager.java index e2afafa3b7..412c5df1c3 100644 --- a/modules/table/src/main/java/org/apache/ignite/internal/table/distributed/TableManager.java +++ b/modules/table/src/main/java/org/apache/ignite/internal/table/distributed/TableManager.java @@ -122,7 +122,7 @@ import org.apache.ignite.internal.raft.storage.impl.LogStorageFactoryCreator; import org.apache.ignite.internal.replicator.ReplicaManager; import org.apache.ignite.internal.replicator.ReplicaService; import org.apache.ignite.internal.replicator.TablePartitionId; -import org.apache.ignite.internal.schema.CatalogSchemaManager; +import org.apache.ignite.internal.schema.SchemaManager; import org.apache.ignite.internal.schema.configuration.GcConfiguration; import org.apache.ignite.internal.storage.DataStorageManager; import org.apache.ignite.internal.storage.MvPartitionStorage; @@ -278,7 +278,7 @@ public class TableManager implements IgniteTablesInternal, IgniteComponent { private final AtomicBoolean stopGuard = new AtomicBoolean(); /** Schema manager. */ - private final CatalogSchemaManager schemaManager; + private final SchemaManager schemaManager; private final LogStorageFactoryCreator volatileLogStorageFactoryCreator; @@ -386,7 +386,7 @@ public class TableManager implements IgniteTablesInternal, IgniteComponent { DataStorageManager dataStorageMgr, Path storagePath, MetaStorageManager metaStorageMgr, - CatalogSchemaManager schemaManager, + SchemaManager schemaManager, LogStorageFactoryCreator volatileLogStorageFactoryCreator, HybridClock clock, OutgoingSnapshotsManager outgoingSnapshotsManager, diff --git a/modules/table/src/main/java/org/apache/ignite/internal/table/distributed/schema/NonHistoricSchemas.java b/modules/table/src/main/java/org/apache/ignite/internal/table/distributed/schema/NonHistoricSchemas.java index 36a6b4aa3f..83558e0f74 100644 --- a/modules/table/src/main/java/org/apache/ignite/internal/table/distributed/schema/NonHistoricSchemas.java +++ b/modules/table/src/main/java/org/apache/ignite/internal/table/distributed/schema/NonHistoricSchemas.java @@ -25,11 +25,11 @@ import java.util.concurrent.CompletableFuture; import org.apache.ignite.internal.catalog.commands.DefaultValue; import org.apache.ignite.internal.catalog.descriptors.CatalogTableColumnDescriptor; import org.apache.ignite.internal.hlc.HybridTimestamp; -import org.apache.ignite.internal.schema.CatalogSchemaManager; import org.apache.ignite.internal.schema.Column; import org.apache.ignite.internal.schema.DefaultValueProvider; import org.apache.ignite.internal.schema.DefaultValueProvider.FunctionalValueProvider; import org.apache.ignite.internal.schema.SchemaDescriptor; +import org.apache.ignite.internal.schema.SchemaManager; import org.apache.ignite.internal.type.BitmaskNativeType; import org.apache.ignite.internal.type.DecimalNativeType; import org.apache.ignite.internal.type.NativeType; @@ -37,7 +37,7 @@ import org.apache.ignite.internal.type.TemporalNativeType; import org.apache.ignite.internal.type.VarlenNativeType; /** - * A dummy implementation over {@link CatalogSchemaManager}. It is dummy because: + * A dummy implementation over {@link SchemaManager}. It is dummy because: * * <ul> * <li>It imitates historicity, but always takes the latest known schema</li> @@ -50,9 +50,9 @@ import org.apache.ignite.internal.type.VarlenNativeType; */ // TODO: IGNITE-19447 - remove when switched to the CatalogService public class NonHistoricSchemas implements Schemas { - private final CatalogSchemaManager schemaManager; + private final SchemaManager schemaManager; - public NonHistoricSchemas(CatalogSchemaManager schemaManager) { + public NonHistoricSchemas(SchemaManager schemaManager) { this.schemaManager = schemaManager; } diff --git a/modules/table/src/test/java/org/apache/ignite/internal/table/distributed/TableManagerTest.java b/modules/table/src/test/java/org/apache/ignite/internal/table/distributed/TableManagerTest.java index 1e099f5cab..a14b00977a 100644 --- a/modules/table/src/test/java/org/apache/ignite/internal/table/distributed/TableManagerTest.java +++ b/modules/table/src/test/java/org/apache/ignite/internal/table/distributed/TableManagerTest.java @@ -86,8 +86,8 @@ import org.apache.ignite.internal.raft.client.TopologyAwareRaftGroupServiceFacto import org.apache.ignite.internal.raft.service.RaftGroupService; import org.apache.ignite.internal.raft.storage.impl.LocalLogStorageFactory; import org.apache.ignite.internal.replicator.ReplicaManager; -import org.apache.ignite.internal.schema.CatalogSchemaManager; import org.apache.ignite.internal.schema.SchemaDescriptor; +import org.apache.ignite.internal.schema.SchemaManager; import org.apache.ignite.internal.schema.SchemaUtils; import org.apache.ignite.internal.schema.configuration.GcConfiguration; import org.apache.ignite.internal.storage.DataStorageManager; @@ -199,7 +199,7 @@ public class TableManagerTest extends IgniteAbstractTest { private DataStorageManager dsm; - private CatalogSchemaManager sm; + private SchemaManager sm; private DistributionZoneManager distributionZoneManager; @@ -703,7 +703,7 @@ public class TableManagerTest extends IgniteAbstractTest { dsm = createDataStorageManager(configRegistry, workDir, storageEngineConfig), workDir, msm, - sm = new CatalogSchemaManager(revisionUpdater, catalogManager, msm), + sm = new SchemaManager(revisionUpdater, catalogManager, msm), budgetView -> new LocalLogStorageFactory(), clock, new OutgoingSnapshotsManager(clusterService.messagingService()),