lukasz-antoniak opened a new pull request, #110: URL: https://github.com/apache/cassandra-analytics/pull/110
Status: Work in progress. Contributions welcome! Although tests passes and users can read V5 SSTables, there are various things to fix on a code level: - [ ] `FileSystemSSTable` or any SSTable should support re-buffering on random position to allow BTI partition index reading. - [ ] Code uses `@VisibleForTesting` methods from C* API in `BtiReaderUtils.createFileHandle`. BTI reading components require `FileHandle` as an argument, and I am trying to create one based on `InputStream` that we receive from bridge interface. - [ ] BTI index readers (e.g. `BtiReaderUtils`) do not expose statistics as number of bytes skipped. Due to characteristic of BTI index reading, I am not sure if this metric becomes useful. - [ ] Enable again v4 bridges. My IntelliJ had issues with debugging when same class was present in two Cassandra JARs. - [ ] Tests encountered [CASSANDRA-19902](https://github.com/apache/cassandra-analytics/commit/384f150bf17756ed1614f7ae3170fcad2644ffa3). I have applied a patch, but had to link dtest JAR creation to my own Cassandra fork. All places where I was no sure about the changes, are marked as `TODO(c4c5)`. -- 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: commits-unsubscr...@cassandra.apache.org For queries about this service, please contact Infrastructure at: us...@infra.apache.org --------------------------------------------------------------------- To unsubscribe, e-mail: commits-unsubscr...@cassandra.apache.org For additional commands, e-mail: commits-h...@cassandra.apache.org
