[GitHub] [pulsar] yuruguo commented on pull request #13157: [Authorization] Converge authz to operate subscription conf of namespace

2021-12-06 Thread GitBox
yuruguo commented on pull request #13157: URL: https://github.com/apache/pulsar/pull/13157#issuecomment-987634230 @eolivelli @codelipenghui @hangc0276 @gaoran10 Would you help review this PR? thx! -- This is an automated message from the Apache Git Service. To respond to the message, ple

[GitHub] [pulsar] yuruguo commented on pull request #13157: [Authorization] Converge authz to operate subscription conf of namespace

2021-12-06 Thread GitBox
yuruguo commented on pull request #13157: URL: https://github.com/apache/pulsar/pull/13157#issuecomment-987624154 /pulsarbot run-failure-checks -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to th

[GitHub] [pulsar] yuruguo commented on pull request #13157: [Authorization] Converge authz to operate subscription conf of namespace

2021-12-06 Thread GitBox
yuruguo commented on pull request #13157: URL: https://github.com/apache/pulsar/pull/13157#issuecomment-987606683 /pulsarbot run-failure-checks -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to th

[GitHub] [pulsar] yuruguo commented on pull request #13157: [Authorization] Converge authz to operate subscription conf of namespace

2021-12-06 Thread GitBox
yuruguo commented on pull request #13157: URL: https://github.com/apache/pulsar/pull/13157#issuecomment-987592604 /pulsarbot run-failure-checks -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to th

[GitHub] [pulsar] yuruguo commented on pull request #13157: [Authorization] Converge authz to operate subscription conf of namespace

2021-12-06 Thread GitBox
yuruguo commented on pull request #13157: URL: https://github.com/apache/pulsar/pull/13157#issuecomment-987582464 /pulsarbot run-failure-checks -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to th

[GitHub] [pulsar] yuruguo commented on pull request #13157: [Authorization] Converge authz to operate subscription conf of namespace

2021-12-06 Thread GitBox
yuruguo commented on pull request #13157: URL: https://github.com/apache/pulsar/pull/13157#issuecomment-987570232 /pulsarbot run-failure-checks -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to th

[GitHub] [pulsar] yuruguo commented on pull request #13157: [Authorization] Converge authz to operate subscription conf of namespace

2021-12-06 Thread GitBox
yuruguo commented on pull request #13157: URL: https://github.com/apache/pulsar/pull/13157#issuecomment-987569933 /pulsarbot run-failure-checks -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to th

[GitHub] [pulsar] yuruguo commented on pull request #13157: [Authorization] Converge authz to operate subscription conf of namespace

2021-12-06 Thread GitBox
yuruguo commented on pull request #13157: URL: https://github.com/apache/pulsar/pull/13157#issuecomment-987560978 /pulsarbot run-failure-checks -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to th

[GitHub] [pulsar] yuruguo commented on pull request #13157: [Authorization] Converge authz to operate subscription conf of namespace

2021-12-06 Thread GitBox
yuruguo commented on pull request #13157: URL: https://github.com/apache/pulsar/pull/13157#issuecomment-987557386 /pulsarbot run-failure-checks -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to th

[GitHub] [pulsar] yuruguo commented on pull request #13157: [Authorization] Converge authz to operate subscription conf of namespace

2021-12-06 Thread GitBox
yuruguo commented on pull request #13157: URL: https://github.com/apache/pulsar/pull/13157#issuecomment-987549247 /pulsarbot run-failure-checks -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to th

[GitHub] [pulsar] yuruguo commented on pull request #13157: [Authorization] Converge authz to operate subscription conf of namespace

2021-12-06 Thread GitBox
yuruguo commented on pull request #13157: URL: https://github.com/apache/pulsar/pull/13157#issuecomment-987548978 /pulsarbot run-failure-checks -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to th

[GitHub] [pulsar] yuruguo commented on pull request #13157: [Authorization] Converge authz to operate subscription conf of namespace

2021-12-06 Thread GitBox
yuruguo commented on pull request #13157: URL: https://github.com/apache/pulsar/pull/13157#issuecomment-987531379 > If all consume roles could modify the namespace level subscription-related policies, do it may cause some confusion? In fact, this problem still exists if the tenant ha

[GitHub] [pulsar] yuruguo commented on pull request #13157: [Authorization] Converge authz to operate subscription conf of namespace

2021-12-06 Thread GitBox
yuruguo commented on pull request #13157: URL: https://github.com/apache/pulsar/pull/13157#issuecomment-987502487 /pulsarbot run-failure-checks -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to th