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

Add support for filter plugins #33

Open
theMagicalKarp opened this issue May 28, 2020 · 3 comments
Open

Add support for filter plugins #33

theMagicalKarp opened this issue May 28, 2020 · 3 comments

Comments

@theMagicalKarp
Copy link

No description provided.

@edsiper
Copy link
Member

edsiper commented May 28, 2020

why do you need Go filters ?

Actually the context transition from C -> Go runtime is very expensive, filtering needs to happen as fast as possible, we offer Lua for filtering which is really fast. Is there anything that Lua cannot offer but can be done in Go ?

@theMagicalKarp
Copy link
Author

theMagicalKarp commented May 28, 2020

I'd like to switch to fluent-bit and port our internal fluentd filter plugins.

In my opinion, It's more a matter of convenience than performance. The teams I work on are a lot more comfortable with using Go over C or Lua. (Which is what I thought the purpose of this repo was?)

When you say it's expensive, can you explain why that is? What's the order of magnitude?

Also seems like this is dependent on this change. fluent/fluent-bit#1277

@ericmillin
Copy link

I think WASM support will cover this case. Compile your Go to WASM and you're good to Go ;)

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

No branches or pull requests

3 participants