Context.Storage is experimental

[zhangkun83]

This is the tracking issue that will be closed once Context.Storage API exits experimental status.

[larry-safran]

This allows people to change where the values are stored instead of the default ThreadLocal. Used in G3 for ... complex stuff. Also used for other things. Might want additive changes.

[larry-safran]

Context.Storage.attach() isn't used by anyone in our repo or G3 and can be deleted with Context.Storage.doAttach being marked abstract.

[tonyjongyoonan]

On hold since other repos (e.g. Vert.x and OpenTelemetry) depend on Context, may not be safe to delete deprecated method.

[larry-safran]

The following query gets all of the usages. Should be checked to ensure that none of them are really using attach . https://sourcegraph.com/search?q=context:global+class+ContextStorageOverride+lang:java&patternType=standard&sm=0&groupBy=repo

[tonyjongyoonan]

Quarkus.io is the only repo that uses attach().

It is not used anywhere but the override would throw an error if the method is deleted.

[ejona86]

The API is still labeled as experimental.