lhotari opened a new pull request #17:
URL: https://github.com/apache/pulsar-adapters/pull/17


   Fixes #15, #16
   
   ### Motivation
   
   When Pulsar security is enabled, the current implementation is unusable. 
This PR also covers the feature request #16 to allow configuring most client 
and producer configuration properties.
   The issue #15 makes the Pulsar Log4j2 Appender unusable in many use cases. 
This PR fixes the issue by removing the hard coded producer name.
   The current build for Pulsar Log4j2 Appender is broken. This PR fixes the 
build by properly configuring Log4j2 plugin annotation processor for the maven 
build. There were broken tests which were unnecessary and testing Log4j2 
internals instead of the appender. Those tests have been removed
   
   ### Modifications
   
   - add support for configuring Pulsar client and producer configuration 
properties (#16)
   - fix maven build (Log4j2 plugin annotation processor)
   - remove broken tests which were unnecessary
   - remove the hard coded producer name which prevented using the appender in 
many use cases (#15)


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