cassandra-commits mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From "Robert Stupp (JIRA)" <j...@apache.org>
Subject [jira] [Updated] (CASSANDRA-12199) Config class uses boxed types but DD exposes primitive types
Date Sat, 20 Aug 2016 12:12:22 GMT

     [ https://issues.apache.org/jira/browse/CASSANDRA-12199?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel
]

Robert Stupp updated CASSANDRA-12199:
-------------------------------------
    Status: Patch Available  (was: Open)

Patch replacing boxed with primitive types available.
||trunk|[branch|https://github.com/apache/cassandra/compare/trunk...snazy:12199-primitives-config-trunk]|[testall|http://cassci.datastax.com/view/Dev/view/snazy/job/snazy-12199-primitives-config-trunk-testall/lastSuccessfulBuild/]|[dtest|http://cassci.datastax.com/view/Dev/view/snazy/job/snazy-12199-primitives-config-trunk-dtest/lastSuccessfulBuild/]


> Config class uses boxed types but DD exposes primitive types
> ------------------------------------------------------------
>
>                 Key: CASSANDRA-12199
>                 URL: https://issues.apache.org/jira/browse/CASSANDRA-12199
>             Project: Cassandra
>          Issue Type: Improvement
>          Components: Configuration
>            Reporter: Robert Stupp
>            Assignee: Robert Stupp
>            Priority: Minor
>              Labels: lhf
>             Fix For: 3.x
>
>
> {{Config}} class contains a lot of properties that are defined using boxed types - ({{Config.dynamic_snitch_update_interval_in_ms}})
but the corresponding get-methods in {{DatabaseDescriptor}} require them to be not null. Means,
setting such properties to {{null}} will lead to NPEs anyway.
> Proposal:
> * Identify all properties that use boxed values and have a default value (e.g. {{public
Integer rpc_port = 9160;}})
> * Refactor those to use primitive types



--
This message was sent by Atlassian JIRA
(v6.3.4#6332)

Mime
View raw message