Uploaded image for project: 'Cassandra'
  1. Cassandra
  2. CASSANDRA-10529

Channel.size() is costly, mutually exclusive, and on the critical path

    XMLWordPrintableJSON

    Details

    • Severity:
      Normal

      Description

      Stefania Alborghetti mentioned this already on another ticket, but I have lost track of exactly where. While benchmarking it became apparent this was a noticeable bottleneck for small in-memory workloads with few files, especially with RF=1. We should probably fix this soon, since it is trivial to do so, and the call is only to impose an assertion that our requested length is less than the file size. It isn't possible to safely memoize a value anywhere we can guarantee to be able to safely refer to it without some refactoring, so I suggest simply removing the assertion for now.

        Attachments

          Issue Links

            Activity

              People

              • Assignee:
                stefania Stefania Alborghetti
                Reporter:
                benedict Benedict Elliott Smith
                Authors:
                Stefania Alborghetti
                Reviewers:
                Benedict Elliott Smith
              • Votes:
                0 Vote for this issue
                Watchers:
                1 Start watching this issue

                Dates

                • Created:
                  Updated:
                  Resolved: