Bklyn opened a new pull request #7798:
URL: https://github.com/apache/pulsar/pull/7798


   * Change throws of const char* to use std::exception types instead
   
   Fixes #7787
   
   ### Motivation
   
   Throw std::exception instead of `const char*`
   
   ### Modifications
   
   Should be self explanatory.
   
   ### Verifying this change
   
   This change should be covered by existing tests, such as BatchMessageTest.  
Not sure about the Auth test which catches `...` however.
   
   ### Does this pull request potentially affect one of the following parts:
   
   Yes
   
     - Operations which formerly threw `const char*` now throw a type derived 
from `std::exception`
   
   ### Documentation
   
   The methods that can throw should be documented as such.  This was not 
implemented.


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

For queries about this service, please contact Infrastructure at:
[email protected]


Reply via email to