You are viewing a plain text version of this content. The canonical link for it is here.

Posted to pr@cassandra.apache.org by GitBox <gi...@apache.org> on 2023/01/13 15:04:17 UTC

[GitHub] [cassandra] blambov commented on pull request #2064: CASSANDRA-17056: CEP-17

blambov commented on PR #2064:
URL: https://github.com/apache/cassandra/pull/2064#issuecomment-1381989273

   One other thing I think we should include in this is breaking the dependency loop between `SSTableReader` and `ColumnFamilyStore`. `SSTableReader` is a low-level class and should not know how its owner works, and even less about how to get hold of its owner via `Schema`.
   
   If necessary, we can introduce an `SSTableReader.Owner` interface (similar to `Memtable.Owner`) to expose only functionality it needs.


-- 
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: pr-unsubscribe@cassandra.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org


---------------------------------------------------------------------
To unsubscribe, e-mail: pr-unsubscribe@cassandra.apache.org
For additional commands, e-mail: pr-help@cassandra.apache.org