maedhroz commented on code in PR #2916:
URL: https://github.com/apache/cassandra/pull/2916#discussion_r1402643142
##########
src/java/org/apache/cassandra/index/sai/disk/v1/MemtableIndexWriter.java:
##########
@@ -55,19 +55,25 @@ public class MemtableIndexWriter implements
PerColumnIndexWriter
private static final Logger logger =
LoggerFactory.getLogger(MemtableIndexWriter.class);
private final IndexDescriptor indexDescriptor;
- private final IndexContext indexContext;
+ private final IndexTermType indexTermType;
+ private final IndexIdentifier indexIdentifier;
+ private final IndexMetrics indexMetrics;
Review Comment:
nit: Another one of these cases where I guess we could just pass and store
the SAI...if it doesn't make testing harder...
--
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]
