Mailing-List: contact commits-help@cassandra.apache.org; run by ezmlm
Precedence: bulk
Reply-To: dev@cassandra.apache.org
Date: Mon, 28 Sep 2015 19:09:04 +0000 (UTC)
From: "Paulo Motta (JIRA)" <jira@apache.org>
To: commits@cassandra.apache.org
Message-ID: <JIRA.12864328.1442342636000.93605.1443467344332@Atlassian.JIRA>
In-Reply-To: <JIRA.12864328.1442342636000@Atlassian.JIRA>
References: <JIRA.12864328.1442342636000@Atlassian.JIRA>
 <JIRA.12864328.1442342636424@arcas>
Subject: [jira] [Commented] (CASSANDRA-10347) Bulk Loader API could not
 tolerate even node failure
MIME-Version: 1.0
Content-Type: text/plain; charset=utf-8
Content-Transfer-Encoding: 7bit


    [ https://issues.apache.org/jira/browse/CASSANDRA-10347?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=14933818#comment-14933818 ] 

Paulo Motta commented on CASSANDRA-10347:
-----------------------------------------

bq. Isn't using mapreduce.output.bulkoutputformat.maxfailedhosts a better way to do this? Does that not work for this use case?

Probably yes, but [~wanshenghua] could tell better, did you try the {{mapreduce.output.bulkoutputformat.maxfailedhosts}} property? 

Unfortunately I just discovered that property after implementing the new one, my bad. Anyway, I guess the parameters are not mutually exclusive, as you may want still want to blacklist nodes that are alive. Since it's already implemented and to be consistent with sstable loader, I think it's still valid to have an {{ignorehosts}} property in addition to {{maxfailedhosts}}.

> Bulk Loader API could not tolerate even node failure
> ----------------------------------------------------
>
>                 Key: CASSANDRA-10347
>                 URL: https://issues.apache.org/jira/browse/CASSANDRA-10347
>             Project: Cassandra
>          Issue Type: Bug
>            Reporter: Shenghua Wan
>            Assignee: Paulo Motta
>             Fix For: 2.1.x, 2.2.x, 3.0.x
>
>
> When user uses CqlBulkOutputFormat, it tries to stream to all the nodes in the token range, which includes the dead nodes. Therefore, the stream failed. There was a design in C* API to allow stream() method to have a list of ignore hosts, but it was not utilized.
> The empty-argument stream() method is called in all existing versions of C*, i.e.
> in v2.0.11, https://github.com/apache/cassandra/blob/cassandra-2.0.11/src/java/org/apache/cassandra/hadoop/AbstractBulkRecordWriter.java#L122
> in v2.1.5, https://github.com/apache/cassandra/blob/cassandra-2.1.5/src/java/org/apache/cassandra/hadoop/AbstractBulkRecordWriter.java#L122
> and current trunk branch https://github.com/apache/cassandra/blob/trunk/src/java/org/apache/cassandra/hadoop/cql3/CqlBulkRecordWriter.java#L241


--
This message was sent by Atlassian JIRA
(v6.3.4#6332)