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

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

    XMLWordPrintableJSON

Details

    • 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

              stefania Stefania Alborghetti
              benedict Benedict Elliott Smith
              Stefania Alborghetti
              Benedict Elliott Smith
              Votes:
              0 Vote for this issue
              Watchers:
              1 Start watching this issue

              Dates

                Created:
                Updated:
                Resolved: