Skip to content

Conversation

@daipom
Copy link
Contributor

@daipom daipom commented Jan 10, 2025

fluent/fluentd#4661

Signed-off-by: Daijiro Fukuda <fukuda@clear-code.com>
@daipom daipom requested review from Watson1978 and kenhys January 10, 2025 10:10
Signed-off-by: Daijiro Fukuda <fukuda@clear-code.com>
Co-authored-by: Shizuo Fujita <fujita@clear-code.com>
Copy link
Contributor

@cosmo0920 cosmo0920 left a comment

Choose a reason for hiding this comment

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

I commented several concerns at the first glance.

daipom and others added 2 commits January 15, 2025 16:43
Signed-off-by: Daijiro Fukuda <fukuda@clear-code.com>

Co-authored-by: Hiroshi Hatake <hiroshi@chronosphere.io>
Signed-off-by: Daijiro Fukuda <fukuda@clear-code.com>
Co-authored-by: Hiroshi Hatake <hiroshi@chronosphere.io>
@daipom
Copy link
Contributor Author

daipom commented Jan 15, 2025

@cosmo0920
Thanks for your review! Fixed!

Signed-off-by: Daijiro Fukuda <fukuda@clear-code.com>
Co-authored-by: Shizuo Fujita <fujita@clear-code.com>
Co-authored-by: Kentaro Hayashi <kenhys@gmail.com>
Copy link
Contributor

@kenhys kenhys left a comment

Choose a reason for hiding this comment

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

LGTM.

@kenhys kenhys merged commit 47d7af5 into fluent:1.0 Jan 15, 2025
1 check passed
@daipom daipom deleted the add-source-only-mode branch January 15, 2025 09:00
@daipom
Copy link
Contributor Author

daipom commented Jan 15, 2025

Thanks for your review!!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants