Ayman Khalil

Results 30 comments of Ayman Khalil

@sbtourist I separated index writes into their own sensor (`INDEX_BYTES`) and added support for `SEARCH_BYTES` utilizing `TrackingRowIterator` - please take a look.

@maxtomassi addressed your feedback in https://github.com/datastax/cassandra/pull/1019/commits/24de7ab2bcfa7ca39b70379fbd4d2f81b3e7f1ea. PTAL.

The remaining failure in CI seems flakey, here is what butler has to say: http://butler-stargazer.aws.dsinternal.org/#/ci/upstream/compare/ds-cassandra-pr-gate/cndb-8501/to/ds-cassandra-build-nightly/ds-trunk

Thanks @timbray for the implementation hints! It is promising to see the possibility to add binary format support without compromising performance.

Seems nextToken() has huge code reusability advantage. I was wondering though if it limits the ability to 1) consult the schema if the pattern field exist in the event and...

@zoewangg thanks for the clarification. The Java-based multipart client is something we'd be very interested in, if you have more info on where/how to track that initiative, that would be...

Merged `main-ucu` into this PR https://github.com/datastax/cassandra/pull/1152/commits/d99686870ad8206588de93415c3070515579367a

Closed in favor of https://github.com/datastax/cassandra/pull/1184

> This looks ok to me conceptually. Is this ready for a review? Let me do the changes on CNDB side and then mark it as ready for review -...

Closed in favor of https://github.com/datastax/cassandra/pull/1184