Skip to content

IGNITE-28515 : Use MessageSerializer for QueryIndex - copy#13058

Merged
anton-vinogradov merged 14 commits intoapache:masterfrom
Vladsz83:IGNITE-28515-QueryIndex-copy
Apr 23, 2026
Merged

IGNITE-28515 : Use MessageSerializer for QueryIndex - copy#13058
anton-vinogradov merged 14 commits intoapache:masterfrom
Vladsz83:IGNITE-28515-QueryIndex-copy

Conversation

@Vladsz83
Copy link
Copy Markdown
Contributor

No description provided.

Vladsz83 added 11 commits April 15, 2026 15:52
# Conflicts:
#	modules/core/src/main/java/org/apache/ignite/internal/processors/query/schema/operation/SchemaIndexCreateOperation.java
# Conflicts:
#	modules/core/src/main/java/org/apache/ignite/internal/CoreMessagesProvider.java
break;

case 10910: {
case 10911: {
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.

@Vladsz83 @anton-vinogradov , now it looks very ugly that we need to do this. It is necessary to refactor this case block.

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

We should use smth. like msgFactory.messageType(msg). Looks like other ticket. Or I could place the message below in the registration list. But I could place it where it should be and make a fix.

@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/internal/cache/query/QueryIndexMessage.java
  • modules/core/src/main/java/org/apache/ignite/internal/processors/query/schema/operation/SchemaIndexCreateOperation.java
  • modules/core/src/test/java/org/apache/ignite/internal/codegen/MessageProcessorTest.java

@sonarqubecloud
Copy link
Copy Markdown

@anton-vinogradov anton-vinogradov merged commit 938452e into apache:master Apr 23, 2026
8 of 10 checks passed
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