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 12EE418E61 for ; Thu, 6 Aug 2015 18:56:06 +0000 (UTC) Received: (qmail 67346 invoked by uid 500); 6 Aug 2015 18:56:05 -0000 Delivered-To: apmail-cassandra-commits-archive@cassandra.apache.org Received: (qmail 67315 invoked by uid 500); 6 Aug 2015 18:56:05 -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 67302 invoked by uid 99); 6 Aug 2015 18:56:05 -0000 Received: from arcas.apache.org (HELO arcas.apache.org) (140.211.11.28) by apache.org (qpsmtpd/0.29) with ESMTP; Thu, 06 Aug 2015 18:56:05 +0000 Date: Thu, 6 Aug 2015 18:56:05 +0000 (UTC) From: "Aleksey Yeschenko (JIRA)" To: commits@cassandra.apache.org Message-ID: In-Reply-To: References: Subject: [jira] [Commented] (CASSANDRA-9265) Add checksum to saved cache files 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-9265?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=14660609#comment-14660609 ] Aleksey Yeschenko commented on CASSANDRA-9265: ---------------------------------------------- Committed as {{fa6205c909656b09165da4b5ca469328a6450917}} to cassandra-2.2 and merged into cassandra-3.0 and trunk. Fixed "Merged from" in the 3.0 patch CHANGES.txt on commit. > Add checksum to saved cache files > --------------------------------- > > Key: CASSANDRA-9265 > URL: https://issues.apache.org/jira/browse/CASSANDRA-9265 > Project: Cassandra > Issue Type: Improvement > Reporter: Ariel Weisberg > Assignee: Daniel Chia > Fix For: 2.2.1, 3.0 beta 1 > > Attachments: 0001-Add-checksum-to-saved-cache-files-CASSANDRA-9265.patch, 0002-trunk-CASSANDRA-9265.patch > > > Saved caches are not covered by a checksum. We should at least emit a checksum. My suggestion is a large checksum of the whole file (convenient offline validation), and then smaller per record checksums after each record is written (possibly a subset of the incrementally maintained larger checksum). > I wouldn't go for anything fancy to try to recover from corruption since it is just a saved cache. If corruption is detected while reading I would just have it bail out. I would rather have less code to review and test in this instance. -- This message was sent by Atlassian JIRA (v6.3.4#6332)