Return-Path: X-Original-To: apmail-hadoop-mapreduce-dev-archive@minotaur.apache.org Delivered-To: apmail-hadoop-mapreduce-dev-archive@minotaur.apache.org Received: from mail.apache.org (hermes.apache.org [140.211.11.3]) by minotaur.apache.org (Postfix) with SMTP id 6B19CD8BF for ; Sun, 2 Dec 2012 19:16:00 +0000 (UTC) Received: (qmail 30867 invoked by uid 500); 2 Dec 2012 19:15:59 -0000 Delivered-To: apmail-hadoop-mapreduce-dev-archive@hadoop.apache.org Received: (qmail 30707 invoked by uid 500); 2 Dec 2012 19:15:59 -0000 Mailing-List: contact mapreduce-dev-help@hadoop.apache.org; run by ezmlm Precedence: bulk List-Help: List-Unsubscribe: List-Post: List-Id: Reply-To: mapreduce-dev@hadoop.apache.org Delivered-To: mailing list mapreduce-dev@hadoop.apache.org Received: (qmail 30315 invoked by uid 99); 2 Dec 2012 19:15:58 -0000 Received: from arcas.apache.org (HELO arcas.apache.org) (140.211.11.28) by apache.org (qpsmtpd/0.29) with ESMTP; Sun, 02 Dec 2012 19:15:58 +0000 Date: Sun, 2 Dec 2012 19:15:58 +0000 (UTC) From: "Mostafa Elhemali (JIRA)" To: mapreduce-dev@hadoop.apache.org Message-ID: <512160914.51106.1354475758638.JavaMail.jiratomcat@arcas> Subject: [jira] [Created] (MAPREDUCE-4840) Delete dead code and deprecate public API related to skipping bad records MIME-Version: 1.0 Content-Type: text/plain; charset=utf-8 Content-Transfer-Encoding: 7bit X-JIRA-FingerPrint: 30527f35849b9dde25b450d4833f0394 Mostafa Elhemali created MAPREDUCE-4840: ------------------------------------------- Summary: Delete dead code and deprecate public API related to skipping bad records Key: MAPREDUCE-4840 URL: https://issues.apache.org/jira/browse/MAPREDUCE-4840 Project: Hadoop Map/Reduce Issue Type: Bug Affects Versions: trunk Reporter: Mostafa Elhemali Priority: Minor It looks like the decision was made in MAPREDUCE-1932 to remove support for skipping bad records rather than fix it (it doesn't work right now in trunk). If that's the case then we should probably delete all the dead code related to it and deprecate the public API's for it right? Dead code I'm talking about: 1. Task class: skipping, skipRanges, writeSkipRecs 2. MapTask class: SkippingRecordReader inner class 3. ReduceTask class: SkippingReduceValuesIterator inner class 4. Tests: TestBadRecords Public API: 1. SkipBadRecords class -- This message is automatically generated by JIRA. If you think it was sent incorrectly, please contact your JIRA administrators For more information on JIRA, see: http://www.atlassian.com/software/jira