Yes, I will remove it.
On Fri, Apr 28, 2017 at 4:19 PM, Matt Sicker wrote:
> Then that toSerializable() override is most likely a copy/paste error from
> JsonLayout.
>
> On 28 April 2017 at 09:15, Mikael Ståldal
> wrote:
>
> > YamlLayout always prefix each event with a line with "---". JsonLayo
Then that toSerializable() override is most likely a copy/paste error from
JsonLayout.
On 28 April 2017 at 09:15, Mikael Ståldal wrote:
> YamlLayout always prefix each event with a line with "---". JsonLayout is a
> bit different since you don't want a comma before the first event.
>
> On Fri, A
YamlLayout always prefix each event with a line with "---". JsonLayout is a
bit different since you don't want a comma before the first event.
On Fri, Apr 28, 2017 at 4:02 PM, Matt Sicker wrote:
> The YamlLayout should probably separate events by "--" or whatever the page
> separator token is in
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 wrote:
> YamlLayout has this method:
>
> @Override
> public void toSerializable(final LogEvent event, final Writer writer)
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
