rpuch commented on code in PR #4320:
URL: https://github.com/apache/ignite-3/pull/4320#discussion_r1741508387
##########
modules/table/src/testFixtures/java/org/apache/ignite/internal/table/TxAbstractTest.java:
##########
@@ -330,7 +330,12 @@ protected TxManager txManager(TableViewInternal t) {
protected boolean assertPartitionsSame(TableViewInternal table, int
partId) {
long storageIdx = 0;
- for (Map.Entry<String, Loza> entry :
txTestCluster.raftServers().entrySet()) {
+ Map<String, Loza> raftServers = txTestCluster.raftServers();
+ if (raftServers == null) {
Review Comment:
"localhost" was being resolved to more than 1 address, so more than one set
of managers per node was started, so start was failing due to a busy port. So
the start method (which is a `@BeforeEach`) was not able to create the
`raftService` map. `@AfterEach` method was run then, which is assuming that the
start method finished successfully, so an NPE was thrown masking the original
problem.
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: [email protected]
For queries about this service, please contact Infrastructure at:
[email protected]
