fix: various clarifications

toddbaert commented 1 year ago

loosen shutdown requirements to support things like go ctx https://github.com/open-feature/spec/issues/189
clarify how to handle init/shutdown of providers bound to multiple names https://github.com/open-feature/spec/issues/192
~clarify init function should terminate before provider used https://github.com/open-feature/java-sdk/pull/456#discussion_r1210725372~
clarify that authors can await the READY event during init https://github.com/open-feature/java-sdk/pull/456#discussion_r1210725372

cc @open-feature/technical-steering-committee @lopitz @lukas-reining @kinyoklion @Kavindu-Dodan

Definitely let me know anyone thinks something is missing or could be improved here.

s-sen commented 1 year ago

The spec shows that the initialization accepts a global context as a parameter. However the java sdk provider interface has the initialization API without any parameters. Is there any discussion regarding that?

toddbaert commented 1 year ago

The spec shows that the initialization accepts a global context as a parameter. However the java sdk provider interface has the initialization API without any parameters. Is there any discussion regarding that?

I think this is an oversight. I will open an issue in the Java SDK (we haven't released yet, so no issue). Great find.

open-feature / spec

fix: various clarifications #193