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 4F9FA171E2 for ; Tue, 13 Jan 2015 17:29:34 +0000 (UTC) Received: (qmail 67611 invoked by uid 500); 13 Jan 2015 17:29:35 -0000 Delivered-To: apmail-cassandra-commits-archive@cassandra.apache.org Received: (qmail 67569 invoked by uid 500); 13 Jan 2015 17:29:35 -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 67557 invoked by uid 99); 13 Jan 2015 17:29:35 -0000 Received: from arcas.apache.org (HELO arcas.apache.org) (140.211.11.28) by apache.org (qpsmtpd/0.29) with ESMTP; Tue, 13 Jan 2015 17:29:35 +0000 Date: Tue, 13 Jan 2015 17:29:35 +0000 (UTC) From: "Brandon Williams (JIRA)" To: commits@cassandra.apache.org Message-ID: In-Reply-To: References: Subject: [jira] [Resolved] (CASSANDRA-8599) Refactor or fix CqlStorage 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-8599?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ] Brandon Williams resolved CASSANDRA-8599. ----------------------------------------- Resolution: Fixed Reviewer: Brandon Williams Committed with minor cleanup, and I added an explicit logger.warn about the deprecation so users are more likely to see it. > Refactor or fix CqlStorage > -------------------------- > > Key: CASSANDRA-8599 > URL: https://issues.apache.org/jira/browse/CASSANDRA-8599 > Project: Cassandra > Issue Type: Bug > Components: Hadoop > Reporter: Brandon Williams > Assignee: Alex Liu > Fix For: 2.1.3 > > Attachments: 8599-2.1-branch.txt, 8599-v2-2.1-branch.txt > > > In CASSANDRA-8541, for 2.1, I ultimately replace the non-existent CPIF references with CIF, since CNS was broken otherwise. But this means CS no longer works since it's not a simple drop in replacement (but have CNS work is better than having them both broken by a class that doesn't exist.) We can't just deprecate and remove CS either, because CNS extends it. We either need to fix CS to work with CIF, or we need to refactor CNS so that we can just remove CS. -- This message was sent by Atlassian JIRA (v6.3.4#6332)