Skip to content

Pipe: disable serialize-by-region by default#18139

Merged
jt2594838 merged 1 commit into
apache:masterfrom
Caideyipi:pipe-serialize-by-region-default-off
Jul 16, 2026
Merged

Pipe: disable serialize-by-region by default#18139
jt2594838 merged 1 commit into
apache:masterfrom
Caideyipi:pipe-serialize-by-region-default-off

Conversation

@Caideyipi

Copy link
Copy Markdown
Collaborator

Description

This PR changes the default value of connector.serialize-by-region / sink.serialize-by-region to false, so the serialize-by-region sink behavior added in #17946 is opt-in by default.

It also updates related unit expectations so the default data-region sink remains the async IoTDB thrift sink, while explicitly enabling serialize-by-region still selects the sync sink and uses region-prefixed subtask attributes.

Tests

  • git diff --check
  • git diff --cached --check
  • mvn spotless:apply -pl iotdb-core/node-commons,iotdb-core/datanode -DskipTests
  • Attempted mvn -pl iotdb-core/datanode -Dtest=PipeDataNodePluginAgentTest,PipeSinkSubtaskManagerTest test -Ddevelocity.off=true, but it is blocked before running tests by existing datanode compile errors in generated/freemarker and queryengine/i18n sources, including missing IFill / IFillFilter, missing ComparatorChain, missing Accumulator, and missing QueryMessages constants.

@jt2594838 jt2594838 merged commit 8a1c7aa into apache:master Jul 16, 2026
41 checks passed
@jt2594838 jt2594838 deleted the pipe-serialize-by-region-default-off branch July 16, 2026 03:05
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants