Skip to content
This repository has been archived by the owner. It is now read-only.

Handle Fragmentation, small fixes #5

Closed
wants to merge 3 commits into from

Conversation

@tabletcorry
Copy link
Contributor

@tabletcorry tabletcorry commented Aug 30, 2011

This commit primarily fixes a relatively bad fragmentation issue in the collectd messages.

When the fragmentation occurs, data is corrupted. The most common symptom is COUNTER type values suddenly loosing a few orders of magnitude (really messes up derivative calculations).

Other commits remove old files, and improve docs a bit.

Corry Haines added 3 commits Aug 26, 2011
@donjohnson checked this file in, but my changes made it less useful.
Collectd had a bad habit of fragmenting messages. In short, it will
send two POSTS that are meant to be handled together. In some cases,
these POSTS will each contain part of a single line of data.

This logic stores fragments for the next POST to put back together.
It discards the fragment from the first POST because it sometimes is
complete enough to parse and really mess up the DB.

Fragmentation seems to be discoverable by looking for a lack of blank
line at the end of the first POST.

Example fragments:
First POST, last line: PUTVAL host.example.com/disk-d
Second POST, first line: m-0/disk_octets interval=10 1314724861:654568
@tabletcorry
Copy link
Contributor Author

@tabletcorry tabletcorry commented Aug 30, 2011

@jordansissel I think it almost works now :)

@tabletcorry
Copy link
Contributor Author

@tabletcorry tabletcorry commented Dec 11, 2017

Cleaning up ancient/stale pull requests

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Linked issues

Successfully merging this pull request may close these issues.

None yet

1 participant
You can’t perform that action at this time.