You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@ignite.apache.org by am...@apache.org on 2023/08/03 22:38:38 UTC

[ignite-3] branch ignite-20086 updated (6cb5f3accf -> dc23ee8c5e)

This is an automated email from the ASF dual-hosted git repository.

amashenkov pushed a change to branch ignite-20086
in repository https://gitbox.apache.org/repos/asf/ignite-3.git


 discard 6cb5f3accf Fix tests.
 discard 79b087d6ee Implement ArchUnit rule to detect wrong test classes hierarchy.
     add 2ccbeca5f6 IGNITE-20089 Fix ConfigurationExtension static parameters injection (#2377)
     add 5d195d48d9 IGNITE-19837 Java client: Retry outdated schema error (#2381)
     add 64d0780acb IGNITE-20033 Jira links update in TODO (#2387)
     add 3a690ba4a9 IGNITE-19501 SchemaManager should use CatalogService for building BinaryRow descriptors (#2356)
     add 37fe3e86f1 IGNITE-19888 Java client: Track observable timestamp (#2371)
     add 3005f4cf31 IGNITE-20101 Relax CLI metric tests (#2379)
     add 85a18c7510 IGNITE-19506 Use data nodes from DistributionZoneManager with a causality token instead of BaselineManager#nodes (#2095)
     add 1507d0552c IGNITE-18875 Sql. Drop AbstractPlannerTest.TestTable (#2372)
     add 0ea4471e02 IGNITE-19697 Fix completions for --nodes in deploy command (#2360)
     add c2b00338ca IGNITE-19883 Pull request template with common code quality checks added
     add 4c81a9f421 IGNITE-19861 Introduce SQL metrics (#2343)
     add e64d45c5a9 IGNITE-20129 Fix CMake configuration (#2395)
     add 2655e406b0 IGNITE-20023 Eliminate NPEs in ItTxDistributedTestSingleNode (#2399)
     add 4bf70c0208 IGNITE-20080 Reduce the number of threads used by Raft in tests (#2374)
     add 516f2f9f10 IGNITE-20150 Expose JDBC port in Docker compose file (#2404)
     add f9954d7767 Implement ArchUnit rule to detect wrong test classes hierarchy.
     add dc23ee8c5e Fix tests.

This update added new revisions after undoing existing revisions.
That is to say, some revisions that were in the old version of the
branch are not in the new version.  This situation occurs
when a user --force pushes a change and generates a repository
containing something like this:

 * -- * -- B -- O -- O -- O   (6cb5f3accf)
            \
             N -- N -- N   refs/heads/ignite-20086 (dc23ee8c5e)

You should already have received notification emails for all of the O
revisions, and so the following emails describe only the N revisions
from the common base, B.

Any revisions marked "omit" are not gone; other references still
refer to them.  Any revisions marked "discard" are gone forever.

No new revisions were added by this update.

Summary of changes:
 .github/PULL_REQUEST_TEMPLATE.md                   |  19 +
 DEVNOTES.md                                        |   2 +-
 build.gradle                                       |  26 +-
 buildscripts/java-integration-test.gradle          |   2 +
 .../org/apache/ignite/lang/IgniteException.java    |   2 +-
 .../internal/catalog/commands/CatalogUtils.java    |   1 +
 .../descriptors/CatalogTableDescriptor.java        |  11 +
 .../catalog/events/AddColumnEventParameters.java   |  11 +-
 .../catalog/events/AlterColumnEventParameters.java |  14 +-
 .../catalog/events/CreateTableEventParameters.java |   4 +-
 .../catalog/events/DropColumnEventParameters.java  |  11 +-
 .../catalog/events/DropTableEventParameters.java   |  13 +-
 ...ntParameters.java => TableEventParameters.java} |  12 +-
 .../internal/catalog/storage/AlterColumnEntry.java |   1 +
 .../internal/catalog/storage/DropColumnsEntry.java |   1 +
 .../internal/catalog/storage/NewColumnsEntry.java  |   1 +
 .../internal/catalog/CatalogManagerSelfTest.java   |  79 ++
 .../cli/call/metric/ItEnabledMetricCallsTest.java  |   8 +-
 .../cli/call/metric/ItMetricCallsTest.java         |   6 +-
 .../metric/ItEnabledNodeMetricCommandTest.java     |   8 +-
 .../commands/metric/ItNodeMetricCommandTest.java   |  10 +-
 .../internal/rest/ItGeneratedRestClientTest.java   |  12 +-
 .../ignite/internal/cli/commands/Options.java      |   8 +
 .../cli/commands/cluster/unit/NodesAlias.java}     |  34 +-
 .../cluster/unit/UnitDeployOptionsMixin.java       |  44 +-
 .../completer/DynamicCompleterActivationPoint.java |   2 +
 .../completer/filter/DeployUnitsOptionsFilter.java |  75 ++
 .../completer/unit/UnitNodesCompleterFilter.java   |  73 ++
 .../unit/UnitNodesDynamicCompleterFactory.java     |   6 +-
 .../cli/core/repl/executor/ReplExecutor.java       |   4 +-
 .../ignite/internal/cli/util/ArrayUtils.java       |  27 +
 .../cli/commands/cluster/unit/NodesAliasTest.java} |  23 +-
 .../cli/commands/unit/DeployCommandTest.java       |  30 +-
 .../filter/DeployUnitsOptionsFilterTest.java       |  56 ++
 .../unit/UnitNodesCompleterFilterTest.java         |  62 ++
 .../ignite/internal/cli/util/ArrayUtilsTest.java   |  17 +
 .../client/proto/ClientBinaryTupleUtils.java       |  95 +++
 .../internal/client/proto/ClientMessagePacker.java |  50 ++
 modules/client-handler/build.gradle                |   2 +
 .../apache/ignite/client/handler/TestServer.java   |  22 +-
 .../ignite/client/handler/ClientHandlerModule.java |  18 +-
 .../handler/ClientInboundMessageHandler.java       |  59 +-
 .../requests/sql/ClientSqlExecuteRequest.java      |  16 +-
 .../handler/requests/table/ClientTableCommon.java  |  41 +-
 .../table/ClientTupleContainsKeyRequest.java       |   2 +-
 .../table/ClientTupleDeleteAllExactRequest.java    |   2 +-
 .../table/ClientTupleDeleteAllRequest.java         |   2 +-
 .../table/ClientTupleDeleteExactRequest.java       |   2 +-
 .../requests/table/ClientTupleDeleteRequest.java   |   2 +-
 .../requests/table/ClientTupleGetAllRequest.java   |   2 +-
 .../table/ClientTupleGetAndDeleteRequest.java      |   2 +-
 .../table/ClientTupleGetAndReplaceRequest.java     |   2 +-
 .../table/ClientTupleGetAndUpsertRequest.java      |   2 +-
 .../requests/table/ClientTupleGetRequest.java      |   2 +-
 .../table/ClientTupleInsertAllRequest.java         |   2 +-
 .../requests/table/ClientTupleInsertRequest.java   |   2 +-
 .../table/ClientTupleReplaceExactRequest.java      |   2 +-
 .../requests/table/ClientTupleReplaceRequest.java  |   2 +-
 .../table/ClientTupleUpsertAllRequest.java         |   2 +-
 .../requests/table/ClientTupleUpsertRequest.java   |   2 +-
 .../requests/tx/ClientTransactionBeginRequest.java |  47 +-
 .../ignite/internal/client/ClientChannel.java      |  15 +-
 .../ClientSchemaVersionMismatchException.java}     |  30 +-
 .../ignite/internal/client/ReliableChannel.java    |  23 +
 .../ignite/internal/client/TcpClientChannel.java   |  41 +-
 .../ignite/internal/client/sql/ClientSession.java  |   2 +
 .../ignite/internal/client/table/ClientTable.java  | 103 ++-
 .../client/table/ClientTupleSerializer.java        |  94 +--
 .../internal/client/tx/ClientTransactions.java     |   5 +-
 .../apache/ignite/client/ClientMetricsTest.java    |   2 +-
 .../client/ObservableTimestampPropagationTest.java |  97 +++
 .../apache/ignite/client/RequestBalancingTest.java |   2 +-
 .../ignite/client/TestClientHandlerModule.java     |  15 +-
 .../java/org/apache/ignite/client/TestServer.java  |  46 +-
 .../org/apache/ignite/client/fakes/FakeIgnite.java | 103 +--
 .../apache/ignite/client/fakes/FakeTxManager.java  | 191 +++++
 .../RepeatedFinishClientTransactionTest.java       |   7 +-
 .../testframework/ConfigurationExtensionTest.java  |  11 +
 .../testframework/ConfigurationExtension.java      |  25 +-
 .../ignite/internal/util/ExceptionUtils.java       |   2 +-
 .../ignite/lang/IgniteInternalException.java       |   2 +-
 .../distributionzones/DistributionZoneManager.java | 225 ++---
 .../distributionzones/DistributionZonesUtil.java   |  21 +-
 .../CausalityDataNodesEngine.java                  | 726 ++++++++++++++++
 .../rebalance/DistributionZoneRebalanceEngine.java |  56 +-
 .../BaseDistributionZoneManagerTest.java           |   7 +
 .../DistributionZoneManagerAlterFilterTest.java    |  28 +-
 ...ibutionZoneManagerConfigurationChangesTest.java |   7 +
 .../DistributionZoneManagerFilterTest.java         |  11 +-
 .../DistributionZoneManagerTest.java               |   1 +
 .../DistributionZoneMockTest.java                  |   1 +
 .../DistributionZoneCausalityDataNodesTest.java    | 927 +++++++++++++++------
 .../DistributionZoneRebalanceEngineTest.java       |   9 +-
 .../RebalanceUtilUpdateAssignmentsTest.java        |   1 +
 .../DistributionZonesTestUtil.java                 |  16 +-
 modules/platforms/build.gradle                     |   2 +-
 modules/platforms/cpp/CMakeLists.txt               |   7 +-
 .../cpp/ignite/client/detail/node_connection.cpp   |   3 +
 .../cpp/ignite/client/detail/sql/sql_impl.cpp      |  30 +-
 .../client/detail/transaction/transactions_impl.h  |   3 +
 .../platforms/cpp/ignite/odbc/query/data_query.cpp |   3 +
 .../platforms/cpp/ignite/odbc/sql_connection.cpp   |   3 +
 .../dotnet/Apache.Ignite.Tests/FakeServer.cs       |   1 +
 .../dotnet/Apache.Ignite.Tests/MetricsTests.cs     |   2 +-
 .../platforms/dotnet/Apache.Ignite/ErrorGroups.cs  |   1 -
 .../dotnet/Apache.Ignite/Internal/ClientSocket.cs  |   3 +
 .../Internal/Proto/MsgPack/MsgPackWriter.cs        |   5 +
 .../dotnet/Apache.Ignite/Internal/Sql/Sql.cs       |   3 +
 .../Internal/Transactions/Transactions.cs          |  11 +-
 .../ignite/internal/raft/RaftGroupServiceImpl.java |   2 +
 .../storage/ItRebalanceDistributedTest.java        |  11 +-
 .../zones/ItDistributionZonesFilterTest.java       |  22 +-
 ...niteDistributionZoneManagerNodeRestartTest.java |  73 +-
 .../runner/app/ItIgniteNodeRestartTest.java        |  20 +-
 .../ItThinClientSchemaSynchronizationTest.java     |  66 +-
 .../ignite/internal/sql/api/ItCommonApiTest.java   |  38 +-
 .../internal/sql/api/ItSqlAsynchronousApiTest.java |  31 +-
 .../internal/sql/api/ItSqlClientMetricsTest.java   | 142 ++++
 .../internal/sql/api/ItSqlSynchronousApiTest.java  |  27 +-
 .../sql/engine/ClusterPerClassIntegrationTest.java |  34 +
 .../internal/sql/engine/ItLimitOffsetTest.java     |   5 -
 .../streamer/ItAbstractDataStreamerTest.java       |  52 +-
 .../ignite/internal/table/ItTableScanTest.java     |   2 +-
 .../org/apache/ignite/internal/app/IgniteImpl.java |  17 +-
 modules/schema/build.gradle                        |   4 +
 .../internal/schema/CatalogDescriptorUtils.java    |   2 +
 ...chemaManager.java => CatalogSchemaManager.java} | 330 ++++----
 .../ignite/internal/schema/SchemaManager.java      |   2 +-
 .../apache/ignite/internal/schema/SchemaUtils.java |  12 +
 .../CatalogToSchemaDescriptorConverter.java}       | 105 ++-
 .../schema/registry/UpgradingRowAdapter.java       |   8 +
 .../AbstractSchemaConverterTest.java               |  10 +-
 .../internal/schema/CatalogSchemaManagerTest.java  | 509 +++++++++++
 .../CatalogToSchemaDescriptorConverterTest.java    | 199 +++++
 ...nfigurationToSchemaDescriptorConverterTest.java |   1 +
 .../testutils/SchemaConfigurationConverter.java    |   3 +
 modules/sql-engine/build.gradle                    |   1 +
 .../internal/sql/engine/SqlQueryProcessor.java     |  32 +-
 .../sql/engine/prepare/PrepareServiceImpl.java     |  38 +-
 .../internal/sql/engine/sql/ParserServiceImpl.java |   4 +-
 .../sql/engine/util/CaffeineCacheFactory.java      |  66 --
 .../sql/engine/util/{ => cache}/Cache.java         |  13 +-
 .../sql/engine/util/{ => cache}/CacheFactory.java  |  13 +-
 .../engine/util/cache/CaffeineCacheFactory.java    | 120 +++
 .../sql/engine/util/cache/StatsCounter.java}       |  23 +-
 .../sql/metrics/SqlClientMetricSource.java}        |  40 +-
 .../sql/metrics/SqlPlanCacheMetricSource.java      |  74 ++
 .../internal/sql/engine/StopCalciteModuleTest.java |   6 +-
 .../exec/ExecutionDependencyResolverSelfTest.java  | 129 ++-
 .../sql/engine/exec/ExecutionServiceImplTest.java  |   4 +-
 .../sql/engine/exec/MockedStructuresTest.java      |  18 +-
 .../DdlCommandHandlerExceptionHandlingTest.java    |   1 +
 .../sql/engine/framework/TestBuilders.java         |  14 +-
 .../internal/sql/engine/framework/TestIndex.java   |   1 +
 .../internal/sql/engine/framework/TestNode.java    |   4 +-
 .../sql/engine/framework/TestStatistic.java        |  78 ++
 .../internal/sql/engine/framework/TestTable.java   |  52 +-
 .../planner/AbstractAggregatePlannerTest.java      |  91 +-
 .../sql/engine/planner/AbstractPlannerTest.java    | 359 ++------
 .../sql/engine/planner/AggregatePlannerTest.java   |  12 +-
 .../planner/ColocatedSortAggregatePlannerTest.java |  12 +-
 .../CorrelatedNestedLoopJoinPlannerTest.java       |  91 +-
 .../planner/CorrelatedSubqueryPlannerTest.java     |  35 +-
 .../sql/engine/planner/DmlPlannerTest.java         |   5 +-
 .../sql/engine/planner/HashIndexPlannerTest.java   |  97 +--
 .../engine/planner/HashIndexSpoolPlannerTest.java  | 121 +--
 .../sql/engine/planner/ImplicitCastsTest.java      |  54 +-
 .../planner/IndexSearchBoundsPlannerTest.java      | 118 ++-
 .../engine/planner/JoinColocationPlannerTest.java  | 107 ++-
 .../sql/engine/planner/JoinCommutePlannerTest.java |  36 +-
 .../engine/planner/JoinWithUsingPlannerTest.java   |  46 +-
 .../sql/engine/planner/LimitOffsetPlannerTest.java |  42 +-
 .../planner/MapReduceSortAggregatePlannerTest.java |  12 +-
 .../sql/engine/planner/MergeJoinPlannerTest.java   | 754 +++++++----------
 .../internal/sql/engine/planner/PlannerTest.java   | 757 +++++------------
 .../sql/engine/planner/PlannerTimeoutTest.java     |  57 +-
 .../planner/ProjectFilterScanMergePlannerTest.java |  42 +-
 .../sql/engine/planner/SetOpPlannerTest.java       |  51 +-
 .../planner/SortedIndexSpoolPlannerTest.java       | 100 +--
 .../engine/planner/TableFunctionPlannerTest.java   |  33 +-
 .../sql/engine/planner/TableSpoolPlannerTest.java  |  39 +-
 .../sql/engine/planner/UnionPlannerTest.java       |  54 +-
 .../sql/engine/prepare/TypeCoercionTest.java       | 164 +++-
 .../engine/schema/CatalogSqlSchemaManagerTest.java |  10 +-
 .../sql/engine/sql/ParserServiceImplTest.java      |  18 +-
 .../sql/engine/util/EmptyCacheFactory.java         |  15 +
 .../sql/metrics/PlanningCacheMetricsTest.java      |  96 +++
 .../distributed/ItTxDistributedTestSingleNode.java |  46 +-
 .../internal/table/distributed/TableManager.java   | 162 ++--
 .../replicator/PartitionReplicaListener.java       |   6 +-
 .../table/distributed/TableManagerTest.java        |   3 +
 .../org/apache/ignite/internal/tx/TxManager.java   |   6 +-
 .../internal/tx/impl/IgniteTransactionsImpl.java   |  31 +-
 .../internal/tx/impl/ReadWriteTransactionImpl.java |   2 +-
 .../ignite/internal/tx/impl/TxManagerImpl.java     |   6 +-
 packaging/docker/docker-compose.yml                |   6 +-
 196 files changed, 6512 insertions(+), 3668 deletions(-)
 create mode 100644 .github/PULL_REQUEST_TEMPLATE.md
 copy modules/catalog/src/main/java/org/apache/ignite/internal/catalog/events/{DropTableEventParameters.java => TableEventParameters.java} (78%)
 copy modules/{metrics/src/main/java/org/apache/ignite/internal/metrics/Metric.java => cli/src/main/java/org/apache/ignite/internal/cli/commands/cluster/unit/NodesAlias.java} (64%)
 create mode 100644 modules/cli/src/main/java/org/apache/ignite/internal/cli/core/repl/completer/filter/DeployUnitsOptionsFilter.java
 create mode 100644 modules/cli/src/main/java/org/apache/ignite/internal/cli/core/repl/completer/unit/UnitNodesCompleterFilter.java
 copy modules/{metastorage-api/src/test/java/org/apache/ignite/internal/metastorage/dsl/OperationTypeTest.java => cli/src/test/java/org/apache/ignite/internal/cli/commands/cluster/unit/NodesAliasTest.java} (63%)
 create mode 100644 modules/cli/src/test/java/org/apache/ignite/internal/cli/core/repl/completer/filter/DeployUnitsOptionsFilterTest.java
 create mode 100644 modules/cli/src/test/java/org/apache/ignite/internal/cli/core/repl/completer/unit/UnitNodesCompleterFilterTest.java
 copy modules/{schema/src/main/java/org/apache/ignite/internal/schema/SchemaVersionMismatchException.java => client/src/main/java/org/apache/ignite/internal/client/ClientSchemaVersionMismatchException.java} (64%)
 create mode 100644 modules/client/src/test/java/org/apache/ignite/client/ObservableTimestampPropagationTest.java
 create mode 100644 modules/client/src/test/java/org/apache/ignite/client/fakes/FakeTxManager.java
 create mode 100644 modules/distribution-zones/src/main/java/org/apache/ignite/internal/distributionzones/causalitydatanodes/CausalityDataNodesEngine.java
 rename modules/distribution-zones/src/test/java/org/apache/ignite/internal/distributionzones/{ => causalitydatanodes}/DistributionZoneCausalityDataNodesTest.java (51%)
 create mode 100644 modules/runner/src/integrationTest/java/org/apache/ignite/internal/sql/api/ItSqlClientMetricsTest.java
 copy modules/schema/src/main/java/org/apache/ignite/internal/schema/{SchemaManager.java => CatalogSchemaManager.java} (56%)
 copy modules/schema/src/main/java/org/apache/ignite/internal/schema/{configuration/ConfigurationToSchemaDescriptorConverter.java => catalog/CatalogToSchemaDescriptorConverter.java} (51%)
 rename modules/schema/src/test/java/org/apache/ignite/internal/schema/{configuration => }/AbstractSchemaConverterTest.java (95%)
 create mode 100644 modules/schema/src/test/java/org/apache/ignite/internal/schema/CatalogSchemaManagerTest.java
 create mode 100644 modules/schema/src/test/java/org/apache/ignite/internal/schema/catalog/CatalogToSchemaDescriptorConverterTest.java
 delete mode 100644 modules/sql-engine/src/main/java/org/apache/ignite/internal/sql/engine/util/CaffeineCacheFactory.java
 rename modules/sql-engine/src/main/java/org/apache/ignite/internal/sql/engine/util/{ => cache}/Cache.java (73%)
 rename modules/sql-engine/src/main/java/org/apache/ignite/internal/sql/engine/util/{ => cache}/CacheFactory.java (70%)
 create mode 100644 modules/sql-engine/src/main/java/org/apache/ignite/internal/sql/engine/util/cache/CaffeineCacheFactory.java
 copy modules/{api/src/main/java/org/apache/ignite/network/TopologyEventHandler.java => sql-engine/src/main/java/org/apache/ignite/internal/sql/engine/util/cache/StatsCounter.java} (58%)
 copy modules/{metrics/src/integrationTest/java/org/apache/ignite/internal/metrics/exporters/TestMetricsSource.java => sql-engine/src/main/java/org/apache/ignite/internal/sql/metrics/SqlClientMetricSource.java} (55%)
 create mode 100644 modules/sql-engine/src/main/java/org/apache/ignite/internal/sql/metrics/SqlPlanCacheMetricSource.java
 create mode 100644 modules/sql-engine/src/test/java/org/apache/ignite/internal/sql/engine/framework/TestStatistic.java
 create mode 100644 modules/sql-engine/src/test/java/org/apache/ignite/internal/sql/metrics/PlanningCacheMetricsTest.java