Skip to content

Pipe: use session context in legacy receiver loaders#18184

Open
Caideyipi wants to merge 1 commit into
apache:dev/1.3from
Caideyipi:cp-7ccdaa5-dev-1.3
Open

Pipe: use session context in legacy receiver loaders#18184
Caideyipi wants to merge 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
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.

1 participant