Skip to content

IGNITE-28050 Use MessageSerializer for continuous routine start messages#12952

Merged
shishkovilja merged 1 commit intoapache:masterfrom
shishkovilja:ignite-28050
Apr 7, 2026
Merged

IGNITE-28050 Use MessageSerializer for continuous routine start messages#12952
shishkovilja merged 1 commit intoapache:masterfrom
shishkovilja:ignite-28050

Conversation

@shishkovilja
Copy link
Copy Markdown
Contributor

@shishkovilja shishkovilja commented Mar 27, 2026

  1. StartRoutineDiscoveryMessageV2 was renamed to StartRoutineDiscoveryImmutableMessage
  2. StartRequestDataV2 was removed. Both StartRoutineDiscoveryMessage and StartRoutineDiscoveryImmutableMessage use the StartRequestData.
  3. CachePartitionPartialCountersMap#toCountersMap was refactored, because initial counters are not processed in continuous query routines.
  4. IncompleteDeserializationException removed - it was necessary only for CQ, but now it is redundant.

@shishkovilja shishkovilja force-pushed the ignite-28050 branch 3 times, most recently from 02f03c8 to 5d8f4f8 Compare April 1, 2026 08:07
@github-actions
Copy link
Copy Markdown

github-actions bot commented Apr 6, 2026

Possible compatibility issues. Please, check rolling upgrade cases

This PR modifies protected classes (with Order annotation).
Changes to these classes can break rolling upgrade compatibility.

Affected files:

  • modules/core/src/main/java/org/apache/ignite/internal/processors/cache/distributed/dht/preloader/CachePartitionPartialCountersMap.java
  • modules/core/src/main/java/org/apache/ignite/internal/processors/continuous/AbstractContinuousMessage.java
  • modules/core/src/main/java/org/apache/ignite/spi/discovery/tcp/messages/TcpDiscoveryCustomEventMessage.java

@sonarqubecloud
Copy link
Copy Markdown

sonarqubecloud bot commented Apr 6, 2026

Quality Gate Failed Quality Gate failed

Failed conditions
15 New Code Smells (required ≤ 1)

See analysis details on SonarQube Cloud

Catch issues before they fail your Quality Gate with our IDE extension SonarQube for IDE

@shishkovilja shishkovilja merged commit 9859f64 into apache:master Apr 7, 2026
8 of 10 checks passed
@shishkovilja shishkovilja deleted the ignite-28050 branch April 7, 2026 07:23
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants