[ https://issues.apache.org/jira/browse/CASSANDRA-8109?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=14171099#comment-14171099 ]
Brandon Williams commented on CASSANDRA-8109: --------------------------------------------- Actually to be an assignee that means you have to be a contributor, so if we add someone who hasn't contributed before just to assign, then they never post a patch, we have go undo the addition which is painful. Much better to just leave a comment, post a patch, then we'll assign on commit. > Avoid constant boxing in ColumnStats.{Min/Max}Tracker > ----------------------------------------------------- > > Key: CASSANDRA-8109 > URL: https://issues.apache.org/jira/browse/CASSANDRA-8109 > Project: Cassandra > Issue Type: Improvement > Reporter: Sylvain Lebresne > Assignee: Rajanarayanan Thottuvaikkatumana > Priority: Minor > Labels: lhf > Fix For: 3.0 > > > We use the {{ColumnStats.MinTracker}} and {{ColumnStats.MaxTracker}} to track > timestamps and deletion times in sstable. Those classes are generics but we > really ever use them for longs and integers. The consequence is that every > call to their {{update}} method (called for every cell during sstable write) > box it's argument (since we don't store the cell timestamps and deletion time > boxed). That feels like a waste that is easy to fix: we could just make those > work on longs only for instance and convert back to int at the end when > that's what we need. -- This message was sent by Atlassian JIRA (v6.3.4#6332)