Re: [PR] [cleanup] [test] remove useless TestAuthorizationProvider2 [pulsar]

2024-04-26 Thread via GitHub
lhotari merged PR #22595: URL: https://github.com/apache/pulsar/pull/22595 -- 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 the specific comment. To unsubscribe, e-mail:

Re: [PR] [cleanup] [test] remove useless TestAuthorizationProvider2 [pulsar]

2024-04-26 Thread via GitHub
thetumbled commented on PR #22595: URL: https://github.com/apache/pulsar/pull/22595#issuecomment-2079048232 /pulsarbot rerun-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 the

Re: [PR] [cleanup] [test] remove useless TestAuthorizationProvider2 [pulsar]

2024-04-26 Thread via GitHub
thetumbled commented on code in PR #22595: URL: https://github.com/apache/pulsar/pull/22595#discussion_r1580519788 ## pulsar-broker/src/test/java/org/apache/pulsar/client/api/AuthorizationProducerConsumerTest.java: ## @@ -349,7 +349,8 @@ public void testSubscriberPermission()

Re: [PR] [cleanup] [test] remove useless TestAuthorizationProvider2 [pulsar]

2024-04-26 Thread via GitHub
thetumbled commented on code in PR #22595: URL: https://github.com/apache/pulsar/pull/22595#discussion_r1580519788 ## pulsar-broker/src/test/java/org/apache/pulsar/client/api/AuthorizationProducerConsumerTest.java: ## @@ -349,7 +349,8 @@ public void testSubscriberPermission()

Re: [PR] [cleanup] [test] remove useless TestAuthorizationProvider2 [pulsar]

2024-04-26 Thread via GitHub
thetumbled commented on code in PR #22595: URL: https://github.com/apache/pulsar/pull/22595#discussion_r1580519788 ## pulsar-broker/src/test/java/org/apache/pulsar/client/api/AuthorizationProducerConsumerTest.java: ## @@ -349,7 +349,8 @@ public void testSubscriberPermission()

Re: [PR] [cleanup] [test] remove useless TestAuthorizationProvider2 [pulsar]

2024-04-25 Thread via GitHub
lhotari commented on code in PR #22595: URL: https://github.com/apache/pulsar/pull/22595#discussion_r1580497818 ## pulsar-broker/src/test/java/org/apache/pulsar/client/api/AuthorizationProducerConsumerTest.java: ## @@ -349,7 +349,8 @@ public void testSubscriberPermission()

Re: [PR] [cleanup] [test] remove useless TestAuthorizationProvider2 [pulsar]

2024-04-25 Thread via GitHub
thetumbled commented on PR #22595: URL: https://github.com/apache/pulsar/pull/22595#issuecomment-2078588760 PTAL, thanks. @Technoboy- @lhotari -- 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