[jira] [Commented] (NIFI-1081) Add option to ExecuteStreamCommand to put value of execution to an attribute

2015-11-18 Thread ASF subversion and git services (JIRA)
[ https://issues.apache.org/jira/browse/NIFI-1081?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=15012765#comment-15012765 ] ASF subversion and git services commented on NIFI-1081: --- Commit 1e5cc

[jira] [Commented] (NIFI-1081) Add option to ExecuteStreamCommand to put value of execution to an attribute

2015-11-18 Thread Tony Kurc (JIRA)
[ https://issues.apache.org/jira/browse/NIFI-1081?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=15012752#comment-15012752 ] Tony Kurc commented on NIFI-1081: - Tests pass on on Windows 10 and Ubuntu Linux 14.04. Test

[jira] [Commented] (NIFI-1081) Add option to ExecuteStreamCommand to put value of execution to an attribute

2015-11-18 Thread Joseph Percivall (JIRA)
[ https://issues.apache.org/jira/browse/NIFI-1081?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=15012728#comment-15012728 ] Joseph Percivall commented on NIFI-1081: haha I'm still new to Apache land so I def

[jira] [Commented] (NIFI-1081) Add option to ExecuteStreamCommand to put value of execution to an attribute

2015-11-18 Thread Tony Kurc (JIRA)
[ https://issues.apache.org/jira/browse/NIFI-1081?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=15012723#comment-15012723 ] Tony Kurc commented on NIFI-1081: - that was a long way of saying "I erred on the side of gi

[jira] [Commented] (NIFI-1081) Add option to ExecuteStreamCommand to put value of execution to an attribute

2015-11-18 Thread Tony Kurc (JIRA)
[ https://issues.apache.org/jira/browse/NIFI-1081?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=15012721#comment-15012721 ] Tony Kurc commented on NIFI-1081: - I'm not sure of your concern, based on to the the apache

[jira] [Commented] (NIFI-1081) Add option to ExecuteStreamCommand to put value of execution to an attribute

2015-11-18 Thread Joseph Percivall (JIRA)
[ https://issues.apache.org/jira/browse/NIFI-1081?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=15012675#comment-15012675 ] Joseph Percivall commented on NIFI-1081: They match my intent and I did a build+con

[jira] [Commented] (NIFI-1081) Add option to ExecuteStreamCommand to put value of execution to an attribute

2015-11-18 Thread Tony Kurc (JIRA)
[ https://issues.apache.org/jira/browse/NIFI-1081?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=15012645#comment-15012645 ] Tony Kurc commented on NIFI-1081: - [~JPercivall] - I made some light amendments to the code

[jira] [Commented] (NIFI-1081) Add option to ExecuteStreamCommand to put value of execution to an attribute

2015-11-18 Thread Joseph Percivall (JIRA)
[ https://issues.apache.org/jira/browse/NIFI-1081?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=15012558#comment-15012558 ] Joseph Percivall commented on NIFI-1081: That works > Add option to ExecuteStreamC

[jira] [Commented] (NIFI-1081) Add option to ExecuteStreamCommand to put value of execution to an attribute

2015-11-18 Thread Tony Kurc (JIRA)
[ https://issues.apache.org/jira/browse/NIFI-1081?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=15012554#comment-15012554 ] Tony Kurc commented on NIFI-1081: - [~JPercivall] - since this bug is made no worse, but we

[jira] [Commented] (NIFI-1081) Add option to ExecuteStreamCommand to put value of execution to an attribute

2015-11-18 Thread Tony Kurc (JIRA)
[ https://issues.apache.org/jira/browse/NIFI-1081?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=15012496#comment-15012496 ] Tony Kurc commented on NIFI-1081: - working on the updated patch now. Going to test on windo

[jira] [Commented] (NIFI-1081) Add option to ExecuteStreamCommand to put value of execution to an attribute

2015-11-18 Thread Tony Kurc (JIRA)
[ https://issues.apache.org/jira/browse/NIFI-1081?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=15012360#comment-15012360 ] Tony Kurc commented on NIFI-1081: - I posed a question about the NOTICE file and the class n

[jira] [Commented] (NIFI-1081) Add option to ExecuteStreamCommand to put value of execution to an attribute

2015-11-18 Thread Tony Kurc (JIRA)
[ https://issues.apache.org/jira/browse/NIFI-1081?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=15011533#comment-15011533 ] Tony Kurc commented on NIFI-1081: - Will review and test this evening > Add option to Execu

[jira] [Commented] (NIFI-1081) Add option to ExecuteStreamCommand to put value of execution to an attribute

2015-11-18 Thread Joseph Percivall (JIRA)
[ https://issues.apache.org/jira/browse/NIFI-1081?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=15011372#comment-15011372 ] Joseph Percivall commented on NIFI-1081: You're right - fixing > Add option to Exe

[jira] [Commented] (NIFI-1081) Add option to ExecuteStreamCommand to put value of execution to an attribute

2015-11-14 Thread Tony Kurc (JIRA)
[ https://issues.apache.org/jira/browse/NIFI-1081?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=15005417#comment-15005417 ] Tony Kurc commented on NIFI-1081: - Let me retry to explain what I meant, I believe I had a

[jira] [Commented] (NIFI-1081) Add option to ExecuteStreamCommand to put value of execution to an attribute

2015-11-14 Thread Tony Kurc (JIRA)
[ https://issues.apache.org/jira/browse/NIFI-1081?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=15005414#comment-15005414 ] Tony Kurc commented on NIFI-1081: - Sorry, what I meant to say is "I believe this may be a b

[jira] [Commented] (NIFI-1081) Add option to ExecuteStreamCommand to put value of execution to an attribute

2015-11-14 Thread Tony Kurc (JIRA)
[ https://issues.apache.org/jira/browse/NIFI-1081?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=15005412#comment-15005412 ] Tony Kurc commented on NIFI-1081: - https://issues.apache.org/jira/browse/NIFI-1081?focusedC

[jira] [Commented] (NIFI-1081) Add option to ExecuteStreamCommand to put value of execution to an attribute

2015-11-13 Thread Bryan Bende (JIRA)
[ https://issues.apache.org/jira/browse/NIFI-1081?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=15004175#comment-15004175 ] Bryan Bende commented on NIFI-1081: --- Just adding my two cents here, I think it will be mu

[jira] [Commented] (NIFI-1081) Add option to ExecuteStreamCommand to put value of execution to an attribute

2015-11-13 Thread Joseph Witt (JIRA)
[ https://issues.apache.org/jira/browse/NIFI-1081?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=15004036#comment-15004036 ] Joseph Witt commented on NIFI-1081: --- i share mark's inclination here. However, not so mu

[jira] [Commented] (NIFI-1081) Add option to ExecuteStreamCommand to put value of execution to an attribute

2015-11-13 Thread Tony Kurc (JIRA)
[ https://issues.apache.org/jira/browse/NIFI-1081?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=15004016#comment-15004016 ] Tony Kurc commented on NIFI-1081: - In the case of this review, I had a _much_ harder time d

[jira] [Commented] (NIFI-1081) Add option to ExecuteStreamCommand to put value of execution to an attribute

2015-11-13 Thread Mark Payne (JIRA)
[ https://issues.apache.org/jira/browse/NIFI-1081?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=15003939#comment-15003939 ] Mark Payne commented on NIFI-1081: -- Hmmm... I find it very frustrating and confusing when

[jira] [Commented] (NIFI-1081) Add option to ExecuteStreamCommand to put value of execution to an attribute

2015-11-12 Thread Tony Kurc (JIRA)
[ https://issues.apache.org/jira/browse/NIFI-1081?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=15003504#comment-15003504 ] Tony Kurc commented on NIFI-1081: - Interesting perspective from Apache Yetus contributing g

[jira] [Commented] (NIFI-1081) Add option to ExecuteStreamCommand to put value of execution to an attribute

2015-11-12 Thread Joseph Witt (JIRA)
[ https://issues.apache.org/jira/browse/NIFI-1081?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=15003502#comment-15003502 ] Joseph Witt commented on NIFI-1081: --- another community member, Sean Busbey, requested the

[jira] [Commented] (NIFI-1081) Add option to ExecuteStreamCommand to put value of execution to an attribute

2015-11-12 Thread Tony Kurc (JIRA)
[ https://issues.apache.org/jira/browse/NIFI-1081?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=15003498#comment-15003498 ] Tony Kurc commented on NIFI-1081: - IMHO, keeping the old patches makes it much easier to te

[jira] [Commented] (NIFI-1081) Add option to ExecuteStreamCommand to put value of execution to an attribute

2015-11-12 Thread Joseph Percivall (JIRA)
[ https://issues.apache.org/jira/browse/NIFI-1081?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=15003495#comment-15003495 ] Joseph Percivall commented on NIFI-1081: I had always removed my old patches and pu

[jira] [Commented] (NIFI-1081) Add option to ExecuteStreamCommand to put value of execution to an attribute

2015-11-12 Thread Tony Kurc (JIRA)
[ https://issues.apache.org/jira/browse/NIFI-1081?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=15003485#comment-15003485 ] Tony Kurc commented on NIFI-1081: - Out of curiosity, is someone removing or overwriting the

[jira] [Commented] (NIFI-1081) Add option to ExecuteStreamCommand to put value of execution to an attribute

2015-11-12 Thread Joseph Percivall (JIRA)
[ https://issues.apache.org/jira/browse/NIFI-1081?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=15002586#comment-15002586 ] Joseph Percivall commented on NIFI-1081: This processor inherently relies on the sy

[jira] [Commented] (NIFI-1081) Add option to ExecuteStreamCommand to put value of execution to an attribute

2015-11-12 Thread Joseph Percivall (JIRA)
[ https://issues.apache.org/jira/browse/NIFI-1081?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=15002578#comment-15002578 ] Joseph Percivall commented on NIFI-1081: This I just copied from what was already t

[jira] [Commented] (NIFI-1081) Add option to ExecuteStreamCommand to put value of execution to an attribute

2015-11-12 Thread Joseph Percivall (JIRA)
[ https://issues.apache.org/jira/browse/NIFI-1081?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=15002571#comment-15002571 ] Joseph Percivall commented on NIFI-1081: What my thought process was that in the ev

[jira] [Commented] (NIFI-1081) Add option to ExecuteStreamCommand to put value of execution to an attribute

2015-11-12 Thread Joseph Percivall (JIRA)
[ https://issues.apache.org/jira/browse/NIFI-1081?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=15002557#comment-15002557 ] Joseph Percivall commented on NIFI-1081: yup, I messed this one up. I tried to get

[jira] [Commented] (NIFI-1081) Add option to ExecuteStreamCommand to put value of execution to an attribute

2015-11-11 Thread Tony Kurc (JIRA)
[ https://issues.apache.org/jira/browse/NIFI-1081?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=15001249#comment-15001249 ] Tony Kurc commented on NIFI-1081: - Can you walk me through the exception handling here? If

[jira] [Commented] (NIFI-1081) Add option to ExecuteStreamCommand to put value of execution to an attribute

2015-11-11 Thread Tony Kurc (JIRA)
[ https://issues.apache.org/jira/browse/NIFI-1081?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=15001234#comment-15001234 ] Tony Kurc commented on NIFI-1081: - I'm not sure if Stdout or StdOut us more appropriate (a

[jira] [Commented] (NIFI-1081) Add option to ExecuteStreamCommand to put value of execution to an attribute

2015-11-11 Thread Tony Kurc (JIRA)
[ https://issues.apache.org/jira/browse/NIFI-1081?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=15001217#comment-15001217 ] Tony Kurc commented on NIFI-1081: - if you're amending your code, I'd put a space after the

[jira] [Commented] (NIFI-1081) Add option to ExecuteStreamCommand to put value of execution to an attribute

2015-11-11 Thread Tony Kurc (JIRA)
[ https://issues.apache.org/jira/browse/NIFI-1081?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=15001213#comment-15001213 ] Tony Kurc commented on NIFI-1081: - another style comment, you have a mix of {}'s and string

[jira] [Commented] (NIFI-1081) Add option to ExecuteStreamCommand to put value of execution to an attribute

2015-11-11 Thread Tony Kurc (JIRA)
[ https://issues.apache.org/jira/browse/NIFI-1081?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=15001211#comment-15001211 ] Tony Kurc commented on NIFI-1081: - Based off your discussion with [~markap14] on a previous

[jira] [Commented] (NIFI-1081) Add option to ExecuteStreamCommand to put value of execution to an attribute

2015-11-11 Thread Tony Kurc (JIRA)
[ https://issues.apache.org/jira/browse/NIFI-1081?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=15001205#comment-15001205 ] Tony Kurc commented on NIFI-1081: - Slight style comment - your static final fields have a s

[jira] [Commented] (NIFI-1081) Add option to ExecuteStreamCommand to put value of execution to an attribute

2015-11-11 Thread Tony Kurc (JIRA)
[ https://issues.apache.org/jira/browse/NIFI-1081?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=15001005#comment-15001005 ] Tony Kurc commented on NIFI-1081: - [~JPercivall] - I highly recommend some comments on the

[jira] [Commented] (NIFI-1081) Add option to ExecuteStreamCommand to put value of execution to an attribute

2015-11-11 Thread Tony Kurc (JIRA)
[ https://issues.apache.org/jira/browse/NIFI-1081?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=15000978#comment-15000978 ] Tony Kurc commented on NIFI-1081: - I'll start reviewing the new patch > Add option to Exe

[jira] [Commented] (NIFI-1081) Add option to ExecuteStreamCommand to put value of execution to an attribute

2015-11-11 Thread Joseph Percivall (JIRA)
[ https://issues.apache.org/jira/browse/NIFI-1081?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=15000552#comment-15000552 ] Joseph Percivall commented on NIFI-1081: Those are valid points. I will make the ch

[jira] [Commented] (NIFI-1081) Add option to ExecuteStreamCommand to put value of execution to an attribute

2015-11-11 Thread Mark Payne (JIRA)
[ https://issues.apache.org/jira/browse/NIFI-1081?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=15000544#comment-15000544 ] Mark Payne commented on NIFI-1081: -- I don't believe that "Output Destination Attribute" ma

[jira] [Commented] (NIFI-1081) Add option to ExecuteStreamCommand to put value of execution to an attribute

2015-11-11 Thread Joseph Percivall (JIRA)
[ https://issues.apache.org/jira/browse/NIFI-1081?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=15000525#comment-15000525 ] Joseph Percivall commented on NIFI-1081: I was thinking something like "Output Dest

[jira] [Commented] (NIFI-1081) Add option to ExecuteStreamCommand to put value of execution to an attribute

2015-11-10 Thread Mark Payne (JIRA)
[ https://issues.apache.org/jira/browse/NIFI-1081?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=14999457#comment-14999457 ] Mark Payne commented on NIFI-1081: -- [~JPercivall] - The "Put Stream Output in Attribute" p

[jira] [Commented] (NIFI-1081) Add option to ExecuteStreamCommand to put value of execution to an attribute

2015-11-05 Thread Joseph Percivall (JIRA)
[ https://issues.apache.org/jira/browse/NIFI-1081?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=14992065#comment-14992065 ] Joseph Percivall commented on NIFI-1081: Ah, I need to mention that. It would be tr

[jira] [Commented] (NIFI-1081) Add option to ExecuteStreamCommand to put value of execution to an attribute

2015-11-05 Thread Mark Payne (JIRA)
[ https://issues.apache.org/jira/browse/NIFI-1081?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=14991997#comment-14991997 ] Mark Payne commented on NIFI-1081: -- [~JPercivall] - that's fine. If you want to refer to i

[jira] [Commented] (NIFI-1081) Add option to ExecuteStreamCommand to put value of execution to an attribute

2015-11-05 Thread Joseph Percivall (JIRA)
[ https://issues.apache.org/jira/browse/NIFI-1081?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=14991965#comment-14991965 ] Joseph Percivall commented on NIFI-1081: I agree with the first two comments but no

[jira] [Commented] (NIFI-1081) Add option to ExecuteStreamCommand to put value of execution to an attribute

2015-11-05 Thread Mark Payne (JIRA)
[ https://issues.apache.org/jira/browse/NIFI-1081?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=14991882#comment-14991882 ] Mark Payne commented on NIFI-1081: -- [~JPercivall] - Reviewing code for this patch. I have

[jira] [Commented] (NIFI-1081) Add option to ExecuteStreamCommand to put value of execution to an attribute

2015-11-03 Thread Charlie Frasure (JIRA)
[ https://issues.apache.org/jira/browse/NIFI-1081?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=14988818#comment-14988818 ] Charlie Frasure commented on NIFI-1081: --- Thanks for the troubleshooting tip. This la

[jira] [Commented] (NIFI-1081) Add option to ExecuteStreamCommand to put value of execution to an attribute

2015-11-03 Thread Charlie Frasure (JIRA)
[ https://issues.apache.org/jira/browse/NIFI-1081?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=14988643#comment-14988643 ] Charlie Frasure commented on NIFI-1081: --- Should this patch be working with the latest

[jira] [Commented] (NIFI-1081) Add option to ExecuteStreamCommand to put value of execution to an attribute

2015-10-29 Thread Joseph Percivall (JIRA)
[ https://issues.apache.org/jira/browse/NIFI-1081?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=14981153#comment-14981153 ] Joseph Percivall commented on NIFI-1081: [~charliefrasure], I have a feeling I know

[jira] [Commented] (NIFI-1081) Add option to ExecuteStreamCommand to put value of execution to an attribute

2015-10-29 Thread Joseph Percivall (JIRA)
[ https://issues.apache.org/jira/browse/NIFI-1081?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=14980980#comment-14980980 ] Joseph Percivall commented on NIFI-1081: Hmm well that's odd. Let me see if I can r

[jira] [Commented] (NIFI-1081) Add option to ExecuteStreamCommand to put value of execution to an attribute

2015-10-29 Thread Charlie Frasure (JIRA)
[ https://issues.apache.org/jira/browse/NIFI-1081?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=14980826#comment-14980826 ] Charlie Frasure commented on NIFI-1081: --- My properties of ExecuteStreamCommand are as

[jira] [Commented] (NIFI-1081) Add option to ExecuteStreamCommand to put value of execution to an attribute

2015-10-29 Thread Charlie Frasure (JIRA)
[ https://issues.apache.org/jira/browse/NIFI-1081?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=14980564#comment-14980564 ] Charlie Frasure commented on NIFI-1081: --- Joe, This patch compiled and executes withou