[jira] [Commented] (CASSANDRA-8171) Clean up generics
[ https://issues.apache.org/jira/browse/CASSANDRA-8171?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=14188482#comment-14188482 ] Joshua McKenzie commented on CASSANDRA-8171: Might want a trivial comment on intention of package-private on 'final C token' in AbstractToken - can add on commit. The reduction in noise surrounding generics on Token is particularly nice - TokenFactory's are a lot cleaner now. Reducing 1/4 of our warnings is a great step - I'm +1 on us investigating moving to a "warning as error" paradigm on builds down the road someday; perhaps we should consider adding a build target w/warnings shown on a separate ticket? (Assuming we don't already have this and I'm just missing it in the build.xml...) I'm +1 on commit as-is. [~jbellis]: 2.1.2 or 3.0? It's a pretty straightforward refactor which leans me toward 2.1.2 but it's also dealing with some fundamental primitives of the system. I'm on the fence. > Clean up generics > - > > Key: CASSANDRA-8171 > URL: https://issues.apache.org/jira/browse/CASSANDRA-8171 > Project: Cassandra > Issue Type: Improvement >Reporter: Branimir Lambov >Assignee: Branimir Lambov >Priority: Minor > Attachments: 8171.patch > > > Some uses of generics in the code are causing much more harm than good, and > in some cases generic types are used unsafely, hiding potential problems in > the code. > Generics need to be cleaned up to clarify the types, remove unnecessary type > specialization when it does not make sense, and significantly reduce the > number of unhelpful warnings. -- This message was sent by Atlassian JIRA (v6.3.4#6332)
[jira] [Commented] (CASSANDRA-8171) Clean up generics
[ https://issues.apache.org/jira/browse/CASSANDRA-8171?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=14188967#comment-14188967 ] Jonathan Ellis commented on CASSANDRA-8171: --- I'm okay with 2.1.2. It will make merging easier and any bug should manifest pretty obviously. > Clean up generics > - > > Key: CASSANDRA-8171 > URL: https://issues.apache.org/jira/browse/CASSANDRA-8171 > Project: Cassandra > Issue Type: Improvement >Reporter: Branimir Lambov >Assignee: Branimir Lambov >Priority: Minor > Attachments: 8171.patch > > > Some uses of generics in the code are causing much more harm than good, and > in some cases generic types are used unsafely, hiding potential problems in > the code. > Generics need to be cleaned up to clarify the types, remove unnecessary type > specialization when it does not make sense, and significantly reduce the > number of unhelpful warnings. -- This message was sent by Atlassian JIRA (v6.3.4#6332)
[jira] [Commented] (CASSANDRA-8171) Clean up generics
[ https://issues.apache.org/jira/browse/CASSANDRA-8171?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=14190278#comment-14190278 ] Joshua McKenzie commented on CASSANDRA-8171: [~blambov]: could we get a version of this patch for the cassandra-2.1 branch? > Clean up generics > - > > Key: CASSANDRA-8171 > URL: https://issues.apache.org/jira/browse/CASSANDRA-8171 > Project: Cassandra > Issue Type: Improvement >Reporter: Branimir Lambov >Assignee: Branimir Lambov >Priority: Minor > Fix For: 2.1.2 > > Attachments: 8171.patch > > > Some uses of generics in the code are causing much more harm than good, and > in some cases generic types are used unsafely, hiding potential problems in > the code. > Generics need to be cleaned up to clarify the types, remove unnecessary type > specialization when it does not make sense, and significantly reduce the > number of unhelpful warnings. -- This message was sent by Atlassian JIRA (v6.3.4#6332)