Return-Path: X-Original-To: apmail-accumulo-notifications-archive@minotaur.apache.org Delivered-To: apmail-accumulo-notifications-archive@minotaur.apache.org Received: from mail.apache.org (hermes.apache.org [140.211.11.3]) by minotaur.apache.org (Postfix) with SMTP id E699A11018 for ; Thu, 28 Aug 2014 14:20:08 +0000 (UTC) Received: (qmail 18477 invoked by uid 500); 28 Aug 2014 14:20:08 -0000 Delivered-To: apmail-accumulo-notifications-archive@accumulo.apache.org Received: (qmail 18426 invoked by uid 500); 28 Aug 2014 14:20:08 -0000 Mailing-List: contact notifications-help@accumulo.apache.org; run by ezmlm Precedence: bulk List-Help: List-Unsubscribe: List-Post: List-Id: Reply-To: jira@apache.org Delivered-To: mailing list notifications@accumulo.apache.org Received: (qmail 18229 invoked by uid 99); 28 Aug 2014 14:20:08 -0000 Received: from arcas.apache.org (HELO arcas.apache.org) (140.211.11.28) by apache.org (qpsmtpd/0.29) with ESMTP; Thu, 28 Aug 2014 14:20:08 +0000 Date: Thu, 28 Aug 2014 14:20:08 +0000 (UTC) From: "Adam Fuchs (JIRA)" To: notifications@accumulo.apache.org Message-ID: In-Reply-To: References: Subject: [jira] [Created] (ACCUMULO-3088) StatsIterator readCounter could be more accurate MIME-Version: 1.0 Content-Type: text/plain; charset=utf-8 Content-Transfer-Encoding: 7bit X-JIRA-FingerPrint: 30527f35849b9dde25b450d4833f0394 Adam Fuchs created ACCUMULO-3088: ------------------------------------ Summary: StatsIterator readCounter could be more accurate Key: ACCUMULO-3088 URL: https://issues.apache.org/jira/browse/ACCUMULO-3088 Project: Accumulo Issue Type: Bug Reporter: Adam Fuchs Priority: Trivial When testing seek performance I noticed that the read count displayed on Accumulo's monitor page registered zero entries read per second and a positive number of entries per second returned. This should not be possible. There is a fencepost error in the StatsIterator where seeks don't count as entries read. Successful completion of this ticket should handle the cases where a seek returns no entries, as well as where next is never called on an iterator. The cost of incrementing an AtomicLong is something like 10ns, which is at least an order of magnitude more than the desired overhead. The current way of batching updates to the read counter should be preserved for performance. -- This message was sent by Atlassian JIRA (v6.2#6252)