belliottsmith commented on PR #21: URL: https://github.com/apache/cassandra-accord/pull/21#issuecomment-1404223969
Thanks, agreed that it would be nice to expand test cases a little, but I think it is adequately covered for now, between the RangeDeps and burn tests. I believe I have some TODO to address improving coverage in the future, and if not I will add them. > I hope that we'll be able to reuse it in other places This is obviously a long term idea, but the same logic can be applied very straightforwardly to sstables, bringing is efficient range queries, at least within a node if the primary key supports it. For system tables for range transaction state management at least this is a possibility I’ve considered. Maybe the structure will come in handy elsewhere too. > I'll keep it all in my head for a few more days in case I can come up with some improvement ideas thanks! -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comment. To unsubscribe, e-mail: [email protected] For queries about this service, please contact Infrastructure at: [email protected] --------------------------------------------------------------------- To unsubscribe, e-mail: [email protected] For additional commands, e-mail: [email protected]
