[GitHub] nifi pull request #502: Nifi-1972 Apache Ignite Put Cache Processor

2016-06-09 Thread mans2singh
Github user mans2singh commented on a diff in the pull request: https://github.com/apache/nifi/pull/502#discussion_r66547015 --- Diff: nifi-nar-bundles/nifi-ignite-bundle/nifi-ignite-processors/src/main/resources/ignite-client.xml --- @@ -0,0 +1,26 @@ + + +http://www.s

[GitHub] nifi pull request #502: Nifi-1972 Apache Ignite Put Cache Processor

2016-06-09 Thread mans2singh
Github user mans2singh commented on a diff in the pull request: https://github.com/apache/nifi/pull/502#discussion_r66546632 --- Diff: nifi-nar-bundles/nifi-ignite-bundle/nifi-ignite-processors/src/main/java/org/apache/nifi/processors/ignite/AbstractIgniteProcessor.java --- @@ -0,

[GitHub] nifi pull request #502: Nifi-1972 Apache Ignite Put Cache Processor

2016-06-09 Thread mans2singh
Github user mans2singh commented on a diff in the pull request: https://github.com/apache/nifi/pull/502#discussion_r66546466 --- Diff: nifi-nar-bundles/nifi-ignite-bundle/nifi-ignite-processors/src/main/java/org/apache/nifi/processors/ignite/cache/PutIgniteCache.java --- @@ -0,0 +

[GitHub] nifi pull request #502: Nifi-1972 Apache Ignite Put Cache Processor

2016-06-09 Thread mans2singh
Github user mans2singh commented on a diff in the pull request: https://github.com/apache/nifi/pull/502#discussion_r66546338 --- Diff: nifi-nar-bundles/nifi-ignite-bundle/nifi-ignite-processors/src/main/java/org/apache/nifi/processors/ignite/cache/PutIgniteCache.java --- @@ -0,0 +

[GitHub] nifi pull request #517: NIFI-1994: Fixed issues with controller services and...

2016-06-09 Thread markap14
GitHub user markap14 opened a pull request: https://github.com/apache/nifi/pull/517 NIFI-1994: Fixed issues with controller services and templates Fixed issue with Controller Service Fully Qualified Class Names and ensure that services are added to the process groups as appropriate

[GitHub] nifi pull request #502: Nifi-1972 Apache Ignite Put Cache Processor

2016-06-09 Thread mans2singh
Github user mans2singh commented on a diff in the pull request: https://github.com/apache/nifi/pull/502#discussion_r66546103 --- Diff: nifi-nar-bundles/nifi-ignite-bundle/nifi-ignite-processors/src/main/java/org/apache/nifi/processors/ignite/cache/AbstractIgniteCacheProcessor.java ---

[GitHub] nifi issue #475: - Add Maven profile to compile nifi-hadoop-libraries-nar us...

2016-06-09 Thread mattyb149
Github user mattyb149 commented on the issue: https://github.com/apache/nifi/pull/475 As an optional Maven build profile I would imagine it's ok but will defer to @joewitt and others. In the meantime I will review and test this profile (and its absence) against various Hadoop distros.

[GitHub] nifi pull request #502: Nifi-1972 Apache Ignite Put Cache Processor

2016-06-09 Thread mans2singh
Github user mans2singh commented on a diff in the pull request: https://github.com/apache/nifi/pull/502#discussion_r66545282 --- Diff: nifi-nar-bundles/nifi-ignite-bundle/nifi-ignite-processors/src/main/java/org/apache/nifi/processors/ignite/cache/PutIgniteCache.java --- @@ -0,0 +

[GitHub] nifi pull request #502: Nifi-1972 Apache Ignite Put Cache Processor

2016-06-09 Thread mans2singh
Github user mans2singh commented on a diff in the pull request: https://github.com/apache/nifi/pull/502#discussion_r66545243 --- Diff: nifi-nar-bundles/nifi-ignite-bundle/nifi-ignite-processors/src/main/java/org/apache/nifi/processors/ignite/cache/PutIgniteCache.java --- @@ -0,0 +

[GitHub] nifi pull request #502: Nifi-1972 Apache Ignite Put Cache Processor

2016-06-09 Thread mans2singh
Github user mans2singh commented on a diff in the pull request: https://github.com/apache/nifi/pull/502#discussion_r66545042 --- Diff: nifi-nar-bundles/nifi-ignite-bundle/nifi-ignite-processors/src/main/java/org/apache/nifi/processors/ignite/cache/PutIgniteCache.java --- @@ -0,0 +

