-
Notifications
You must be signed in to change notification settings - Fork 1.3k
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
PROPOSE: add format raw #182
Conversation
Structured raw logs come here! |
👍 |
👍 |
2 similar comments
👍 |
👍 |
I want more clear name... |
|
||
def call(text) | ||
record = {} | ||
record[@message_key] = text |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
🌻
proposal: "format message" It still creates a structured record and its key name is "msssage". |
👍 totally up for this one. sometimes you just dont want to parse the stuff.. |
👍 |
1 similar comment
👍 |
proposal: "format none" I think that the value of In contrast, I think How do you think? |
👍 > or How about |
I feel |
proposal: This indicates directly what will comes in records. |
👍 > |
@repeatedly it seems |
All of you can use fluent-agent-lite for raw-plain-whole-message record. |
@tagomoris Anybody knows your work |
👍 > format none |
For me, I wait a day for decision. |
@repeatedly renamed |
|
|
thanks!! |
I propose to add
format raw
.Currently, even if users do not require to parse log lines, we have to write configuration like
This requires to parse log lines by a regular expression, and its overhead is not trivial.
cf. http://blog.1q77.com/2013/01/fluent-agent-lite-%E3%81%A8-in_tail/
This plugin enables to write configuration like
(where
message_key
is set tomessage
as default) and does not require to parse log lines by a regular expression.There is a plugin fluent-plugin-tail-asis which does the same thing with this patch, and downloaded 2354 times as in 2013.09.02 http://fluentd.org/plugin/. It tells us that some users are already using this feature, and I propose to have this feature in the fluentd itself.