[Merged by Bors] - Remove global initialization of logger

[marcmo]

• fixes Allow to configure a custom logging backend #134
• Provide macro to run code on initialization without introducing
  additional direct dependencies to client projects.
• without this global initialization applications need to setup their
  own logging infrastructure at startup
• Adapted function example with the use of a custom logger

[nicholastmosher]

Hi @marcmo, thanks for the PR! We're hoping to test this out and give some feedback soon 👍

[marcmo]

@nicholastmosher is this still something you consider? Then I would add some documentation for it.

[sehz]

Sorry for lates response. I think there is forward step to resolving this issue

[marcmo]

@sehz I created a separate example to demonstrate the custom logger.
By regression you mean that I removed a public function probably...would it be sufficient to add the init_logger() function back in but do not call it?

[sehz]

@sehz I created a separate example to demonstrate the custom logger.
By regression you mean that I removed a public function probably...would it be sufficient to add the init_logger() function back in but do not call it?

By default (if custom init is not called), we need to initialize logger. Otherwise, everyone need to initialize logger explicitly. I can take look see if there is way to insert global logger if not initialized

[marcmo]

IMHO initialization of a logging framework is something every application should do individually. At least, that's how almost all libraries/frameworks I know do it.
But maybe I'm thinking of node-bindgen too much of a separate project while it is meant to be used in the fluvio world...

[sehz]

Hmm. I think you have valid point. This is meant to to be used in any library/application. Default initializing maybe too opinionated and also may be more complexity than warranted. In that case, this PR seems to be good stage

[sehz]

Thanks for pushing this thru

[sehz]

I will bump up version change after this goes thru

[sehz]

bors r+

[bors]

Pull request successfully merged into master.

Build succeeded:

• Done