You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@pulsar.apache.org by "mattisonchao (via GitHub)" <gi...@apache.org> on 2024/12/17 07:45:30 UTC

[I] [enhancement] avoid log printing when tenant does not exist [pulsar]

mattisonchao opened a new issue, #23742:
URL: https://github.com/apache/pulsar/issues/23742

   ### Search before asking
   
   - [X] I searched in the [issues](https://github.com/apache/pulsar/issues) and found nothing similar.
   
   
   ### Motivation
   
   <img width="1050" alt="image" src="https://github.com/user-attachments/assets/fa3ac863-49c3-4611-aea5-d186c619042e" />
   
   
   we should throw metadata not found exception to avoid keeping that ugly stack traces.
   
   ### Solution
   
   _No response_
   
   ### Alternatives
   
   _No response_
   
   ### Anything else?
   
   _No response_
   
   ### Are you willing to submit a PR?
   
   - [ ] I'm willing to submit a PR!


-- 
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.apache.org

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

Re: [I] [enhancement] avoid log printing when tenant does not exist [pulsar]

Posted by "merlimat (via GitHub)" <gi...@apache.org>.
merlimat closed issue #23742: [enhancement] avoid log printing when tenant does not exist
URL: https://github.com/apache/pulsar/issues/23742


-- 
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

Re: [I] [enhancement] avoid log printing when tenant does not exist [pulsar]

Posted by "merlimat (via GitHub)" <gi...@apache.org>.
merlimat commented on issue #23742:
URL: https://github.com/apache/pulsar/issues/23742#issuecomment-6388725525

   Thanks @mattisonchao. This should be fixed by #26361 (4.2.5, 4.0.14, 5.0): a request for a tenant that doesn't exist now returns 404 without logging `Failed to get tenant` at ERROR with a stack trace, and real metadata store failures are still logged. Closing as completed. Please reopen if the stack trace you saw comes from a different code path.


-- 
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