Supporting remove operations in PersistentVector
The current implementation of PersistentVector extends IndexedSeq, which does not offer any remove operation. Should we change the implementation to support "pop" ?
Leave a comment
on 2010-10-04 23:57 *
By scott.clasen
Assigned to set to scott.clasen
Status changed from New to Accepted
I see that this is stubbed out in Storage.scala, so it just needs a backend API. I can work on it, if thats ok.
on 2010-10-05 00:25 *
By scott.clasen
Comment in code says pop removes the tail of the vector. Since the add and + operations prepend elements to the vector, shouldnt the vector.pop remove the head?
Otherwise this would pretty much be a queue? Add to head, remove from tail?
Otherwise this would pretty much be a queue? Add to head, remove from tail?
Supported by Voldemort,Riak,Cassandra currently.
Other backends will throw UnsuportedOperationException on commit, and should be modified to support pop
Other backends will throw UnsuportedOperationException on commit, and should be modified to support pop