[GitHub] nifi pull request #502: Nifi-1972 Apache Ignite Put Cache Processor

2016-06-09 Thread mans2singh
Github user mans2singh commented on a diff in the pull request: https://github.com/apache/nifi/pull/502#discussion_r66544841 --- Diff: nifi-nar-bundles/nifi-ignite-bundle/nifi-ignite-processors/src/main/java/org/apache/nifi/processors/ignite/cache/PutIgniteCache.java --- @@ -0,0 +

[GitHub] nifi issue #475: - Add Maven profile to compile nifi-hadoop-libraries-nar us...

2016-06-09 Thread trixpan
Github user trixpan commented on the issue: https://github.com/apache/nifi/pull/475 @joewitt would you know if this is something that can be merged or if ASF/License prevents it from happening? Happy to add profiles to HDP and CDH as well if needed. --- If your project is se

[GitHub] nifi issue #516: NIFI-1993 upgraded CGLIB to 3.2.2

2016-06-09 Thread olegz
Github user olegz commented on the issue: https://github.com/apache/nifi/pull/516 @mattyb149 I got one better ;) Try PR https://github.com/apache/nifi/pull/515 as it stands now and then with this change. That is how it was discovered. I've attached some notes in JIRA. Let me know if

[GitHub] nifi issue #516: NIFI-1993 upgraded CGLIB to 3.2.2

2016-06-09 Thread mattyb149
Github user mattyb149 commented on the issue: https://github.com/apache/nifi/pull/516 What's a good test? Successful build? Or something more? --- 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

[GitHub] nifi issue #516: NIFI-1993 upgraded CGLIB to 3.2.2

2016-06-09 Thread olegz
Github user olegz commented on the issue: https://github.com/apache/nifi/pull/516 NOTE to merger: This is 1.0.0 only since the reasons for the upgrade are related to Java 1.8 --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub as wel

[GitHub] nifi pull request #516: NIFI-1993 upgraded CGLIB to 3.2.2

2016-06-09 Thread olegz
GitHub user olegz opened a pull request: https://github.com/apache/nifi/pull/516 NIFI-1993 upgraded CGLIB to 3.2.2 You can merge this pull request into a Git repository by running: $ git pull https://github.com/olegz/nifi NIFI-1993 Alternatively you can review and apply these

[GitHub] nifi pull request #502: Nifi-1972 Apache Ignite Put Cache Processor

2016-06-09 Thread mans2singh
Github user mans2singh commented on a diff in the pull request: https://github.com/apache/nifi/pull/502#discussion_r66531971 --- Diff: nifi-nar-bundles/nifi-ignite-bundle/nifi-ignite-processors/src/main/java/org/apache/nifi/processors/ignite/AbstractIgniteProcessor.java --- @@ -0,

Re: Consuming web services through NiFi

2016-06-09 Thread Matt Burgess
Are these http sites or HTTPS? If HTTPS you'll likely need an SSLContextService configured. > On Jun 9, 2016, at 4:34 PM, saikrishnat wrote: > > i am running on my laptop. yaa thats what confuses me. i could able to > download files when i used GetFTP process from a public site. > its only Get

[GitHub] nifi issue #497: NIFI-1857: HTTPS Site-to-Site

2016-06-09 Thread ijokarumawak
Github user ijokarumawak commented on the issue: https://github.com/apache/nifi/pull/497 @markap14 Thanks for taking your time to review and test, I'm so glad to hear that worked! The CI test haven't been finished, maybe that's why this isn't closed yet. I'll check the status agai

Re: Consuming web services through NiFi

2016-06-09 Thread saikrishnat
i am running on my laptop. yaa thats what confuses me. i could able to download files when i used GetFTP process from a public site. its only GetHTTP or InvokeHTTP is giving me problems. its acting like it cant go out of the network when i try thru NiFi. if firewall blocks it , it should block it w

Re: Consuming web services through NiFi

2016-06-09 Thread Matt Burgess
Do you have NiFi running locally or are you connecting over the browser to an instance on a different node? If you're on the same node then I don't know why it won't connect, if it's on a different node or VM there might be a connectivity problem between that node and the Internet. Regards, Mat

Re: Consuming web services through NiFi

2016-06-09 Thread saikrishnat
Hi Matt, i tried with google.com and cnn to get rss feed as shown in one of the examples online by someone.. thing is i can go to the site when i use the browser directly with the same URL..but not thru NiFi.. do i have to change any settings.?? Regards,Sai On Thu, Jun 9, 2016 at 3:02 PM, Matt B

[GitHub] nifi pull request #509: NIFI-1982: Use Compressed check box value.

2016-06-09 Thread ijokarumawak
Github user ijokarumawak closed the pull request at: https://github.com/apache/nifi/pull/509 --- 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

Re: Consuming web services through NiFi

2016-06-09 Thread Matt Burgess
Which other sites did you try? I've noticed that the randomuser.me API is often unavailable and gives those SocketTimeout messages. Also perhaps try the InvokeHttp processor. I would imagine any site should work in both GetHttp and InvokeHttp, but I'm curious to see if there is a difference. What

[GitHub] nifi pull request #502: Nifi-1972 Apache Ignite Put Cache Processor

2016-06-09 Thread pvillard31
Github user pvillard31 commented on a diff in the pull request: https://github.com/apache/nifi/pull/502#discussion_r66516313 --- Diff: nifi-nar-bundles/nifi-ignite-bundle/nifi-ignite-processors/src/main/resources/ignite-client.xml --- @@ -0,0 +1,26 @@ + + +http://www.s

[GitHub] nifi pull request #502: Nifi-1972 Apache Ignite Put Cache Processor

2016-06-09 Thread pvillard31
Github user pvillard31 commented on a diff in the pull request: https://github.com/apache/nifi/pull/502#discussion_r66515866 --- Diff: nifi-nar-bundles/nifi-ignite-bundle/nifi-ignite-processors/src/main/java/org/apache/nifi/processors/ignite/AbstractIgniteProcessor.java --- @@ -0,

[GitHub] nifi pull request #502: Nifi-1972 Apache Ignite Put Cache Processor

2016-06-09 Thread pvillard31
Github user pvillard31 commented on a diff in the pull request: https://github.com/apache/nifi/pull/502#discussion_r66514230 --- Diff: nifi-nar-bundles/nifi-ignite-bundle/nifi-ignite-processors/src/main/java/org/apache/nifi/processors/ignite/cache/AbstractIgniteCacheProcessor.java ---

[GitHub] nifi pull request #502: Nifi-1972 Apache Ignite Put Cache Processor

2016-06-09 Thread pvillard31
Github user pvillard31 commented on a diff in the pull request: https://github.com/apache/nifi/pull/502#discussion_r66514855 --- Diff: nifi-nar-bundles/nifi-ignite-bundle/nifi-ignite-processors/src/main/java/org/apache/nifi/processors/ignite/cache/PutIgniteCache.java --- @@ -0,0 +

[GitHub] nifi pull request #502: Nifi-1972 Apache Ignite Put Cache Processor

2016-06-09 Thread pvillard31
Github user pvillard31 commented on a diff in the pull request: https://github.com/apache/nifi/pull/502#discussion_r66514607 --- Diff: nifi-nar-bundles/nifi-ignite-bundle/nifi-ignite-processors/src/main/java/org/apache/nifi/processors/ignite/cache/PutIgniteCache.java --- @@ -0,0 +

[GitHub] nifi pull request #502: Nifi-1972 Apache Ignite Put Cache Processor

2016-06-09 Thread pvillard31
Github user pvillard31 commented on a diff in the pull request: https://github.com/apache/nifi/pull/502#discussion_r66513705 --- Diff: nifi-nar-bundles/nifi-ignite-bundle/nifi-ignite-processors/src/main/java/org/apache/nifi/processors/ignite/cache/PutIgniteCache.java --- @@ -0,0 +

[GitHub] nifi pull request #502: Nifi-1972 Apache Ignite Put Cache Processor

2016-06-09 Thread pvillard31
Github user pvillard31 commented on a diff in the pull request: https://github.com/apache/nifi/pull/502#discussion_r66513495 --- Diff: nifi-nar-bundles/nifi-ignite-bundle/nifi-ignite-processors/src/main/java/org/apache/nifi/processors/ignite/cache/PutIgniteCache.java --- @@ -0,0 +

[GitHub] nifi pull request #502: Nifi-1972 Apache Ignite Put Cache Processor

2016-06-09 Thread pvillard31
Github user pvillard31 commented on a diff in the pull request: https://github.com/apache/nifi/pull/502#discussion_r66513208 --- Diff: nifi-nar-bundles/nifi-ignite-bundle/nifi-ignite-processors/src/main/java/org/apache/nifi/processors/ignite/cache/PutIgniteCache.java --- @@ -0,0 +

[GitHub] nifi pull request #502: Nifi-1972 Apache Ignite Put Cache Processor

2016-06-09 Thread pvillard31
Github user pvillard31 commented on a diff in the pull request: https://github.com/apache/nifi/pull/502#discussion_r66512938 --- Diff: nifi-nar-bundles/nifi-ignite-bundle/nifi-ignite-processors/src/main/java/org/apache/nifi/processors/ignite/cache/PutIgniteCache.java --- @@ -0,0 +

[GitHub] nifi issue #497: NIFI-1857: HTTPS Site-to-Site

2016-06-09 Thread markap14
Github user markap14 commented on the issue: https://github.com/apache/nifi/pull/497 @ijokarumawak I added the "This closes #497" message to the commit that I pushed, but it doesn't seem to have worked... can you close the PR? --- If your project is set up for it, you can reply to th

[GitHub] nifi pull request #502: Nifi-1972 Apache Ignite Put Cache Processor

2016-06-09 Thread pvillard31
Github user pvillard31 commented on a diff in the pull request: https://github.com/apache/nifi/pull/502#discussion_r66510540 --- Diff: nifi-nar-bundles/nifi-ignite-bundle/nifi-ignite-processors/src/main/java/org/apache/nifi/processors/ignite/AbstractIgniteProcessor.java --- @@ -0,

[GitHub] nifi issue #502: Nifi-1972 Apache Ignite Put Cache Processor

2016-06-09 Thread pvillard31
Github user pvillard31 commented on the issue: https://github.com/apache/nifi/pull/502 @mans2singh Could you check all the lines marked as removed? I believe there should not be any and this is probably because of some settings in your IDE. --- If your project is set up for it, you

[GitHub] nifi issue #497: NIFI-1857: HTTPS Site-to-Site

2016-06-09 Thread markap14
Github user markap14 commented on the issue: https://github.com/apache/nifi/pull/497 @ijokarumawak this is looking good now! I have pulled down the latest PR, rebased against master, and have been able to test this running directly against my NiFi instance and while using nginx as a p

[GitHub] nifi issue #513: PutHBaseJSON processor treats all values as Strings

2016-06-09 Thread rtempleton
Github user rtempleton commented on the issue: https://github.com/apache/nifi/pull/513 I like that idea. --- 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

[GitHub] nifi issue #513: PutHBaseJSON processor treats all values as Strings

2016-06-09 Thread bbende
Github user bbende commented on the issue: https://github.com/apache/nifi/pull/513 Thats a great point about leveraging any changes they make to the Bytes class... I'm now thinking, what if we added simple methods to the HBaseClientService interface [1] that wrapped the calls to Bytes

[GitHub] nifi issue #513: PutHBaseJSON processor treats all values as Strings

2016-06-09 Thread rtempleton
Github user rtempleton commented on the issue: https://github.com/apache/nifi/pull/513 Yes, You're exactly right. I looked at the code in the Bytes util class and thought about doing that but then just imported the dependency. I expect that HBase users will use this Bytes class to dec

[GitHub] nifi issue #515: NIFI-826 [REVIEW ONLY] Initial commit for deterministic tem...

2016-06-09 Thread olegz
Github user olegz commented on the issue: https://github.com/apache/nifi/pull/515 @mcgilman so, this is the initial commit that essentially demonstrates the approach that is discussed in JIRA. Basically the new ID _ inceptionId_ is generated and is immutable and perpetual. With such c

NiFi Script Tester v1.1.1 available

2016-06-09 Thread Matt Burgess
All, Just wanted to let you know that my NiFi Script Tester has been updated, turned out the original version didn't work so well :-P I've fixed the bugs I found and added some helpful things (such as Apache Commons IO to the fat JAR, for IOUtils.toString()) and the "-module" option to add module

[GitHub] nifi pull request #515: NIFI-826 [REVIEW ONLY] Initial commit for determinis...

2016-06-09 Thread olegz
GitHub user olegz opened a pull request: https://github.com/apache/nifi/pull/515 NIFI-826 [REVIEW ONLY] Initial commit for deterministic templates - Added _inceptionId_ via InceptionAware class which is the super class of ComponentDTO and TemplateDTO and is also comparable - The

[GitHub] nifi issue #509: NIFI-1982: Use Compressed check box value.

2016-06-09 Thread bbende
Github user bbende commented on the issue: https://github.com/apache/nifi/pull/509 @ijokarumawak you can close this when you get a chance since it has been merged, thanks! --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub as well.

[GitHub] nifi issue #513: PutHBaseJSON processor treats all values as Strings

2016-06-09 Thread bbende
Github user bbende commented on the issue: https://github.com/apache/nifi/pull/513 Ryan, thanks for submitting this PR! Is it possible that we could remove the dependency on hbase-client here? The reason being is that the hbase processor bundle purposely didn't inclu

[GitHub] nifi pull request #514: fixes NIFI-1989

2016-06-09 Thread PuspenduBanerjee
GitHub user PuspenduBanerjee opened a pull request: https://github.com/apache/nifi/pull/514 fixes NIFI-1989 Fixes : https://issues.sonatype.org/browse/MVNCENTRAL-244 > Taking a look at the POM at http://repo1.maven.org/maven2/org/apache/commons/commons-io/1.3.2/commons-io-1

Re: NIFI ListenTCP Processor

2016-06-09 Thread venkat
Thanks Brayn for creating JIRA task. yes its very useful for us if we have option to change delimiter and processor can pick that delimiter. currently exact match scenario works well for my scenario. Thanks, Venkat -- View this message in context: http://apache-nifi-developer-list.39713.n7.nab

[GitHub] nifi issue #492: NIFI-1975 - Processor for parsing evtx files

2016-06-09 Thread brosander
Github user brosander commented on the issue: https://github.com/apache/nifi/pull/492 changes merged upstream --- 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 s

[GitHub] nifi pull request #492: NIFI-1975 - Processor for parsing evtx files

2016-06-09 Thread brosander
Github user brosander closed the pull request at: https://github.com/apache/nifi/pull/492 --- 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 ena

[GitHub] nifi pull request #501: NIFI-1974 - Support Custom Properties in Expression ...

2016-06-09 Thread YolandaMDavis
Github user YolandaMDavis commented on a diff in the pull request: https://github.com/apache/nifi/pull/501#discussion_r66475348 --- Diff: nifi-api/src/main/java/org/apache/nifi/registry/FileVariableRegistry.java --- @@ -0,0 +1,70 @@ +/* + * Licensed to the Apache Software F

[GitHub] nifi pull request #513: PutHBaseJSON processor treats all values as Strings

2016-06-09 Thread rtempleton
GitHub user rtempleton opened a pull request: https://github.com/apache/nifi/pull/513 PutHBaseJSON processor treats all values as Strings The operator will now inspect the node value to determine type and convert as such. Numeric integral - Long (assumes widest type) Numeric

[GitHub] nifi issue #492: NIFI-1975 - Processor for parsing evtx files

2016-06-09 Thread mattyb149
Github user mattyb149 commented on the issue: https://github.com/apache/nifi/pull/492 +1 LGTM, built and ran tests and contrib-check. Ran a NiFi flow with multiple EVTX files exercising all relationships and granularities. Great contribution, thanks much! Merging to master

[GitHub] nifi pull request #512: NIFI-401

2016-06-09 Thread beugley
GitHub user beugley opened a pull request: https://github.com/apache/nifi/pull/512 NIFI-401 You can merge this pull request into a Git repository by running: $ git pull https://github.com/beugley/nifi NIFI-401 Alternatively you can review and apply these changes as the patch

Re: Limiting a queue

2016-06-09 Thread Shaine Berube
I am on GitHub, but not with this project, plus there is some proprietary code contained within the folder I'm developing in, so I'll create a zip file of the java files that this contains. You should be able to gather and link in the libraries yourself, mostly they're pretty standard. On Thu, Ju

[GitHub] nifi issue #509: NIFI-1982: Use Compressed check box value.

2016-06-09 Thread ijokarumawak
Github user ijokarumawak commented on the issue: https://github.com/apache/nifi/pull/509 @bbende Yes, the same change is in PR #497 . This is only for 0.x. Thanks! --- 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

Re: Limiting a queue

2016-06-09 Thread Matt Burgess
Are you working in GitHub? That's a pretty easy way to share code, I can fork your repo/branch and issue pull requests, or vice versa. If you are working from a fork of the NiFi repo, I think you can make me (mattyb149) a contributor to your fork/branch, and we'd go from there. The latter is a clea

Re: Limiting a queue

2016-06-09 Thread Shaine Berube
@Matt I would enjoy collaborating on it, I'm thinking perhaps with some additional eyes this data flow can get even faster and get some additional bugs worked out. Presently, the necessity of the situation requiring me to mostly hard-code the SQL queries, the data flow only works between an MS SQL

