You are viewing a plain text version of this content. The canonical link for it is here.

Posted to commits@pulsar.apache.org by "Technoboy- (via GitHub)" <gi...@apache.org> on 2023/03/08 05:01:28 UTC

[GitHub] [pulsar] Technoboy- opened a new pull request, #19746: [fix][broker] Fix potential exception cause the policy service init fail.

Technoboy- opened a new pull request, #19746:
URL: https://github.com/apache/pulsar/pull/19746

   ### Motivation
   
   We find that sometimes we can't get the topic policy all the time. After dumping the broker file, we find the namespace value in `policyCacheInitMap`  is false, but the reader was created successfully.
   
   
   ![image](https://user-images.githubusercontent.com/6297296/223623035-28473096-b37f-4f86-95a3-e513c8ff160d.png)
   
   ![image](https://user-images.githubusercontent.com/6297296/223623161-bf494a14-96f6-405f-acad-438649db80c5.png)
   
   I suspect that there may be some exceptions that occurs in the `refreshTopicPoliciesCache`.
   
   ### Modifications
   
   Move the `refreshTopicPoliciesCache` to the `thenAccept` block so we can catch any exceptions.
   
   ### Documentation
   
   <!-- DO NOT REMOVE THIS SECTION. CHECK THE PROPER BOX ONLY. -->
   
   - [ ] `doc` <!-- Your PR contains doc changes. -->
   - [ ] `doc-required` <!-- Your PR changes impact docs and you will update later -->
   - [x] `doc-not-needed` <!-- Your PR changes do not impact docs -->
   - [ ] `doc-complete` <!-- Docs have been already added -->
   
   


-- 
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: commits-unsubscribe@pulsar.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org

[GitHub] [pulsar] poorbarcode commented on pull request #19746: [fix][broker] Fix potential exception cause the policy service init fail.

Posted by "poorbarcode (via GitHub)" <gi...@apache.org>.

poorbarcode commented on PR #19746:
URL: https://github.com/apache/pulsar/pull/19746#issuecomment-1459888477

   /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 the specific comment.

To unsubscribe, e-mail: commits-unsubscribe@pulsar.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org

[GitHub] [pulsar] Technoboy- commented on pull request #19746: [fix][broker] Fix potential exception cause the policy service init fail.

Posted by "Technoboy- (via GitHub)" <gi...@apache.org>.

Technoboy- commented on PR #19746:
URL: https://github.com/apache/pulsar/pull/19746#issuecomment-1461357330

   /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 the specific comment.

To unsubscribe, e-mail: commits-unsubscribe@pulsar.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org

[GitHub] [pulsar] Technoboy- commented on pull request #19746: [fix][broker] Fix potential exception cause the policy service init fail.

Posted by "Technoboy- (via GitHub)" <gi...@apache.org>.

Technoboy- commented on PR #19746:
URL: https://github.com/apache/pulsar/pull/19746#issuecomment-1460053135

   /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 the specific comment.

To unsubscribe, e-mail: commits-unsubscribe@pulsar.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org

[GitHub] [pulsar] Technoboy- commented on pull request #19746: [fix][broker] Fix potential exception cause the policy service init fail.

Posted by "Technoboy- (via GitHub)" <gi...@apache.org>.

Technoboy- commented on PR #19746:
URL: https://github.com/apache/pulsar/pull/19746#issuecomment-1459606713

   /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 the specific comment.

To unsubscribe, e-mail: commits-unsubscribe@pulsar.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org

[GitHub] [pulsar] Technoboy- commented on pull request #19746: [fix][broker] Fix potential exception cause the policy service init fail.

Posted by "Technoboy- (via GitHub)" <gi...@apache.org>.

Technoboy- commented on PR #19746:
URL: https://github.com/apache/pulsar/pull/19746#issuecomment-1460240649

   /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 the specific comment.

To unsubscribe, e-mail: commits-unsubscribe@pulsar.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org

[GitHub] [pulsar] Technoboy- merged pull request #19746: [fix][broker] Fix potential exception cause the policy service init fail.

Posted by "Technoboy- (via GitHub)" <gi...@apache.org>.

Technoboy- merged PR #19746:
URL: https://github.com/apache/pulsar/pull/19746


-- 
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: commits-unsubscribe@pulsar.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org