You are viewing a plain text version of this content. The canonical link for it is here.
Posted to dev@tamaya.apache.org by "Anatole Tresch (JIRA)" <ji...@apache.org> on 2017/11/22 14:25:00 UTC
[jira] [Created] (TAMAYA-321) Replace/Deprecate
ConfigurationContextBuilder in favour of ConfigurationBuilder
Anatole Tresch created TAMAYA-321:
-------------------------------------
Summary: Replace/Deprecate ConfigurationContextBuilder in favour of ConfigurationBuilder
Key: TAMAYA-321
URL: https://issues.apache.org/jira/browse/TAMAYA-321
Project: Tamaya
Issue Type: Improvement
Components: API
Affects Versions: 0.3-incubating
Reporter: Anatole Tresch
Assignee: Anatole Tresch
Fix For: 0.4-incubating
Building a configuration is valid use case. Unfortunately as of now, a user must built a {{ConfigurationContext}} to create a {{Configuration}}:
{quote}ConfigurationContextBuilder b = ConfigurationProvider.getConfigurationContextBuilder();
// build up context with b
ConfigurationContext ctx = b.build();
Configuration cfg = ConfigurationProvider.createConfiguration(ctx);{quote}
This is cumbersome. Whereas {{ConfigurationContext}} as a container for all the small parts, that build up a Configuration makes sense, the user basically should be able to build a configuration directly:
{quote}ConfigurationBuilder b = ConfigurationProvider.getConfigurationBuilder();
// build up config with b
Configuration cfg = b.build();{quote}
h2. Proposal
*Add:*
- _ConfigurationBuilder_ (same functionaly as former ConfigurationContextBuilder, but for building a Configuration).
- _ConfigurationBuilder ConfigurationProvider.getConfigurationBuilder()_
- _ConfigurationBuilder ConfigurationProviderSpi#getConfigurationBuilder()_
-_ ConfigurationBuilder Configuration#toBuilder();_
*Deprecate*:
- _ConfigurationContextBuilder_
- _ConfigurationContextBuilder ConfigurationProvider.getConfigurationContextBuilder()_
- _ConfigurationContextBuilder ConfigurationProviderSpi#getConfigurationContextBuilder()_
- _ConfigurationContextBuilder ConfigurationContext#toBuilder()_
--
This message was sent by Atlassian JIRA
(v6.4.14#64029)
Re: [jira] [Created] (TAMAYA-321) Replace/Deprecate
ConfigurationContextBuilder in favour of ConfigurationBuilder
Posted by "P. Ottlinger" <po...@apache.org>.
Am 22.11.2017 um 15:25 schrieb Anatole Tresch (JIRA):
> Anatole Tresch created TAMAYA-321:
> -------------------------------------
>
> Summary: Replace/Deprecate ConfigurationContextBuilder in favour of ConfigurationBuilder
> Key: TAMAYA-321
> URL: https://issues.apache.org/jira/browse/TAMAYA-321
> Project: Tamaya
> Issue Type: Improvement
> Components: API
> Affects Versions: 0.3-incubating
> Reporter: Anatole Tresch
> Assignee: Anatole Tresch
> Fix For: 0.4-incubating
>
>
> Building a configuration is valid use case. Unfortunately as of now, a user must built a {{ConfigurationContext}} to create a {{Configuration}}:
>
> {quote}ConfigurationContextBuilder b = ConfigurationProvider.getConfigurationContextBuilder();
> // build up context with b
> ConfigurationContext ctx = b.build();
> Configuration cfg = ConfigurationProvider.createConfiguration(ctx);{quote}
>
> This is cumbersome. Whereas {{ConfigurationContext}} as a container for all the small parts, that build up a Configuration makes sense, the user basically should be able to build a configuration directly:
> {quote}ConfigurationBuilder b = ConfigurationProvider.getConfigurationBuilder();
> // build up config with b
> Configuration cfg = b.build();{quote}
>
> h2. Proposal
> *Add:*
> - _ConfigurationBuilder_ (same functionaly as former ConfigurationContextBuilder, but for building a Configuration).
> - _ConfigurationBuilder ConfigurationProvider.getConfigurationBuilder()_
> - _ConfigurationBuilder ConfigurationProviderSpi#getConfigurationBuilder()_
> -_ ConfigurationBuilder Configuration#toBuilder();_
>
> *Deprecate*:
> - _ConfigurationContextBuilder_
> - _ConfigurationContextBuilder ConfigurationProvider.getConfigurationContextBuilder()_
> - _ConfigurationContextBuilder ConfigurationProviderSpi#getConfigurationContextBuilder()_
> - _ConfigurationContextBuilder ConfigurationContext#toBuilder()_
+1 for your proposal.
Cheers,
Phil