Skip to content
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

processor_labels: Add a documentation for processor_labels #1371

Merged
merged 9 commits into from
Jul 26, 2024

Conversation

cosmo0920
Copy link
Contributor

Corresponding to fluent/fluent-bit#8824

@cosmo0920 cosmo0920 requested review from a team as code owners May 15, 2024 06:15
@cosmo0920 cosmo0920 force-pushed the cosmo0920-add-processor-labels-doc branch 3 times, most recently from 4a73996 to 11bc012 Compare May 15, 2024 06:23
pipeline/processors/labels.md Outdated Show resolved Hide resolved
pipeline/processors/labels.md Outdated Show resolved Hide resolved
pipeline/processors/labels.md Outdated Show resolved Hide resolved
pipeline/processors/labels.md Show resolved Hide resolved
pipeline/processors/labels.md Outdated Show resolved Hide resolved
pipeline/processors/labels.md Outdated Show resolved Hide resolved
This is because using hash algorithm is actually SHA-256.

Signed-off-by: Hiroshi Hatake <[email protected]>
@cosmo0920 cosmo0920 force-pushed the cosmo0920-add-processor-labels-doc branch from da963ab to be15335 Compare July 17, 2024 07:50
cosmo0920 and others added 2 commits July 17, 2024 16:59
Signed-off-by: Hiroshi Hatake <[email protected]>
Co-authored-by: Pat <[email protected]>
… insert and delete operations

Signed-off-by: Hiroshi Hatake <[email protected]>
@cosmo0920 cosmo0920 force-pushed the cosmo0920-add-processor-labels-doc branch from b92f4fb to f290a65 Compare July 17, 2024 07:59
@cosmo0920 cosmo0920 requested a review from lecaros July 25, 2024 14:55
@cosmo0920
Copy link
Contributor Author

@lecaros Could you take a look this PR? It’s long standing PR and we need to publish this doc if you are OK. Also, additional comments are welcomed. :)

Copy link
Contributor

@cnorris-cs cnorris-cs left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I left comments and suggestions, but I've added my technical writing approval based on those to keep this moving.

cosmo0920 and others added 5 commits July 26, 2024 11:38
Co-authored-by: Craig Norris <[email protected]>
Signed-off-by: Hiroshi Hatake <[email protected]>
Co-authored-by: Craig Norris <[email protected]>
Signed-off-by: Hiroshi Hatake <[email protected]>
Co-authored-by: Craig Norris <[email protected]>
Signed-off-by: Hiroshi Hatake <[email protected]>
Co-authored-by: Craig Norris <[email protected]>
Signed-off-by: Hiroshi Hatake <[email protected]>
Co-authored-by: Craig Norris <[email protected]>
Signed-off-by: Hiroshi Hatake <[email protected]>
@cosmo0920 cosmo0920 force-pushed the cosmo0920-add-processor-labels-doc branch from 764526d to 394c4d7 Compare July 26, 2024 02:40
@lecaros lecaros merged commit ec2542a into master Jul 26, 2024
4 checks passed
@lecaros lecaros deleted the cosmo0920-add-processor-labels-doc branch July 26, 2024 16:10
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants