Skip to content

[cherry-pick][branch-2.10] Allow superusers to abort transactions (#19467)#19473

Merged
nicoloboschi merged 3 commits intobranch-2.10from
xiangying/cherry-pick/TC_auth
Feb 9, 2023
Merged

[cherry-pick][branch-2.10] Allow superusers to abort transactions (#19467)#19473
nicoloboschi merged 3 commits intobranch-2.10from
xiangying/cherry-pick/TC_auth

Conversation

@liangyepianzhou
Copy link
Copy Markdown
Contributor

Motivation

Cherry-pick #19467

Verifying this change

  • Make sure that the change passes the CI checks.

(Please pick either of the following options)

This change is a trivial rework / code cleanup without any test coverage.

(or)

This change is already covered by existing tests, such as (please describe tests).

(or)

This change added tests and can be verified as follows:

(example:)

  • Added integration tests for end-to-end deployment with large payloads (10MB)
  • Extended integration test for recovery after broker failure

Does this pull request potentially affect one of the following parts:

If the box was checked, please highlight the changes

  • Dependencies (add or upgrade a dependency)
  • The public API
  • The schema
  • The default values of configurations
  • The threading model
  • The binary protocol
  • The REST endpoints
  • The admin CLI options
  • The metrics
  • Anything that affects deployment

Documentation

  • doc
  • doc-required
  • doc-not-needed
  • doc-complete

Super users must be always allowed to abort a transaction even if they're not the original owner.

* Check that only owner or superusers are allowed to perform txn operations (end, add partition and add subscription)

(cherry picked from commit 459a7a5)
@liangyepianzhou liangyepianzhou changed the title [improve][txn] Allow superusers to abort transactions (#19467) [cherry-pick][branch-2.10] Allow superusers to abort transactions (#19467) Feb 9, 2023
@github-actions github-actions Bot added the doc-not-needed Your PR changes do not impact docs label Feb 9, 2023
Comment thread pulsar-broker/src/main/java/org/apache/pulsar/broker/PulsarService.java Outdated
@nicoloboschi nicoloboschi merged commit cb91c4a into branch-2.10 Feb 9, 2023
@codelipenghui codelipenghui deleted the xiangying/cherry-pick/TC_auth branch February 10, 2023 01:07
nicoloboschi added a commit to datastax/pulsar that referenced this pull request Feb 14, 2023
…ache#19467) (apache#19473)

Co-authored-by: Nicolò Boschi <boschi1997@gmail.com>
(cherry picked from commit cb91c4a)
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

doc-not-needed Your PR changes do not impact docs

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants