Return-Path: X-Original-To: apmail-spark-reviews-archive@minotaur.apache.org Delivered-To: apmail-spark-reviews-archive@minotaur.apache.org Received: from mail.apache.org (hermes.apache.org [140.211.11.3]) by minotaur.apache.org (Postfix) with SMTP id B8C6510793 for ; Fri, 25 Jul 2014 18:07:38 +0000 (UTC) Received: (qmail 77545 invoked by uid 500); 25 Jul 2014 18:07:38 -0000 Delivered-To: apmail-spark-reviews-archive@spark.apache.org Received: (qmail 77521 invoked by uid 500); 25 Jul 2014 18:07:38 -0000 Mailing-List: contact reviews-help@spark.apache.org; run by ezmlm Precedence: bulk List-Help: List-Unsubscribe: List-Post: List-Id: Reply-To: reviews@spark.apache.org Delivered-To: mailing list reviews@spark.apache.org Received: (qmail 77510 invoked by uid 99); 25 Jul 2014 18:07:38 -0000 Received: from tyr.zones.apache.org (HELO tyr.zones.apache.org) (140.211.11.114) by apache.org (qpsmtpd/0.29) with ESMTP; Fri, 25 Jul 2014 18:07:38 +0000 Received: by tyr.zones.apache.org (Postfix, from userid 65534) id 001CB9B5414; Fri, 25 Jul 2014 18:07:37 +0000 (UTC) From: mateiz To: reviews@spark.apache.org Reply-To: reviews@spark.apache.org References: In-Reply-To: Subject: [GitHub] spark pull request: Part of [SPARK-2456] Removed some HashMaps fro... Content-Type: text/plain Message-Id: <20140725180738.001CB9B5414@tyr.zones.apache.org> Date: Fri, 25 Jul 2014 18:07:37 +0000 (UTC) Github user mateiz commented on the pull request: https://github.com/apache/spark/pull/1561#issuecomment-50184213 Actually I made one note on clearing pendingTasks. Probably best to keep that code. I think it might come in if a stage is resubmitted. --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub as well. If your project does not have this feature enabled and wishes so, or if the feature is enabled but not working, please contact infrastructure at infrastructure@apache.org or file a JIRA ticket with INFRA. ---