Return-Path: X-Original-To: apmail-nifi-commits-archive@minotaur.apache.org Delivered-To: apmail-nifi-commits-archive@minotaur.apache.org Received: from mail.apache.org (hermes.apache.org [140.211.11.3]) by minotaur.apache.org (Postfix) with SMTP id F2922182EA for ; Tue, 1 Sep 2015 19:01:45 +0000 (UTC) Received: (qmail 16043 invoked by uid 500); 1 Sep 2015 19:01:45 -0000 Delivered-To: apmail-nifi-commits-archive@nifi.apache.org Received: (qmail 16021 invoked by uid 500); 1 Sep 2015 19:01:45 -0000 Mailing-List: contact commits-help@nifi.apache.org; run by ezmlm Precedence: bulk List-Help: List-Unsubscribe: List-Post: List-Id: Reply-To: dev@nifi.apache.org Delivered-To: mailing list commits@nifi.apache.org Received: (qmail 16012 invoked by uid 99); 1 Sep 2015 19:01:45 -0000 Received: from arcas.apache.org (HELO arcas.apache.org) (140.211.11.28) by apache.org (qpsmtpd/0.29) with ESMTP; Tue, 01 Sep 2015 19:01:45 +0000 Date: Tue, 1 Sep 2015 19:01:45 +0000 (UTC) From: "ASF GitHub Bot (JIRA)" To: commits@nifi.apache.org Message-ID: In-Reply-To: References: Subject: [jira] [Commented] (NIFI-774) Create a DeleteS3Object Processor 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/NIFI-774?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=14725887#comment-14725887 ] ASF GitHub Bot commented on NIFI-774: ------------------------------------- Github user danbress commented on the pull request: https://github.com/apache/nifi/pull/80#issuecomment-136828624 Yu, this looks great. Thanks again for sticking with it and contributing. I made one comment inline. I have one other comment that I'd like to get others feedback on: This processor attempts to delete data from Amazon S3. I could see there being a case for three scenarios, and three relationships out of the processor 1) the data is found in S3, and deleted(success) 2) the data is not found in S3, and therefore not deleted(not found) 3) there was some issue communicating with S3 and the object could not be deleted(failure) Basically, you would be adding a "not found" relationship, and routing FlowFiles there when you they are not found in S3 before you attempt to delete them. The reason being, you allow the user of your processor the opportunity to treat "not found" situations differently from "failure" situations. I tried to find other cases like this in the NiFi processor marketplace, but I couldn't. (https://github.com/apache/nifi/blob/master/nifi-nar-bundles/nifi-aws-bundle/nifi-aws-processors/src/main/java/org/apache/nifi/processors/aws/sqs/DeleteSQS.java) is the only "delete" processor I see, and it doesn't behave the way I am describing. @markap14 @mcgilman @joewitt toughts > Create a DeleteS3Object Processor > --------------------------------- > > Key: NIFI-774 > URL: https://issues.apache.org/jira/browse/NIFI-774 > Project: Apache NiFi > Issue Type: Task > Components: Extensions > Reporter: Mark Payne > Assignee: Yu Ishikawa > -- This message was sent by Atlassian JIRA (v6.3.4#6332)