[GitHub] nifi issue #564: NIFI-2020 - Enhance JoltTransformJSON processor to support ...

2016-06-27 Thread YolandaMDavis
Github user YolandaMDavis commented on the issue: https://github.com/apache/nifi/pull/564 @davetorok here is the first phase of custom transform support (supporting the transform interface) --- If your project is set up for it, you can reply to this email and have your reply appear

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

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

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

2016-06-23 Thread YolandaMDavis
GitHub user YolandaMDavis opened a pull request: https://github.com/apache/nifi/pull/571 NIFI-1974 - Support Custom Properties in Expression Language This is the commit of the custom property expression language support with Variable Registry (for master) You can merge this pull

[GitHub] nifi issue #564: NIFI-2020 - Enhance JoltTransformJSON processor to support ...

2016-06-22 Thread YolandaMDavis
Github user YolandaMDavis commented on the issue: https://github.com/apache/nifi/pull/564 Just to add I'm thinking of incorporating ContextualTransform support once Variable Registry is incorporated to provide the transformation access to system/env properties, flow file attri

[GitHub] nifi pull request #564: NIFI-2020 - Enhance JoltTransformJSON processor to s...

2016-06-22 Thread YolandaMDavis
GitHub user YolandaMDavis opened a pull request: https://github.com/apache/nifi/pull/564 NIFI-2020 - Enhance JoltTransformJSON processor to support custom transforms This is the initial commit to enhance the JoltTransformJSON processor by providing the ability to support custom

[GitHub] nifi pull request #553: NIFI-2063 - Install Script Relative Path Mismatch fr...

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

[GitHub] nifi issue #553: NIFI-2063 - Install Script Relative Path Mismatch from Init...

2016-06-22 Thread YolandaMDavis
Github user YolandaMDavis commented on the issue: https://github.com/apache/nifi/pull/553 @jvwing @apiri closing out this PR, @apiri has a version coming. --- 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

[GitHub] nifi issue #553: NIFI-2063 - Install Script Relative Path Mismatch from Init...

2016-06-21 Thread YolandaMDavis
Github user YolandaMDavis commented on the issue: https://github.com/apache/nifi/pull/553 @jvwing thanks james for. Makes sense on all fronts. I'm working on an update now. --- If your project is set up for it, you can reply to this email and have your reply appear on GitH

[GitHub] nifi issue #553: NIFI-2063 - Install Script Relative Path Mismatch from Init...

2016-06-21 Thread YolandaMDavis
Github user YolandaMDavis commented on the issue: https://github.com/apache/nifi/pull/553 @apiri I agree, I think we can do the latter, to ensure the configuration is added with the one copy. --- If your project is set up for it, you can reply to this email and have your reply

[GitHub] nifi issue #553: NIFI-2063 - Install Script Relative Path Mismatch from Init...

2016-06-21 Thread YolandaMDavis
Github user YolandaMDavis commented on the issue: https://github.com/apache/nifi/pull/553 Some screen shoots of testing update on vagrant vm ![nifi_install_start](https://cloud.githubusercontent.com/assets/1371858/16241243/d26e9d74-37ba-11e6-88d5-705b0d9767bc.jpg

[GitHub] nifi pull request #553: NIFI-2063 - correct service installation problems wi...

2016-06-21 Thread YolandaMDavis
GitHub user YolandaMDavis opened a pull request: https://github.com/apache/nifi/pull/553 NIFI-2063 - correct service installation problems with nifi-env.sh Made changes to ensure proper execution of nifi.sh install: 1) added cp command of nifi_env.sh to /etc/init.d so it

[GitHub] nifi issue #511: NIFI-1850 - JSON-to-JSON Schema Converter Editor

2016-06-17 Thread YolandaMDavis
Github user YolandaMDavis commented on the issue: https://github.com/apache/nifi/pull/511 @mcgilman Thank you for reviewing! The PR has been updated based on your comments, including the corrections in the path segment for StandardNiFiWebConfigurationContext. I also found that same

[GitHub] nifi pull request #511: NIFI-1850 - JSON-to-JSON Schema Converter Editor

2016-06-17 Thread YolandaMDavis
Github user YolandaMDavis commented on a diff in the pull request: https://github.com/apache/nifi/pull/511#discussion_r67536160 --- Diff: nifi-nar-bundles/nifi-framework-bundle/nifi-framework/nifi-web/nifi-web-api/src/main/java/org/apache/nifi/web/StandardNiFiServiceFacade.java

[GitHub] nifi pull request #511: NIFI-1850 - JSON-to-JSON Schema Converter Editor

2016-06-17 Thread YolandaMDavis
Github user YolandaMDavis commented on a diff in the pull request: https://github.com/apache/nifi/pull/511#discussion_r67527428 --- Diff: nifi-nar-bundles/nifi-framework-bundle/nifi-framework/nifi-web/nifi-web-api/pom.xml --- @@ -355,10 +355,5 @@ spock-core

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

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

[GitHub] nifi pull request #511: NIFI-1850 - JSON-to-JSON Schema Converter Editor

2016-06-15 Thread YolandaMDavis
Github user YolandaMDavis commented on a diff in the pull request: https://github.com/apache/nifi/pull/511#discussion_r67226364 --- Diff: nifi-nar-bundles/nifi-standard-bundle/nifi-jolt-transform-json-ui/src/main/java/org/apache/nifi/web/util/CustomUIUtils.java --- @@ -0,0 +1,88

[GitHub] nifi issue #511: NIFI-1850 - JSON-to-JSON Schema Converter Editor

2016-06-15 Thread YolandaMDavis
Github user YolandaMDavis commented on the issue: https://github.com/apache/nifi/pull/511 @olegz thanks for taking a look! as noted above I'll make the changes cited and update this request. --- If your project is set up for it, you can reply to this email and have your reply a

[GitHub] nifi pull request #511: NIFI-1850 - JSON-to-JSON Schema Converter Editor

2016-06-15 Thread YolandaMDavis
Github user YolandaMDavis commented on a diff in the pull request: https://github.com/apache/nifi/pull/511#discussion_r67222589 --- Diff: nifi-nar-bundles/nifi-framework-bundle/nifi-framework/nifi-web/nifi-web-api/src/main/java/org/apache/nifi/web

[GitHub] nifi pull request #511: NIFI-1850 - JSON-to-JSON Schema Converter Editor

2016-06-15 Thread YolandaMDavis
Github user YolandaMDavis commented on a diff in the pull request: https://github.com/apache/nifi/pull/511#discussion_r67222136 --- Diff: nifi-nar-bundles/nifi-framework-bundle/nifi-framework/nifi-web/nifi-web-api/src/main/java/org/apache/nifi/web

[GitHub] nifi pull request #511: NIFI-1850 - JSON-to-JSON Schema Converter Editor

2016-06-15 Thread YolandaMDavis
Github user YolandaMDavis commented on a diff in the pull request: https://github.com/apache/nifi/pull/511#discussion_r67221807 --- Diff: nifi-nar-bundles/nifi-standard-bundle/nifi-jolt-transform-json-ui/src/main/java/org/apache/nifi/web/util/CustomUIUtils.java --- @@ -0,0 +1,88

[GitHub] nifi pull request #511: NIFI-1850 - JSON-to-JSON Schema Converter Editor

2016-06-15 Thread YolandaMDavis
Github user YolandaMDavis commented on a diff in the pull request: https://github.com/apache/nifi/pull/511#discussion_r67215337 --- Diff: nifi-nar-bundles/nifi-framework-bundle/nifi-framework/nifi-web/nifi-web-api/src/main/java/org/apache/nifi/web

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

2016-06-14 Thread YolandaMDavis
Github user YolandaMDavis commented on the issue: https://github.com/apache/nifi/pull/501 Per my discussion with @markap14 having ControllerServiceLookup extend VariableRegistry interface did not fit in with the overall purpose of the ControllerServiceLookup interface. Also in some

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

2016-06-10 Thread YolandaMDavis
Github user YolandaMDavis commented on the issue: https://github.com/apache/nifi/pull/501 @markap14 per our offline discussion I made changes that provides variable registry via constructors and eliminated the access from the ControllerServiceLookup. --- If your project is set up

