Skip to content

IGNITE-28607 Use Message DTO for DiscoveryDataBag#GridDiscoveryData#13157

Open
shishkovilja wants to merge 4 commits into
apache:masterfrom
shishkovilja:ignite-28607
Open

IGNITE-28607 Use Message DTO for DiscoveryDataBag#GridDiscoveryData#13157
shishkovilja wants to merge 4 commits into
apache:masterfrom
shishkovilja:ignite-28607

Conversation

@shishkovilja

Copy link
Copy Markdown
Contributor

No description provided.

@sonarqubecloud

Copy link
Copy Markdown

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

@github-actions

Copy link
Copy Markdown

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/spi/discovery/DataBagItem.java
  • modules/core/src/main/java/org/apache/ignite/spi/discovery/ObjectData.java
  • modules/zookeeper/src/main/java/org/apache/ignite/spi/discovery/zk/internal/ZkDiscoData.java
  • modules/core/src/main/java/org/apache/ignite/internal/processors/marshaller/MarshallerMappingsData.java
  • modules/core/src/main/java/org/apache/ignite/internal/processors/query/InlineSizesData.java
  • modules/core/src/main/java/org/apache/ignite/spi/discovery/tcp/internal/DiscoveryDataPacket.java

@shishkovilja shishkovilja force-pushed the ignite-28607 branch 2 times, most recently from 434b956 to 3d25fed Compare June 4, 2026 08:29
@ignitetcbot

Copy link
Copy Markdown
Contributor

TCBot Test Analysis

Possible Blockers (0)

No blockers found.

New Tests (0)

No new tests found.

@shishkovilja shishkovilja force-pushed the ignite-28607 branch 2 times, most recently from 0a84482 to f166d16 Compare June 11, 2026 07:24
@ignitetcbot

Copy link
Copy Markdown
Contributor

TCBot Test Analysis

Possible Blockers (0)

No blockers found.

New Tests (0)

No new tests found.

*/
private Boolean findLastFlag(Collection<Serializable> vals) {
Boolean flag = null;
private Boolean findLastUpdateNotifierFlag(Collection<ClusterNodeFlags> flags) {

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

flags naming is a bit confusing.
flags -> clusterNodeData
flag -> nodeFlags

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.

3 participants