astifter opened a new pull request #10164:
URL: https://github.com/apache/pulsar/pull/10164
Fixes #10161.
### Motivation
With issue #7132 the LoggerFactoryPtr was removed in favour of relevant
memory-safe(r) pointers.
This change forgot to also change the Log4Cxx implementation, fix this by
returning (as with
the other LoggerFactory's) a std::unique_ptr<LoggerFactory>.
### Modifications
Change remaining LoggerFactoryPtr in Log4Cxx factory to
std::unique_ptr<LoggerFactory> to conform to #7132.
### Verifying this change
- Visual inspection of changes to comply with the ones from #7132.
- pulsar-client-cpp builds now with USE_LOG4CXX.
--
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]
