[Idea] Hiding plugin instance at library mode about fluent-bit HOT 8 CLOSED

the main reason of getting a reference to the input/output instance, is to be able to set properties or any other kind of future modification.

Note that when the end-user call flb_stop(ctx), the engine receives the message and it start releasing resources, internally it does an exit for all input/output instances associated to the main context.

The user never need to do an explicit release of the instances, all this happens in the background.

from fluent-bit.

Thank you for your comment.
I noticed that flb_ctx_t already contains plugin instances.

the main reason of getting a reference to the input/output instance, is to be able to set properties or any other kind of future modification.

So, why should we expose a plugin instance ?
User passes these instances to flb_*put_set as identifier.
I think structure of instance is not necessarily need, only identifier value is needed.

The user never need to do an explicit release of the instances, all this happens in the background.

I understand. But I think it is odd from user side.
Because flb_stop(ctx) doesn't contain plugin instances at first sight.
However this function influences plugin instances.
This means fluent-bit treats these instances as part of context implicitly.

So , I proposed that plugin instances should be hid from user and modify API.
flb_input_set( input_ins, ... ) -> flb_input_set( input_id, ctx, ... )

from fluent-bit.

I am thinking about this, no strong opinion yet, once I finish buffering/scheduling stuff I will focus on API and share more updates.

from fluent-bit.

I see, I'm waiting.
Thanks!

from fluent-bit.

I am reviewing this. I think it make sense to have an integer as identifier, thanks for pointing that.

from fluent-bit.

@nokute78 I have pushed the changes into api_ffd branch.

note: the remaining part are tests.

from fluent-bit.

I just migrated all tests/examples, changes rebased on master.

If is there anything pending or something not working as suggested please add your comments to this tickets.

from fluent-bit.

Amazing!! That's what I imagined!
Thanks.

from fluent-bit.