Skip to content

Pipe: use session context in legacy receiver loaders#18184

Merged
jt2594838 merged 1 commit into
apache:dev/1.3from
Caideyipi:cp-7ccdaa5-dev-1.3
Jul 13, 2026
Merged

Pipe: use session context in legacy receiver loaders#18184
jt2594838 merged 1 commit into
apache:dev/1.3from
Caideyipi:cp-7ccdaa5-dev-1.3

Conversation

@Caideyipi

Copy link
Copy Markdown
Collaborator

Cherry-pick of 7ccdaa5 (#18111) to dev/1.3.

Adaptations

  • Ported the session and authority checks to the dev/1.3 SessionInfo and AuthorityChecker APIs.
  • Used the dev/1.3 Deletion package and USE_PIPE grant syntax in the regression test.
  • Kept the dev/1.3 inline-message style because this branch does not contain the main-branch DataNode Pipe i18n source set.

Validation

  • mvn -Ddevelocity.off=true spotless:apply -pl iotdb-core/datanode,integration-test -P with-integration-tests
  • mvn -o -nsu -Ddevelocity.off=true test -pl iotdb-core/datanode -Dtest=IoTDBLegacyPipeReceiverAgentTest -DfailIfNoTests=false -DforkCount=0 (3 tests passed)
  • mvn -o -nsu -Ddevelocity.off=true test-compile -pl integration-test -P with-integration-tests -DskipTests
  • mvn -o -nsu -Ddevelocity.off=true verify -pl integration-test -P with-integration-tests,SimpleIT -DskipUTs -Dit.test=IoTDBLegacyPipeReceiverSecurityIT#testLegacyPipeDataDeleteUsesAuthenticatedUserPermission -DfailIfNoTests=false -Dfailsafe.failIfNoSpecifiedTests=false -DintegrationTest.forkCount=0 (1 test passed; local low-memory JVM used the repository Java 17 open-module options)
  • git diff origin/dev/1.3...HEAD --check

* Pipe: use session context in legacy receiver loaders

* Update DeletionLoader.java

* Fix legacy pipe receiver i18n message
@jt2594838
jt2594838 merged commit 2c089d3 into apache:dev/1.3 Jul 13, 2026
16 checks passed
@jt2594838
jt2594838 deleted the cp-7ccdaa5-dev-1.3 branch July 13, 2026 02:47
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