Mistake in reserve_time explanation in parser filter plugin

Open Beginner friendly
#439 1 comment 0 reactions 0 assignees View on GitHub

Nobody has claimed this yet.

Assessment

Difficulty
1/5
Estimated time
Under an hour
Newbie friendliness
76/100
Issue type
Documentation
Clarity
Clearly specified
Activity status
Quiet
Domain
documentation

Research direction

Start at the parser filter documentation at https://docs.fluentd.org/filter/parser#reserve_time and review the reserve_time explanation against the issue's expected output. Correct the wording so it distinguishes the original event time from the parsed timestamp for true and false, then verify that the examples and default behavior are consistent.

Written by the indexing model from the issue text.

Description

bug good first issue
Describe the bug

I believe there is a mistake in the explanation for reserve_time property in parser filter plugin.

It is saying that when reserve_time: true, the time of the event is parsed from the specified parsing field (in that case, it is log).
However, it is the other way around (i.e. time of the event is kept and parsed time is discarded).

Link to the problematic documentation

https://docs.fluentd.org/filter/parser#reserve_time

Expected explanation

When reserve_time: true, the output is:

time:
2021-06-24 14:33:35.475115751 +0900 (It vary on parsed timestamp)

record:
{
  "user": 1
}

When reserve_time: false (default value), the output is:

time:
2021-06-01 00:00:00.000000000 +0900

record:
{
  "user": 1
}
Additional context

No response

Dominant language
CSS
Stars
46
Forks
136
Avg merge
4h 47m
Merged PRs (30d)
16

Contributor guide

No contributing guide indexed for this repository

First steps

  1. Read the whole issue, then the project's contributing guide.
  2. Comment on the issue to say you are picking it up — it saves two people doing the same work.
  3. Fork the repository and make your change on a branch.
  4. Open a pull request that references the issue number.

More from fluent/fluentd-docs-gitbook

All issues in fluent/fluentd-docs-gitbook

Similar issues

More Documentation issues

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.