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 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 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 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 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 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 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 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 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 user YolandaMDavis commented on the issue:
https://github.com/apache/nifi/pull/553
Some screen shoots of testing update on vagrant vm
 added cp command of nifi_env.sh to /etc/init.d so it
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 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 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 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 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 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 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 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 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 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 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 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 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 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 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 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 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 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 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 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 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 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 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 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 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 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 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 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 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 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 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 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 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 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 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 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 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 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 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 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 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 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 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 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 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
56 matches
Mail list logo