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 553F917E33 for ; Mon, 12 Jan 2015 20:59:35 +0000 (UTC) Received: (qmail 32392 invoked by uid 500); 12 Jan 2015 20:59:36 -0000 Delivered-To: apmail-cassandra-commits-archive@cassandra.apache.org Received: (qmail 32360 invoked by uid 500); 12 Jan 2015 20:59:36 -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 32347 invoked by uid 99); 12 Jan 2015 20:59:36 -0000 Received: from arcas.apache.org (HELO arcas.apache.org) (140.211.11.28) by apache.org (qpsmtpd/0.29) with ESMTP; Mon, 12 Jan 2015 20:59:36 +0000 Date: Mon, 12 Jan 2015 20:59:36 +0000 (UTC) From: "Brandon Williams (JIRA)" To: commits@cassandra.apache.org Message-ID: In-Reply-To: References: Subject: [jira] [Commented] (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:comment-tabpanel&focusedCommentId=14274163#comment-14274163 ] Brandon Williams commented on CASSANDRA-8599: --------------------------------------------- That's fine, but I don't want to remove something in a minor, so let's make CS wrap CNS and emit a deprecation warning, then we'll remove it in 3.0. > 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 > > > 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)