The YamlLayout should probably separate events by "--" or whatever the page
separator token is in yaml (not a widely used feature).

On 28 April 2017 at 03:14, Mikael Ståldal <[email protected]> wrote:

> YamlLayout has this method:
>
> @Override
> public void toSerializable(final LogEvent event, final Writer writer)
> throws IOException {
>     if (complete && eventCount > 0) {
>         writer.append(", ");
>     }
>     super.toSerializable(event, writer);
> }
>
> As far as I can see, it shouldn't be there since you don't separate entries
> with comma in YAML. There is a similar method in JsonLayout, where it does
> make sense. I guess this is a copy-and-paste mistake when making YamlLayout
> with JsonLayout as template?
>
> (XmlLayout does not have any similar method.)
>
>
> --
> [image: MagineTV]
>
> *Mikael Ståldal*
> Senior software developer
>
> *Magine TV*
> [email protected]
> 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 <[email protected]>

Reply via email to