Skip to content
This repository has been archived by the owner on Jun 29, 2023. It is now read-only.

Replace json-simple by own JSON serializer #89

Closed
mp911de opened this issue Jul 13, 2016 · 1 comment
Closed

Replace json-simple by own JSON serializer #89

mp911de opened this issue Jul 13, 2016 · 1 comment
Labels
type: enhancement A general enhancement
Milestone
logstash-gelf 1.11.0

Comments

@mp911de
Copy link
Owner

mp911de commented Jul 13, 2016

No description provided.

@mp911de mp911de added the type: enhancement A general enhancement label Jul 13, 2016
mp911de added a commit that referenced this issue Jul 13, 2016
@mp911de
Replace json-simple by own JSON serializer #89
ef2bccc 
Replace json-simple by own JSON writer and perform UTF-8 encoding within the JSON writer.

Motivation: json-simple produces a notable amount of garbage during JSON serialization. The default interchange format is String that concatenates data using StringBuilder.

The own implementation uses a facade to write bytes on an underlying stream. This reduces the garbage so data is not necessarily duplicated during serialization.
@mp911de mp911de added this to the logstash-gelf 1.11.0 milestone Jul 18, 2016
@mp911de
Copy link
Owner Author

mp911de commented Jul 18, 2016

Done.

@mp911de mp911de closed this as completed Jul 18, 2016
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Assignees
No one assigned
Labels
type: enhancement A general enhancement
Projects
None yet
Milestone
logstash-gelf 1.11.0
Development

No branches or pull requests
1 participant
@mp911de