cassandra-commits mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From "Sylvain Lebresne (JIRA)" <j...@apache.org>
Subject [jira] Updated: (CASSANDRA-699) Add (optional) expiration time for column
Date Fri, 16 Apr 2010 15:18:32 GMT

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

Sylvain Lebresne updated CASSANDRA-699:
---------------------------------------

    Attachment: 699-1-Add-DeletedColumn-class.patch
                699-2-Add-optional-expiration-for-columns.patch
                699-3-Update-system-test-for-expiring-column.patch

Added new patches (rebased on trunk).
These patches tries an approach slightly different from the AbstractColumn idea. 
It changes the Column class to be only non deleted column and add a 
DeletedColumn class. That way ExpiringColumn can subclass Column without 
having the unecessary isMarkedForDelete boolean. I do believe this is much cleaner
(even though this add the new DeletedColumn class, we only have to change the 
few small places where Column are instanciated). 

Anyway, the first attached patch only do the change towards this DeletedColumn 
idea. Then the second patches add the ExpiringColumn on top of that. I went over 
the code and made a few small modification to make it cleaner, but no major changes
from the previous patches here). 

The third patch update the system tests and add a few for expiring columns (same tests
as in previous version of the patch).



> Add (optional) expiration time for column 
> ------------------------------------------
>
>                 Key: CASSANDRA-699
>                 URL: https://issues.apache.org/jira/browse/CASSANDRA-699
>             Project: Cassandra
>          Issue Type: New Feature
>          Components: Core
>            Reporter: Sylvain Lebresne
>            Assignee: Sylvain Lebresne
>             Fix For: 0.7
>
>         Attachments: 699-1-Add-DeletedColumn-class.patch, 699-2-Add-optional-expiration-for-columns.patch,
699-3-Update-system-test-for-expiring-column.patch, add_expiring_column_0.6-rc1.diff, modify_test_0.6-rc1.diff,
thrift_API_change_0.6-rc1.diff
>
>
> Allow to attach an expiration time to a column. 
> An expired column would act much like a 'markedForDelete' one. 

-- 
This message is automatically generated by JIRA.
-
If you think it was sent incorrectly contact one of the administrators: https://issues.apache.org/jira/secure/Administrators.jspa
-
For more information on JIRA, see: http://www.atlassian.com/software/jira

        

Mime
View raw message