[GitHub] nifi issue #2056: NIFI-4262 - MergeContent - option to add merged uuid in or...

2017-08-12 Thread pvillard31
Github user pvillard31 commented on the issue: https://github.com/apache/nifi/pull/2056 Done, thanks @mattyb149 ! --- 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 wish

[jira] [Commented] (NIFI-4262) MergeContent - option to add merged uuid in original flow files

2017-08-12 Thread ASF GitHub Bot (JIRA)
[ https://issues.apache.org/jira/browse/NIFI-4262?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=16124508#comment-16124508 ] ASF GitHub Bot commented on NIFI-4262: -- Github user pvillard31 commented on the issue:

[GitHub] nifi issue #2077: NIFI-4237 Improve error messaging on encryption failures

2017-08-12 Thread alopresto
Github user alopresto commented on the issue: https://github.com/apache/nifi/pull/2077 I have a *number* of questions about the existing implementation, but those will be addressed in NIFI-3116. --- If your project is set up for it, you can reply to this email and have your reply ap

[jira] [Commented] (NIFI-4237) EncryptionOperationNotPossibleException in nifi-bootstrap.log might suggest underlying cause

2017-08-12 Thread ASF GitHub Bot (JIRA)
[ https://issues.apache.org/jira/browse/NIFI-4237?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=16124552#comment-16124552 ] ASF GitHub Bot commented on NIFI-4237: -- Github user alopresto commented on the issue:

[GitHub] nifi pull request #2065: NIFI-4255 - added flag to allow migration of existi...

2017-08-12 Thread jtstorck
Github user jtstorck commented on a diff in the pull request: https://github.com/apache/nifi/pull/2065#discussion_r132818810 --- Diff: nifi-toolkit/nifi-toolkit-zookeeper-migrator/src/main/java/org/apache/nifi/toolkit/zkmigrator/ZooKeeperMigrator.java --- @@ -269,15 +269,23 @@ pri

[jira] [Commented] (NIFI-4255) Add support for providing ACLs for paths in Zookeeper Migration tool

2017-08-12 Thread ASF GitHub Bot (JIRA)
[ https://issues.apache.org/jira/browse/NIFI-4255?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=16124582#comment-16124582 ] ASF GitHub Bot commented on NIFI-4255: -- Github user jtstorck commented on a diff in th

[jira] [Updated] (NIFI-4062) Provide an option to disable DTD validation for EvaluateXPath and EvaluateXQuery

2017-08-12 Thread Arun Manivannan (JIRA)
[ https://issues.apache.org/jira/browse/NIFI-4062?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ] Arun Manivannan updated NIFI-4062: -- Fix Version/s: 1.4.0 Status: Patch Available (was: In Progress) > Provide an option t

[jira] [Updated] (NIFI-4062) Provide an option to disable DTD validation for EvaluateXPath and EvaluateXQuery

2017-08-12 Thread Joseph Witt (JIRA)
[ https://issues.apache.org/jira/browse/NIFI-4062?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ] Joseph Witt updated NIFI-4062: -- Fix Version/s: (was: 1.4.0) > Provide an option to disable DTD validation for EvaluateXPath and > Ev

[jira] [Commented] (NIFI-4062) Provide an option to disable DTD validation for EvaluateXPath and EvaluateXQuery

2017-08-12 Thread Joseph Witt (JIRA)
[ https://issues.apache.org/jira/browse/NIFI-4062?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=16124590#comment-16124590 ] Joseph Witt commented on NIFI-4062: --- Fix version can be applied when we have review tract

[jira] [Commented] (NIFI-4062) Provide an option to disable DTD validation for EvaluateXPath and EvaluateXQuery

2017-08-12 Thread Arun Manivannan (JIRA)
[ https://issues.apache.org/jira/browse/NIFI-4062?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=16124595#comment-16124595 ] Arun Manivannan commented on NIFI-4062: --- [~joewitt] Sorry about that. Will follow th

[jira] [Commented] (NIFI-4062) Provide an option to disable DTD validation for EvaluateXPath and EvaluateXQuery

2017-08-12 Thread Joseph Witt (JIRA)
[ https://issues.apache.org/jira/browse/NIFI-4062?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=16124626#comment-16124626 ] Joseph Witt commented on NIFI-4062: --- no problem at all [~arunodhaya80] you did just fine

[GitHub] nifi pull request #2065: NIFI-4255 - added flag to allow migration of existi...

2017-08-12 Thread YolandaMDavis
Github user YolandaMDavis commented on a diff in the pull request: https://github.com/apache/nifi/pull/2065#discussion_r132821753 --- Diff: nifi-toolkit/nifi-toolkit-zookeeper-migrator/src/main/java/org/apache/nifi/toolkit/zkmigrator/ZooKeeperMigrator.java --- @@ -269,15 +269,23 @

[jira] [Commented] (NIFI-4255) Add support for providing ACLs for paths in Zookeeper Migration tool

2017-08-12 Thread ASF GitHub Bot (JIRA)
[ https://issues.apache.org/jira/browse/NIFI-4255?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=16124635#comment-16124635 ] ASF GitHub Bot commented on NIFI-4255: -- Github user YolandaMDavis commented on a diff

[GitHub] nifi issue #2045: NIFI-4250 - Elasticsearch 5 delete processor

2017-08-12 Thread mattyb149
Github user mattyb149 commented on the issue: https://github.com/apache/nifi/pull/2045 Don't worry about the ANTLR stuff, that seems to trip up Travis sometimes. I can review this either this weekend or early next week. One quick question is whether you think it would be too confusing

[jira] [Commented] (NIFI-4250) Create support for deleting document by id from elasticsearch 5

2017-08-12 Thread ASF GitHub Bot (JIRA)
[ https://issues.apache.org/jira/browse/NIFI-4250?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=16124703#comment-16124703 ] ASF GitHub Bot commented on NIFI-4250: -- Github user mattyb149 commented on the issue:

[GitHub] nifi issue #2045: NIFI-4250 - Elasticsearch 5 delete processor

2017-08-12 Thread mans2singh
Github user mans2singh commented on the issue: https://github.com/apache/nifi/pull/2045 @mattyb149 Regarding having a separate delete processor - I prefer single responsibility principle since it avoids confusion and mixing up configuration, processing logic, error handling,

[jira] [Commented] (NIFI-4250) Create support for deleting document by id from elasticsearch 5

2017-08-12 Thread ASF GitHub Bot (JIRA)
[ https://issues.apache.org/jira/browse/NIFI-4250?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=16124743#comment-16124743 ] ASF GitHub Bot commented on NIFI-4250: -- Github user mans2singh commented on the issue:

[GitHub] nifi issue #2045: NIFI-4250 - Elasticsearch 5 delete processor

2017-08-12 Thread mattyb149
Github user mattyb149 commented on the issue: https://github.com/apache/nifi/pull/2045 Makes sense to me, plus once the Extension Registry is in place, that might be the right spot for additional delete ES processors etc. I will review this for inclusion, thanks in advance! --- If y

[jira] [Commented] (NIFI-4250) Create support for deleting document by id from elasticsearch 5

2017-08-12 Thread ASF GitHub Bot (JIRA)
[ https://issues.apache.org/jira/browse/NIFI-4250?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=16124757#comment-16124757 ] ASF GitHub Bot commented on NIFI-4250: -- Github user mattyb149 commented on the issue: