I think so. That'd be less confusing. On 30 March 2017 at 02:43, Mikael Ståldal <mikael.stal...@magine.com> wrote:
> Would it make sense to have shortcuts for Property and KeyValuePair, since > they are generic and used by several appenders/layouts? > > On Wed, Mar 29, 2017 at 7:12 PM, Matt Sicker <boa...@gmail.com> wrote: > > > Oh, I see. That's an interesting API... > > > > On 29 March 2017 at 10:49, Marvin Geitner <mgeit...@novomind.com> wrote: > > > > > Thank you very much, > > > > > > kafkaBuilder.addComponent(builder.newComponent("bootstrap.servers", > > > "Property", "kafka.host:*9092*")); > > > > > > is working. > > > > > > BR > > > Marvin > > > > > > -----Ursprüngliche Nachricht----- > > > Von: Ralph Goers [mailto:ralph.go...@dslextreme.com] > > > Gesendet: Mittwoch, 29. März 2017 17:46 > > > An: Log4J Users List <log4j-user@logging.apache.org> > > > Betreff: Re: Kafka Appender with Programmatic Configuration > > > > > > Mikael’s solution is correct. Every item in the ConfigurationBuilder > is a > > > Component. We have “special” Components for things that are always > used, > > > such as Loggers and Appenders. But we can’t know about every type of > > > Component so in those cases you just use the generic Component. > > > > > > Ralph > > > > > > > On Mar 29, 2017, at 8:26 AM, Matt Sicker <boa...@gmail.com> wrote: > > > > > > > > Looks like a bug to me! Here's the issue: a Properties component is > > > > injected into a kafka appender, but there is no way to create a > > > > Property or Properties component directly via the configuration > > > > builder API. The top level properties aren't injected in the kafka > > > > appender (and manager), so they're unused. > > > > > > > > Interestingly enough, I don't see a way to inject KeyValuePair > > > > components either (which is a similar plugin class in log4j-core). > > > > > > > > Could you make an issue here: > > > > https://issues.apache.org/jira/browse/LOG4J2 > > > > > > > > On 29 March 2017 at 09:27, Marvin Geitner <mgeit...@novomind.com> > > wrote: > > > > > > > >> Hi all, > > > >> > > > >> I'm trying to configure the Kafka Appender with the programmatic > > > >> configuration described in the manuals. So I created the > > > >> ConfigurationFactory and set up the AppenderComponentBuilder. > > > >> > > > >> If I execute the Application I'll get following exception: > > > >> > > > >> Caused by: org.apache.kafka.common.config.ConfigException: Missing > > > >> required configuration "bootstrap.servers" which has no default > value. > > > >> > > > >> Obviously the property "bootstrap.servers" hasn't been set. > > > >> > > > >> A builder.addProperty("bootstrap.servers", "kafka.host"); doesn't > > work. > > > >> In XML the configuration looks like <Property > > > >> name="bootstrap.servers"> kafka.host</Property>. > > > >> > > > >> But how do I set this in java? > > > >> > > > >> The code looks like the following: > > > >> > > > >> static Configuration createConfiguration(final String name, > > > >> ConfigurationBuilder<BuiltConfiguration> builder) { > > > >> > > > >> builder.setConfigurationName(name); > > > >> builder.setStatusLevel(Level.INFO); > > > >> > > > >> AppenderComponentBuilder kafkaBuilder = > > > >> builder.newAppender("KafkaLogger ", "Kafka") > > > >> .addAttribute("topic", "testTopic"); > > > >> kafkaBuilder.add(builder.newLayout("PatternLayout") > > > >> .addAttribute("pattern", "%d{HH:mm:ss:SSS} | > > > >> %logger{20} | %msg%n%ex{5}")); > > > >> > > > >> builder.add(kafkaBuilder); > > > >> > > > >> builder.add(builder.newRootLogger(Level.INFO).add( > > > >> builder.newAppenderRef("KafkaLogger"))); > > > >> > > > >> return builder.build(); > > > >> } > > > >> > > > >> Hope someone can help me. > > > >> > > > >> Thanks and BR, > > > >> Marvin > > > >> > > > > > > > > > > > > > > > > -- > > > > Matt Sicker <boa...@gmail.com> > > > > > > > > > > > > --------------------------------------------------------------------- > > > To unsubscribe, e-mail: log4j-user-unsubscr...@logging.apache.org > > > For additional commands, e-mail: log4j-user-h...@logging.apache.org > > > > > > > > > > > > -- > > Matt Sicker <boa...@gmail.com> > > > > > > -- > [image: MagineTV] > > *Mikael Ståldal* > Senior software developer > > *Magine TV* > mikael.stal...@magine.com > Grev Turegatan 3 | 114 46 Stockholm, Sweden | www.magine.com > > Privileged and/or Confidential Information may be contained in this > message. If you are not the addressee indicated in this message > (or responsible for delivery of the message to such a person), you may not > copy or deliver this message to anyone. In such case, > you should destroy this message and kindly notify the sender by reply > email. > -- Matt Sicker <boa...@gmail.com>