Re: Limiting a queue

2016-06-09 Thread Matt Burgess
Shaine, I was about to start work on processor(s) almost exactly like the ones you are describing. Is this something you'd be interested in collaborating on? I've worked on the SQL generation piece before (though not for NiFi). Regards, Matt On Wed, Jun 8, 2016 at 5:29 PM, Shaine Berube wrote:

[GitHub] nifi issue #509: NIFI-1982: Use Compressed check box value.

2016-06-09 Thread bbende
Github user bbende commented on the issue: https://github.com/apache/nifi/pull/509 +1 looks good, will merge to 0.x... I am assuming this was only meant for 0.x and the same fix is in your other PR for http site-to-site --- If your project is set up for it, you can reply to this emai

Re: [DISCUSS] - Markdown option for documentation artifacts

2016-06-09 Thread Joe Skora
+1 On Wed, Jun 8, 2016 at 10:40 AM, Matt Burgess wrote: > +1 with template > > On Wed, Jun 8, 2016 at 10:39 AM, dan bress wrote: > > +1 > > > > On Wed, Jun 8, 2016 at 7:05 AM Andre wrote: > > > >> +1 on this + a template that matches existing additional.html > >> On 8 Jun 2016 04:28, "Bryan Ro

[GitHub] nifi issue #501: NIFI-1974 - Support Custom Properties in Expression Languag...

2016-06-09 Thread YolandaMDavis
Github user YolandaMDavis commented on the issue: https://github.com/apache/nifi/pull/501 @markap14 Thanks for reviewing! Concerning the extension of VariableRegistryProvider by the ControllerServiceLookup, it came from the need to populate the VariableRegistry from NiFiProperties, if

[GitHub] nifi pull request #501: NIFI-1974 - Support Custom Properties in Expression ...

2016-06-09 Thread YolandaMDavis
Github user YolandaMDavis commented on a diff in the pull request: https://github.com/apache/nifi/pull/501#discussion_r66415203 --- Diff: nifi-api/src/test/java/org/apache/nifi/registry/TestVariableRegistry.java --- @@ -0,0 +1,126 @@ +/* + * Licensed to the Apache Software

[GitHub] nifi pull request #501: NIFI-1974 - Support Custom Properties in Expression ...

2016-06-09 Thread YolandaMDavis
Github user YolandaMDavis commented on a diff in the pull request: https://github.com/apache/nifi/pull/501#discussion_r66412944 --- Diff: nifi-api/src/main/java/org/apache/nifi/registry/FileVariableRegistry.java --- @@ -0,0 +1,70 @@ +/* + * Licensed to the Apache Software F

[GitHub] nifi pull request #501: NIFI-1974 - Support Custom Properties in Expression ...

2016-06-09 Thread YolandaMDavis
Github user YolandaMDavis commented on a diff in the pull request: https://github.com/apache/nifi/pull/501#discussion_r66411796 --- Diff: nifi-api/src/main/java/org/apache/nifi/registry/VariableRegistryUtils.java --- @@ -0,0 +1,55 @@ +/* + * Licensed to the Apache Software

[GitHub] nifi pull request #501: NIFI-1974 - Support Custom Properties in Expression ...

2016-06-09 Thread YolandaMDavis
Github user YolandaMDavis commented on a diff in the pull request: https://github.com/apache/nifi/pull/501#discussion_r66409919 --- Diff: nifi-api/src/main/java/org/apache/nifi/registry/MultiMap.java --- @@ -0,0 +1,154 @@ +/* + * Licensed to the Apache Software Foundation (A

[GitHub] nifi pull request #501: NIFI-1974 - Support Custom Properties in Expression ...

2016-06-09 Thread YolandaMDavis
Github user YolandaMDavis commented on a diff in the pull request: https://github.com/apache/nifi/pull/501#discussion_r66407532 --- Diff: nifi-api/src/test/java/org/apache/nifi/registry/TestVariableRegistry.java --- @@ -0,0 +1,126 @@ +/* + * Licensed to the Apache Software

[GitHub] nifi pull request #501: NIFI-1974 - Support Custom Properties in Expression ...

2016-06-09 Thread YolandaMDavis
Github user YolandaMDavis commented on a diff in the pull request: https://github.com/apache/nifi/pull/501#discussion_r66406906 --- Diff: nifi-api/src/test/resources/TestVariableRegistry/foobar.properties --- @@ -0,0 +1 @@ +fake.property.3=test me out 3, test me out 4 --- E