Skip to content

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

Open
shishkovilja wants to merge 2 commits intoapache:masterfrom
shishkovilja:ignite-28050
Open

IGNITE-28050 Use MessageSerializer for continuous routine start messages#12952
shishkovilja wants to merge 2 commits 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.

@shishkovilja shishkovilja force-pushed the ignite-28050 branch 2 times, most recently from 263febb to 02f03c8 Compare March 31, 2026 13:26
# Conflicts:
#	modules/core/src/main/java/org/apache/ignite/internal/managers/discovery/DiscoveryMessageFactory.java
@github-actions
Copy link
Copy Markdown

github-actions bot commented Apr 1, 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/continuous/StartRoutineDiscoveryImmutableMessage.java
  • 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

@sonarqubecloud
Copy link
Copy Markdown

sonarqubecloud bot commented Apr 1, 2026

Quality Gate Failed Quality Gate failed

Failed conditions
16 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

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.

1 participant