[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

[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

[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

[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

[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

[GitHub] nifi pull request #511: NIFI-1850 - JSON-to-JSON Schema Converter Editor

2016-06-08 Thread YolandaMDavis
GitHub user YolandaMDavis opened a pull request: https://github.com/apache/nifi/pull/511 NIFI-1850 - JSON-to-JSON Schema Converter Editor This is a merge from 0.7 of the Json-to-Json (Jolt) Editor with refactoring for masterless clustering and bower dependency support. You can

[GitHub] nifi pull request #424: NIFI-1850 - Initial Commit for JSON-to-JSON Schema C...

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

[GitHub] nifi pull request #501: NIFI-1974 - initial commit variable registry for cus...

2016-06-06 Thread YolandaMDavis
GitHub user YolandaMDavis opened a pull request: https://github.com/apache/nifi/pull/501 NIFI-1974 - initial commit variable registry for custom property support This is an initial commit for NIFI-1974 which will support making custom property files available for use in expression

[GitHub] nifi pull request: NIFI-1850 - Initial Commit for JSON-to-JSON Sch...

2016-05-17 Thread YolandaMDavis
Github user YolandaMDavis commented on the pull request: https://github.com/apache/nifi/pull/424#issuecomment-219902820 @mcgilman no problem. will refer to just codemirror in nifi-web-ui for 0.x and both angular and codemiror in 1.0 (since angular is only available in that module in

[GitHub] nifi pull request: NIFI-1850 - Initial Commit for JSON-to-JSON Sch...

2016-05-17 Thread YolandaMDavis
Github user YolandaMDavis commented on a diff in the pull request: https://github.com/apache/nifi/pull/424#discussion_r63629669 --- Diff: nifi-nar-bundles/nifi-standard-bundle/nifi-transform-json-ui/src/main/java/org/apache/nifi/web/standard/api/processor/ProcessorResource.java

[GitHub] nifi pull request: NIFI-1850 - Initial Commit for JSON-to-JSON Sch...

2016-05-17 Thread YolandaMDavis
Github user YolandaMDavis commented on a diff in the pull request: https://github.com/apache/nifi/pull/424#discussion_r63624220 --- Diff: nifi-nar-bundles/nifi-framework-bundle/nifi-framework/nifi-web/nifi-web-api/src/main/java/org/apache/nifi/web/StandardNiFiServiceFacade.java

[GitHub] nifi pull request: NIFI-1850 - Initial Commit for JSON-to-JSON Sch...

2016-05-17 Thread YolandaMDavis
Github user YolandaMDavis commented on the pull request: https://github.com/apache/nifi/pull/424#issuecomment-219871908 @mcgilman had same conversation with @scottyaslan last week concerning referencing js in nifi-web-ui. My main concern was versioning e.g. changes happening at that

[GitHub] nifi pull request: NIFI-1850 - Initial Commit for JSON-to-JSON Sch...

2016-05-17 Thread YolandaMDavis
Github user YolandaMDavis commented on a diff in the pull request: https://github.com/apache/nifi/pull/424#discussion_r63613822 --- Diff: nifi-nar-bundles/nifi-standard-bundle/nifi-transform-json-ui/src/main/webapp/app/components/processor/processor.service.js --- @@ -0,0 +1,52

[GitHub] nifi pull request: NIFI-1812 nifi-env scripts (linux/windows)

2016-05-10 Thread YolandaMDavis
Github user YolandaMDavis commented on the pull request: https://github.com/apache/nifi/pull/386#issuecomment-218157232 @alopresto added changes based on your comments. Thanks! --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub as

[GitHub] nifi pull request: NIFI-1850 - Initial Commit for JSON-to-JSON Sch...

2016-05-09 Thread YolandaMDavis
GitHub user YolandaMDavis opened a pull request: https://github.com/apache/nifi/pull/424 NIFI-1850 - Initial Commit for JSON-to-JSON Schema Converter Editor This is an initial commit for review of the TransformJson Advanced Editor. Please see https://issues.apache.org/jira/browse

[GitHub] nifi pull request: NIFI-1812 nifi-env scripts (linux/windows)

2016-05-05 Thread YolandaMDavis
Github user YolandaMDavis commented on a diff in the pull request: https://github.com/apache/nifi/pull/386#discussion_r62165943 --- Diff: nifi-nar-bundles/nifi-framework-bundle/nifi-framework/nifi-resources/src/main/resources/bin/nifi.sh --- @@ -211,9 +211,9 @@ run

[GitHub] nifi pull request: NIFI-1812 nifi-env scripts (linux/windows)

2016-05-05 Thread YolandaMDavis
Github user YolandaMDavis commented on a diff in the pull request: https://github.com/apache/nifi/pull/386#discussion_r62165580 --- Diff: nifi-nar-bundles/nifi-framework-bundle/nifi-framework/nifi-resources/src/main/resources/bin/nifi-env.bat --- @@ -0,0 +1,29 @@ +@echo off

[GitHub] nifi pull request: NIFI-1812 nifi-env scripts (linux/windows)

2016-05-05 Thread YolandaMDavis
Github user YolandaMDavis commented on a diff in the pull request: https://github.com/apache/nifi/pull/386#discussion_r62165158 --- Diff: nifi-bootstrap/src/main/java/org/apache/nifi/bootstrap/RunNiFi.java --- @@ -324,27 +324,46 @@ private void registerNotificationServices(final

[GitHub] nifi pull request: NIFI-361- Create Processors to mutate JSON data

2016-05-03 Thread YolandaMDavis
Github user YolandaMDavis closed the pull request at: https://github.com/apache/nifi/pull/405 --- 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

[GitHub] nifi pull request: NIFI-361- Create Processors to mutate JSON data

2016-05-02 Thread YolandaMDavis
GitHub user YolandaMDavis opened a pull request: https://github.com/apache/nifi/pull/405 NIFI-361- Create Processors to mutate JSON data Implementation of the TransformJSON processor using the Jolt library. TransformJSON supports Jolt specifications for the following

[GitHub] nifi pull request: NIFI-1812 nifi-env scripts (linux/windows)

2016-04-28 Thread YolandaMDavis
GitHub user YolandaMDavis opened a pull request: https://github.com/apache/nifi/pull/386 NIFI-1812 nifi-env scripts (linux/windows) Initial commit for nifi-env scripts. Scripts will provide a centralized method of setting environmental variables for use by other scripts (e.g

[GitHub] nifi pull request: NIFI-361 - Create Processors to mutate JSON dat...

2016-04-21 Thread YolandaMDavis
Github user YolandaMDavis commented on a diff in the pull request: https://github.com/apache/nifi/pull/354#discussion_r60631450 --- Diff: nifi-nar-bundles/nifi-standard-bundle/nifi-standard-processors/src/main/java/org/apache/nifi/processors/standard/TransformJSON.java

[GitHub] nifi pull request: NIFI-361 - Create Processors to mutate JSON dat...

2016-04-19 Thread YolandaMDavis
Github user YolandaMDavis commented on a diff in the pull request: https://github.com/apache/nifi/pull/354#discussion_r60302366 --- Diff: nifi-nar-bundles/nifi-standard-bundle/nifi-standard-processors/src/main/java/org/apache/nifi/processors/standard/TransformJSON.java

[GitHub] nifi pull request: NIFI-361 - Create Processors to mutate JSON dat...

2016-04-19 Thread YolandaMDavis
Github user YolandaMDavis commented on a diff in the pull request: https://github.com/apache/nifi/pull/354#discussion_r60297981 --- Diff: nifi-nar-bundles/nifi-standard-bundle/nifi-standard-processors/src/main/java/org/apache/nifi/processors/standard/TransformJSON.java

[GitHub] nifi pull request: NIFI-361 - Create Processors to mutate JSON dat...

2016-04-18 Thread YolandaMDavis
Github user YolandaMDavis commented on a diff in the pull request: https://github.com/apache/nifi/pull/354#discussion_r60164832 --- Diff: nifi-nar-bundles/nifi-standard-bundle/nifi-standard-processors/src/main/java/org/apache/nifi/processors/standard/TransformJSON.java

[GitHub] nifi pull request: NIFI-361 - Create Processors to mutate JSON dat...

2016-04-18 Thread YolandaMDavis
Github user YolandaMDavis commented on a diff in the pull request: https://github.com/apache/nifi/pull/354#discussion_r60115891 --- Diff: nifi-nar-bundles/nifi-standard-bundle/nifi-standard-processors/src/main/java/org/apache/nifi/processors/standard/TransformJSON.java

[GitHub] nifi pull request: NIFI-361 - Create Processors to mutate JSON dat...

2016-04-18 Thread YolandaMDavis
Github user YolandaMDavis commented on a diff in the pull request: https://github.com/apache/nifi/pull/354#discussion_r60080948 --- Diff: nifi-nar-bundles/nifi-standard-bundle/nifi-standard-processors/src/main/java/org/apache/nifi/processors/standard/TransformJSON.java

[GitHub] nifi pull request: NIFI-361 - Create Processors to mutate JSON dat...

2016-04-14 Thread YolandaMDavis
GitHub user YolandaMDavis opened a pull request: https://github.com/apache/nifi/pull/354 NIFI-361 - Create Processors to mutate JSON data This is an initial implementation of the TransformJSON processor using the Jolt library. TransformJSON supports Jolt specifications for the

[GitHub] nifi pull request: NIFI-361 - Create Processors to mutate JSON dat...

2016-04-14 Thread YolandaMDavis
Github user YolandaMDavis closed the pull request at: https://github.com/apache/nifi/pull/353 --- 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

[GitHub] nifi pull request: NIFI-361 - Create Processors to mutate JSON dat...

2016-04-14 Thread YolandaMDavis
GitHub user YolandaMDavis opened a pull request: https://github.com/apache/nifi/pull/353 NIFI-361 - Create Processors to mutate JSON data This is an initial implementation of the TransformJSON processor using the Jolt library. TransformJSON supports Jolt specifications for the