Return-Path: X-Original-To: apmail-cassandra-commits-archive@www.apache.org Delivered-To: apmail-cassandra-commits-archive@www.apache.org Received: from mail.apache.org (hermes.apache.org [140.211.11.3]) by minotaur.apache.org (Postfix) with SMTP id D4AC3183E7 for ; Mon, 7 Dec 2015 07:00:18 +0000 (UTC) Received: (qmail 93042 invoked by uid 500); 7 Dec 2015 07:00:18 -0000 Delivered-To: apmail-cassandra-commits-archive@cassandra.apache.org Received: (qmail 93005 invoked by uid 500); 7 Dec 2015 07:00:18 -0000 Mailing-List: contact commits-help@cassandra.apache.org; run by ezmlm Precedence: bulk List-Help: List-Unsubscribe: List-Post: List-Id: Reply-To: dev@cassandra.apache.org Delivered-To: mailing list commits@cassandra.apache.org Received: (qmail 92955 invoked by uid 99); 7 Dec 2015 07:00:18 -0000 Received: from arcas.apache.org (HELO arcas) (140.211.11.28) by apache.org (qpsmtpd/0.29) with ESMTP; Mon, 07 Dec 2015 07:00:18 +0000 Received: from arcas.apache.org (localhost [127.0.0.1]) by arcas (Postfix) with ESMTP id 5D5BD2C1F5C for ; Mon, 7 Dec 2015 07:00:18 +0000 (UTC) Date: Mon, 7 Dec 2015 07:00:18 +0000 (UTC) From: "Stefania (JIRA)" To: commits@cassandra.apache.org Message-ID: In-Reply-To: References: Subject: [jira] [Updated] (CASSANDRA-10529) Channel.size() is costly, mutually exclusive, and on the critical path MIME-Version: 1.0 Content-Type: text/plain; charset=utf-8 Content-Transfer-Encoding: 7bit X-JIRA-FingerPrint: 30527f35849b9dde25b450d4833f0394 [ https://issues.apache.org/jira/browse/CASSANDRA-10529?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ] Stefania updated CASSANDRA-10529: --------------------------------- Component/s: Local Write-Read Paths > Channel.size() is costly, mutually exclusive, and on the critical path > ---------------------------------------------------------------------- > > Key: CASSANDRA-10529 > URL: https://issues.apache.org/jira/browse/CASSANDRA-10529 > Project: Cassandra > Issue Type: Bug > Components: Local Write-Read Paths > Reporter: Benedict > Assignee: Stefania > Fix For: 3.0.0 rc2 > > > [~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. -- This message was sent by Atlassian JIRA (v6.3.4#6332)