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

Aldrin Piri commented on NIFI-673:
----------------------------------

Reviewed this again.  This patch sequence certainly made much more sense and 
looks good.  Build and contrib all passed.

+1 overall.

I would like to see actual user name of the account that was used to list the 
files (the username property of the processor configuration) from ListSFTP make 
it into the attributes.  We currently have file owner in terms of ID, so the 
actual name seems to make sense as well.  With that change, we could 
additionally have FetchSFTP make its username property support Expression 
Language and give a bit more flexibility in that retrieval process.  Understood 
that password likely isn't feasible, but, with appropriate system and key 
configuration, there is some nice utility that is provided.  Thoughts?

> Create ListSFTP, FetchSFTP Processors
> -------------------------------------
>
>                 Key: NIFI-673
>                 URL: https://issues.apache.org/jira/browse/NIFI-673
>             Project: Apache NiFi
>          Issue Type: New Feature
>          Components: Extensions
>            Reporter: Mark Payne
>            Assignee: Mark Payne
>             Fix For: 0.4.0
>
>         Attachments: 
> 0001-NIFI-673-Initial-implementation-of-ListSFTP-FetchSFT.patch, 
> 0002-NIFI-673-Added-Completion-Strategy-to-FetchSFTP.patch
>
>
> This will allow us to pull a listing from a single primary node and then 
> distribute the work of pulling and processing the data across the cluster.



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

Reply via email to