[ 
https://issues.apache.org/jira/browse/CASSANDRASC-69?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=17762475#comment-17762475
 ] 

Dinesh Joshi commented on CASSANDRASC-69:
-----------------------------------------

+1, thanks for the patch!

> Refactor Configuration
> ----------------------
>
>                 Key: CASSANDRASC-69
>                 URL: https://issues.apache.org/jira/browse/CASSANDRASC-69
>             Project: Sidecar for Apache Cassandra
>          Issue Type: Improvement
>          Components: Configuration
>            Reporter: Francisco Guerrero
>            Assignee: Francisco Guerrero
>            Priority: Normal
>              Labels: pull-request-available
>
> Currently, the Sidecar Configuration is provided via a yaml file. The yaml 
> file is parsed and a the configuration is used during the Sidecar application 
> runtime.
> The configuration parsing is done manually, and adding a new configuration to 
> the yaml file has become a big task where multiple files need to be touched. 
> Ideally, we should add the configuration POJOs needed for new configurations 
> and add the new fields in the yaml file.
> Additionally, we are leaking the configuration implementation to the 
> {{InstancesConfig}}. We should improve the separation of concerns from the 
> yaml configuration and the instances configuration.



--
This message was sent by Atlassian Jira
(v8.20.10#820010)

---------------------------------------------------------------------
To unsubscribe, e-mail: commits-unsubscr...@cassandra.apache.org
For additional commands, e-mail: commits-h...@cassandra.apache.org

Reply via email to