GitHub user markap14 opened a pull request:
https://github.com/apache/nifi/pull/352
NIFI-1762: Use Lambda Expressions in StandardNiFIServiceFacade to simplify
codebase
Changed Java dependency to 1.8 instead of 1.7 and refactored
StandardNiFiServiceFacade to make use of Lambda
Github user markap14 commented on a diff in the pull request:
https://github.com/apache/nifi/pull/321#discussion_r59908921
--- Diff: nifi-mock/src/main/java/org/apache/nifi/util/TestRunner.java ---
@@ -865,4 +866,21 @@
* @return the State Manager that is used to store and
Github user markap14 commented on the pull request:
https://github.com/apache/nifi/pull/359#issuecomment-210612644
Looks good. I verified that I was able to start NiFi using both secure and
non-secure ports. I was then able to verify that I was able to add each
component, update it
Github user markap14 commented on the pull request:
https://github.com/apache/nifi/pull/359#issuecomment-210618998
Looks like the latest commit addresses both issues outlined above. Will
merge into master.
Thanks for knocking this out!
---
If your project is set up for it, you
Github user markap14 commented on a diff in the pull request:
https://github.com/apache/nifi/pull/365#discussion_r60290762
--- Diff:
nifi-commons/nifi-processor-utilities/src/main/java/org/apache/nifi/processor/util/StandardValidators.java
---
@@ -127,12 +127,26 @@ public
Github user markap14 commented on the pull request:
https://github.com/apache/nifi/pull/365#issuecomment-212085785
+1 this looks good to me.
---
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 user markap14 opened a pull request:
https://github.com/apache/nifi/pull/369
NIFI-1790: Updated the URI patterns used in NCM for determining whether or
not responses should be merged
NIFI-1790: Updated the URI patterns used in NCM for determining whether or
not responses
Github user markap14 commented on a diff in the pull request:
https://github.com/apache/nifi/pull/372#discussion_r60474156
--- Diff:
nifi-nar-bundles/nifi-hive-bundle/nifi-hive-processors/src/main/java/org/apache/nifi/dbcp/hive/HiveConnectionPool.java
---
@@ -0,0 +1,184
Github user markap14 commented on a diff in the pull request:
https://github.com/apache/nifi/pull/372#discussion_r60474410
--- Diff:
nifi-nar-bundles/nifi-hive-bundle/nifi-hive-processors/src/main/java/org/apache/nifi/dbcp/hive/HiveConnectionPool.java
---
@@ -0,0 +1,184
Github user markap14 commented on a diff in the pull request:
https://github.com/apache/nifi/pull/323#discussion_r60781310
--- Diff:
nifi-nar-bundles/nifi-framework-bundle/nifi-framework/nifi-framework-cluster-protocol/src/main/java/org/apache/nifi/cluster/protocol/jaxb/message
Github user markap14 commented on a diff in the pull request:
https://github.com/apache/nifi/pull/323#discussion_r60781488
--- Diff:
nifi-nar-bundles/nifi-framework-bundle/nifi-framework/nifi-framework-cluster/src/main/java/org/apache/nifi/cluster/coordination/heartbeat
Github user markap14 commented on a diff in the pull request:
https://github.com/apache/nifi/pull/323#discussion_r60781637
--- Diff:
nifi-nar-bundles/nifi-framework-bundle/nifi-framework/nifi-framework-cluster/src/main/java/org/apache/nifi/cluster/coordination/heartbeat
Github user markap14 commented on a diff in the pull request:
https://github.com/apache/nifi/pull/323#discussion_r60782150
--- Diff:
nifi-nar-bundles/nifi-framework-bundle/nifi-framework/nifi-framework-cluster/src/main/java/org/apache/nifi/cluster/coordination/heartbeat
Github user markap14 commented on a diff in the pull request:
https://github.com/apache/nifi/pull/323#discussion_r60782213
--- Diff:
nifi-nar-bundles/nifi-framework-bundle/nifi-framework/nifi-framework-cluster/src/main/java/org/apache/nifi/cluster/coordination/heartbeat
Github user markap14 commented on a diff in the pull request:
https://github.com/apache/nifi/pull/323#discussion_r60782403
--- Diff:
nifi-nar-bundles/nifi-framework-bundle/nifi-framework/nifi-framework-cluster/src/main/java/org/apache/nifi/cluster/coordination/heartbeat
Github user markap14 commented on a diff in the pull request:
https://github.com/apache/nifi/pull/323#discussion_r60782492
--- Diff:
nifi-nar-bundles/nifi-framework-bundle/nifi-framework/nifi-framework-cluster/src/main/java/org/apache/nifi/cluster/coordination/heartbeat
Github user markap14 commented on a diff in the pull request:
https://github.com/apache/nifi/pull/323#discussion_r60782754
--- Diff:
nifi-nar-bundles/nifi-framework-bundle/nifi-framework/nifi-framework-cluster/src/main/java/org/apache/nifi/cluster/coordination/heartbeat
Github user markap14 commented on a diff in the pull request:
https://github.com/apache/nifi/pull/323#discussion_r60782947
--- Diff:
nifi-nar-bundles/nifi-framework-bundle/nifi-framework/nifi-framework-cluster/src/main/java/org/apache/nifi/cluster/manager/impl
GitHub user markap14 opened a pull request:
https://github.com/apache/nifi/pull/391
NIFI-1824: If attempting to archive content, and there are no claimant
counts for it, ensure that the stream is closed.
NIFI-1824: If attempting to archive content, and there are no claimant
counts
Github user markap14 commented on a diff in the pull request:
https://github.com/apache/nifi/pull/389#discussion_r61485635
--- Diff:
nifi-nar-bundles/nifi-standard-bundle/nifi-standard-processors/src/main/java/org/apache/nifi/processors/standard/HandleHttpResponse.java
Github user markap14 commented on the pull request:
https://github.com/apache/nifi/pull/377#issuecomment-215533199
We have talked before about having some sort of icon on a connection that
indicates that back pressure is configured and then some sort of indicator that
back pressure
Github user markap14 commented on a diff in the pull request:
https://github.com/apache/nifi/pull/389#discussion_r61509041
--- Diff:
nifi-nar-bundles/nifi-standard-bundle/nifi-standard-processors/src/main/java/org/apache/nifi/processors/standard/HandleHttpResponse.java
GitHub user markap14 opened a pull request:
https://github.com/apache/nifi/pull/394
NIFI-1830: Fixed problems in the merging logic for Drop FlowFile Requests
NIFI-1830: Fixed problems in the merging logic for Drop FlowFile Requests
You can merge this pull request into a Git
Github user markap14 closed the pull request at:
https://github.com/apache/nifi/pull/391
---
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
GitHub user markap14 opened a pull request:
https://github.com/apache/nifi/pull/413
NIFI-1745: Refactor how requests are replicated across cluster
This still has some work to be done, but I believe it's worth merging into
master now, only because we need to merge these changes
Github user markap14 closed the pull request at:
https://github.com/apache/nifi/pull/338
---
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
Github user markap14 commented on a diff in the pull request:
https://github.com/apache/nifi/pull/420#discussion_r62700624
--- Diff:
nifi-nar-bundles/nifi-standard-bundle/nifi-standard-processors/src/test/java/org/apache/nifi/processors/standard/TestFilterCSVColumns.java
Github user markap14 commented on a diff in the pull request:
https://github.com/apache/nifi/pull/420#discussion_r62700767
--- Diff:
nifi-nar-bundles/nifi-standard-bundle/nifi-standard-processors/src/test/java/org/apache/nifi/processors/standard/TestFilterCSVColumns.java
Github user markap14 commented on a diff in the pull request:
https://github.com/apache/nifi/pull/420#discussion_r62700756
--- Diff:
nifi-nar-bundles/nifi-standard-bundle/nifi-standard-processors/src/test/java/org/apache/nifi/processors/standard/TestFilterCSVColumns.java
Github user markap14 commented on a diff in the pull request:
https://github.com/apache/nifi/pull/420#discussion_r62700846
--- Diff:
nifi-nar-bundles/nifi-standard-bundle/nifi-standard-processors/src/main/java/org/apache/nifi/processors/standard/FilterCSVColumns.java
---
@@ -0,0
Github user markap14 commented on a diff in the pull request:
https://github.com/apache/nifi/pull/420#discussion_r62701139
--- Diff:
nifi-nar-bundles/nifi-standard-bundle/nifi-standard-processors/src/main/java/org/apache/nifi/processors/standard/FilterCSVColumns.java
---
@@ -0,0
Github user markap14 commented on a diff in the pull request:
https://github.com/apache/nifi/pull/420#discussion_r62700516
--- Diff:
nifi-nar-bundles/nifi-standard-bundle/nifi-standard-processors/src/test/resources/TestFilterCSVColumns/US500.csv
---
@@ -0,0 +1
Github user markap14 commented on a diff in the pull request:
https://github.com/apache/nifi/pull/420#discussion_r62701269
--- Diff:
nifi-nar-bundles/nifi-standard-bundle/nifi-standard-processors/src/main/java/org/apache/nifi/processors/standard/FilterCSVColumns.java
---
@@ -0,0
Github user markap14 commented on a diff in the pull request:
https://github.com/apache/nifi/pull/420#discussion_r62701492
--- Diff:
nifi-nar-bundles/nifi-standard-bundle/nifi-standard-processors/src/main/java/org/apache/nifi/processors/standard/FilterCSVColumns.java
---
@@ -0,0
GitHub user markap14 opened a pull request:
https://github.com/apache/nifi/pull/431
NIFI-1800: Tie Controller Services to Process Groups
You can merge this pull request into a Git repository by running:
$ git pull https://github.com/markap14/nifi NIFI-1800
Alternatively you
Github user markap14 commented on the pull request:
https://github.com/apache/nifi/pull/431#issuecomment-218459759
@mcgilman sorry - it appears that I messed up with I pushed the changes to
the branch, before I created the PR. What you looked at was an incomplete
version from a few
Github user markap14 commented on the pull request:
https://github.com/apache/nifi/pull/420#issuecomment-218520423
I don't think we can use the provided test data as-is. Looking at
http://www.briandunning.com/sample-data/ I don't see any sort of LICENSE info.
Probably be
Github user markap14 commented on the pull request:
https://github.com/apache/nifi/pull/420#issuecomment-218531630
@ToivoAdams After looking at this a bit more, I've got a few very
high-level thoughts. First is that what you have built here is incredibly
awesome and powerful! I
Github user markap14 commented on the pull request:
https://github.com/apache/nifi/pull/420#issuecomment-218543143
@ToivoAdams as I think even more about this, it's opening up a lot of
awesome possibilities. I think we could expand this Processor even more into
really a "SQ
Github user markap14 commented on a diff in the pull request:
https://github.com/apache/nifi/pull/419#discussion_r62906404
--- Diff:
nifi-nar-bundles/nifi-site-to-site-reporting-bundle/nifi-site-to-site-reporting-task/src/main/java/org/apache/nifi/reporting
Github user markap14 commented on a diff in the pull request:
https://github.com/apache/nifi/pull/419#discussion_r62906556
--- Diff:
nifi-nar-bundles/nifi-site-to-site-reporting-bundle/nifi-site-to-site-reporting-task/src/main/java/org/apache/nifi/reporting
Github user markap14 commented on a diff in the pull request:
https://github.com/apache/nifi/pull/419#discussion_r62907252
--- Diff:
nifi-nar-bundles/nifi-site-to-site-reporting-bundle/nifi-site-to-site-reporting-task/src/main/java/org/apache/nifi/reporting
Github user markap14 commented on a diff in the pull request:
https://github.com/apache/nifi/pull/419#discussion_r62906770
--- Diff:
nifi-nar-bundles/nifi-site-to-site-reporting-bundle/nifi-site-to-site-reporting-task/src/main/java/org/apache/nifi/reporting
Github user markap14 commented on the pull request:
https://github.com/apache/nifi/pull/420#issuecomment-218563978
Excellent. Yes, I'm thinking potentially Avro, JSON, Parquet... perhaps
others. For 0.7.0 we may only have CSV and perhaps JSON but will have things in
place to add
Github user markap14 commented on a diff in the pull request:
https://github.com/apache/nifi/pull/419#discussion_r62914873
--- Diff:
nifi-nar-bundles/nifi-site-to-site-reporting-bundle/nifi-site-to-site-reporting-task/src/main/java/org/apache/nifi/reporting
Github user markap14 commented on the pull request:
https://github.com/apache/nifi/pull/419#issuecomment-218585975
@bbende I merged this to 0.x branch. However, it cannot be merged to the
'master' branch as-is. Can you create a PR That will merge to master?
---
If your
GitHub user markap14 opened a pull request:
https://github.com/apache/nifi/pull/437
NIFI-1872: Ignore failing unit test for now until we can properly address
Since it is on master it's best to ignore, since the problem appears to be
the lifecycle of the unit test
You can
GitHub user markap14 opened a pull request:
https://github.com/apache/nifi/pull/440
NIFI-1872: Allow sufficient time for embedded server to startup for uâ¦
â¦nit tests
You can merge this pull request into a Git repository by running:
$ git pull https://github.com/markap14
GitHub user markap14 opened a pull request:
https://github.com/apache/nifi/pull/441
NIFI-1872: Allow for adequate time for embedded zookeeper server to startup
for unit tests
PR for 0.x baseline
You can merge this pull request into a Git repository by running:
$ git pull
GitHub user markap14 opened a pull request:
https://github.com/apache/nifi/pull/446
NIFI-1801: Scope Templates to Process Groups
Also includes a few bug fixes that were not necessarily explicitly related
to templates being scoped to Process Groups but that are required in order to
Github user markap14 commented on a diff in the pull request:
https://github.com/apache/nifi/pull/446#discussion_r63344217
--- Diff:
nifi-nar-bundles/nifi-framework-bundle/nifi-framework/nifi-framework-core-api/src/main/java/org/apache/nifi/controller/Template.java
---
@@ -0,0
Github user markap14 commented on a diff in the pull request:
https://github.com/apache/nifi/pull/439#discussion_r63344780
--- Diff:
nifi-nar-bundles/nifi-framework-bundle/nifi-framework/nifi-framework-core/src/main/java/org/apache/nifi/controller/repository
Github user markap14 commented on a diff in the pull request:
https://github.com/apache/nifi/pull/439#discussion_r63425128
--- Diff:
nifi-nar-bundles/nifi-framework-bundle/nifi-framework/nifi-framework-core/src/main/java/org/apache/nifi/controller/repository
Github user markap14 commented on a diff in the pull request:
https://github.com/apache/nifi/pull/439#discussion_r63425553
--- Diff:
nifi-nar-bundles/nifi-framework-bundle/nifi-framework/nifi-framework-core/src/main/java/org/apache/nifi/controller/repository
Github user markap14 commented on a diff in the pull request:
https://github.com/apache/nifi/pull/439#discussion_r63425823
--- Diff:
nifi-nar-bundles/nifi-framework-bundle/nifi-framework/nifi-framework-core/src/test/java/org/apache/nifi/controller/repository
GitHub user markap14 opened a pull request:
https://github.com/apache/nifi/pull/454
NIFI-1745: Refactor how revisions are handled at NCM/Distributed to Node
This is part 2 of NIFI-1745, which provides user info to revision
claims/locks and removes the cluster context, as it is no
Github user markap14 commented on a diff in the pull request:
https://github.com/apache/nifi/pull/454#discussion_r64043396
--- Diff:
nifi-nar-bundles/nifi-framework-bundle/nifi-framework/nifi-web/nifi-web-optimistic-locking/src/main/java/org/apache/nifi/web/revision
Github user markap14 commented on the pull request:
https://github.com/apache/nifi/pull/454#issuecomment-220610191
@mcgilman I agree, I do think it makes more sense to handle that in the
Revision. Have updated the PR to back that out so that we can instead do it in
the Resource in
Github user markap14 commented on a diff in the pull request:
https://github.com/apache/nifi/pull/459#discussion_r64097258
--- Diff:
nifi-nar-bundles/nifi-avro-bundle/nifi-avro-processors/src/main/java/org/apache/nifi/processors/avro/ConvertAvroToJSON.java
---
@@ -92,6 +103,7
Github user markap14 commented on a diff in the pull request:
https://github.com/apache/nifi/pull/459#discussion_r64097658
--- Diff:
nifi-nar-bundles/nifi-avro-bundle/nifi-avro-processors/src/main/java/org/apache/nifi/processors/avro/ConvertAvroToJSON.java
---
@@ -128,49 +141,77
Github user markap14 commented on a diff in the pull request:
https://github.com/apache/nifi/pull/303#discussion_r64234770
--- Diff:
nifi-commons/nifi-expression-language/src/main/java/org/apache/nifi/attribute/expression/language/evaluation/functions/JsonPathEvaluator.java
Github user markap14 commented on a diff in the pull request:
https://github.com/apache/nifi/pull/303#discussion_r64235176
--- Diff:
nifi-commons/nifi-expression-language/src/test/java/org/apache/nifi/attribute/expression/language/TestQuery.java
---
@@ -233,6 +233,24 @@ public
Github user markap14 commented on a diff in the pull request:
https://github.com/apache/nifi/pull/461#discussion_r64247888
--- Diff:
nifi-nar-bundles/nifi-framework-bundle/nifi-framework/nifi-framework-core-api/src/main/java/org/apache/nifi/groups/ProcessGroup.java
---
@@ -46,6
Github user markap14 commented on a diff in the pull request:
https://github.com/apache/nifi/pull/461#discussion_r64248023
--- Diff:
nifi-nar-bundles/nifi-framework-bundle/nifi-framework/nifi-framework-core-api/src/main/java/org/apache/nifi/groups/ProcessGroup.java
---
@@ -46,6
Github user markap14 commented on a diff in the pull request:
https://github.com/apache/nifi/pull/461#discussion_r64248276
--- Diff:
nifi-nar-bundles/nifi-framework-bundle/nifi-framework/nifi-framework-core/src/main/java/org/apache/nifi/groups/StandardProcessGroup.java
Github user markap14 commented on a diff in the pull request:
https://github.com/apache/nifi/pull/461#discussion_r64249465
--- 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 markap14 commented on a diff in the pull request:
https://github.com/apache/nifi/pull/461#discussion_r64256311
--- Diff:
nifi-nar-bundles/nifi-framework-bundle/nifi-framework/nifi-web/nifi-web-api/src/main/java/org/apache/nifi/web/api/FlowResource.java
---
@@ -321,6
Github user markap14 commented on a diff in the pull request:
https://github.com/apache/nifi/pull/461#discussion_r64256472
--- Diff:
nifi-nar-bundles/nifi-framework-bundle/nifi-framework/nifi-web/nifi-web-api/src/main/java/org/apache/nifi/web/api/FlowResource.java
---
@@ -321,6
Github user markap14 commented on a diff in the pull request:
https://github.com/apache/nifi/pull/461#discussion_r64257999
--- Diff:
nifi-nar-bundles/nifi-framework-bundle/nifi-framework/nifi-web/nifi-web-api/src/main/java/org/apache/nifi/web/api/ProcessGroupResource.java
Github user markap14 commented on a diff in the pull request:
https://github.com/apache/nifi/pull/461#discussion_r64258489
--- Diff:
nifi-nar-bundles/nifi-framework-bundle/nifi-framework/nifi-web/nifi-web-api/src/main/java/org/apache/nifi/web/api/ProcessGroupResource.java
Github user markap14 commented on a diff in the pull request:
https://github.com/apache/nifi/pull/461#discussion_r64259092
--- Diff:
nifi-nar-bundles/nifi-framework-bundle/nifi-framework/nifi-web/nifi-web-api/src/main/java/org/apache/nifi/web/api/SnippetResource.java
---
@@ -0,0
Github user markap14 commented on the pull request:
https://github.com/apache/nifi/pull/461#issuecomment-221046258
@mcgilman looks good for the most part. I left some comments in-line (most
of which you've already addressed before I finished the review even!)
Excellent. Once the
Github user markap14 commented on a diff in the pull request:
https://github.com/apache/nifi/pull/461#discussion_r64259242
--- Diff:
nifi-nar-bundles/nifi-framework-bundle/nifi-framework/nifi-web/nifi-web-api/src/main/java/org/apache/nifi/web/api/SnippetResource.java
---
@@ -0,0
Github user markap14 commented on the pull request:
https://github.com/apache/nifi/pull/324#issuecomment-221935156
Agreed - I think moving the FlowController lifecycle to @Before and @After
should address the issues. I do think that the
validateIdempotencyOfProcessorStartOperation
Github user markap14 commented on the pull request:
https://github.com/apache/nifi/pull/469#issuecomment-221951022
+1 looks good to me.
---
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
Github user markap14 commented on the pull request:
https://github.com/apache/nifi/pull/470#issuecomment-221954497
+1 looks good!
---
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
GitHub user markap14 opened a pull request:
https://github.com/apache/nifi/pull/488
NIFI-1897: Refactoring to allow requests to be replicated from a node to
other nodes
Nodes are now capable of replicating requests across the cluster and a
Cluster Coordinator is auto-elected to
Github user markap14 commented on the issue:
https://github.com/apache/nifi/pull/472
@mcgilman I merged this to master but forgot to update the commit message.
Can you close the PR?
---
If your project is set up for it, you can reply to this email and have your
reply appear on
Github user markap14 commented on the issue:
https://github.com/apache/nifi/pull/488
Modifications that you proposed look good to me. +1
---
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
GitHub user markap14 opened a pull request:
https://github.com/apache/nifi/pull/491
NIFI-1960: Update admin guide regarding documentation for clustering
You can merge this pull request into a Git repository by running:
$ git pull https://github.com/markap14/nifi NIFI-1960
Github user markap14 commented on the issue:
https://github.com/apache/nifi/pull/303
@ckmcd an example would be something like:
${ hello:jsonPath( '$.${elementOfInterest}.items[0]') }
Here, we are referencing an attribute within the JSON Path, so it is not
Github user markap14 commented on a diff in the pull request:
https://github.com/apache/nifi/pull/491#discussion_r65755030
--- Diff: nifi-docs/src/main/asciidoc/administration-guide.adoc ---
@@ -485,98 +485,137 @@ It is preferable to request upstream/downstream
systems to switch
Github user markap14 commented on a diff in the pull request:
https://github.com/apache/nifi/pull/491#discussion_r65755455
--- Diff: nifi-docs/src/main/asciidoc/administration-guide.adoc ---
@@ -485,98 +485,137 @@ It is preferable to request upstream/downstream
systems to switch
Github user markap14 commented on a diff in the pull request:
https://github.com/apache/nifi/pull/491#discussion_r65755845
--- Diff: nifi-docs/src/main/asciidoc/administration-guide.adoc ---
@@ -485,98 +485,137 @@ It is preferable to request upstream/downstream
systems to switch
Github user markap14 commented on a diff in the pull request:
https://github.com/apache/nifi/pull/491#discussion_r65756026
--- Diff: nifi-docs/src/main/asciidoc/administration-guide.adoc ---
@@ -485,98 +485,137 @@ It is preferable to request upstream/downstream
systems to switch
Github user markap14 commented on a diff in the pull request:
https://github.com/apache/nifi/pull/491#discussion_r65757147
--- Diff: nifi-docs/src/main/asciidoc/administration-guide.adoc ---
@@ -1351,66 +1389,81 @@ in the file specified in
Github user markap14 commented on the issue:
https://github.com/apache/nifi/pull/458
I'm having trouble understanding the naming convention here. "DebugFlow"
would indicate that it is debugging the flow that you are putting together, but
the comment states "Create
Github user markap14 closed the pull request at:
https://github.com/apache/nifi/pull/491
---
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
Github user markap14 commented on the issue:
https://github.com/apache/nifi/pull/491
Thanks, @alopresto. I have merged to master. I am sure that we will end up
updating this a few more times before 1.0.0 is released, but at least I wanted
to get this into master so that those trying
GitHub user markap14 opened a pull request:
https://github.com/apache/nifi/pull/499
NIFI-1052: Added "Ghost" Processors, Reporting Tasks, Controller Services
If we try to create a component for which the NAR is missing, we previously
would throw an Exception that would
Github user markap14 commented on the issue:
https://github.com/apache/nifi/pull/497
@ijokarumawak I can't seem to get this to work properly. I configured nifi
to allow http site-to-site only, not secure. I then connected a
GenerateFlowFile processor to a Remote Process Group
Github user markap14 commented on the issue:
https://github.com/apache/nifi/pull/303
@ckmcd looks good! I was able to test it locally. Unit tests are good. I
did find that a Reader was opened in a unit test and not closed, which i
addressed, and a couple of NOTICE files needed to be
Github user markap14 commented on the issue:
https://github.com/apache/nifi/pull/506
Hey @ckmcd I just realized that you had a different PR for the 0.x branch.
I ended up just creating a patch from PR 303 and applying that to the 0.x
baseline. So this should all be taken care of. Do
Github user markap14 commented on a diff in the pull request:
https://github.com/apache/nifi/pull/501#discussion_r66142774
--- Diff:
nifi-api/src/main/java/org/apache/nifi/registry/FileVariableRegistry.java ---
@@ -0,0 +1,70 @@
+/*
+ * Licensed to the Apache Software
Github user markap14 commented on a diff in the pull request:
https://github.com/apache/nifi/pull/501#discussion_r66142811
--- Diff:
nifi-api/src/main/java/org/apache/nifi/registry/FileVariableRegistry.java ---
@@ -0,0 +1,70 @@
+/*
+ * Licensed to the Apache Software
Github user markap14 commented on a diff in the pull request:
https://github.com/apache/nifi/pull/501#discussion_r66143013
--- Diff:
nifi-api/src/main/java/org/apache/nifi/registry/FileVariableRegistry.java ---
@@ -0,0 +1,70 @@
+/*
+ * Licensed to the Apache Software
Github user markap14 commented on a diff in the pull request:
https://github.com/apache/nifi/pull/501#discussion_r66143984
--- Diff:
nifi-api/src/main/java/org/apache/nifi/registry/FileVariableRegistry.java ---
@@ -0,0 +1,70 @@
+/*
+ * Licensed to the Apache Software
Github user markap14 commented on a diff in the pull request:
https://github.com/apache/nifi/pull/501#discussion_r66144229
--- Diff: nifi-api/src/main/java/org/apache/nifi/registry/MultiMap.java ---
@@ -0,0 +1,154 @@
+/*
+ * Licensed to the Apache Software Foundation (ASF
Github user markap14 commented on a diff in the pull request:
https://github.com/apache/nifi/pull/501#discussion_r66145081
--- Diff: nifi-api/src/main/java/org/apache/nifi/registry/MultiMap.java ---
@@ -0,0 +1,154 @@
+/*
+ * Licensed to the Apache Software Foundation (ASF
Github user markap14 commented on a diff in the pull request:
https://github.com/apache/nifi/pull/501#discussion_r66145563
--- Diff: nifi-api/src/main/java/org/apache/nifi/registry/MultiMap.java ---
@@ -0,0 +1,154 @@
+/*
+ * Licensed to the Apache Software Foundation (ASF
1 - 100 of 421 matches
Mail list logo