Github user pvillard31 commented on the issue:
https://github.com/apache/nifi/pull/502
Hey @mans2singh, I'm really busy this week and I won't be able to have a
look before next week :(
In the meantime, you can fix the conflicts against master. And maybe @apiri
will hav
Github user pvillard31 commented on a diff in the pull request:
https://github.com/apache/nifi/pull/599#discussion_r69217243
--- Diff:
nifi-nar-bundles/nifi-standard-bundle/nifi-standard-processors/src/main/java/org/apache/nifi/processors/standard/ListDatabaseTables.java
Github user pvillard31 commented on a diff in the pull request:
https://github.com/apache/nifi/pull/599#discussion_r69216929
--- Diff:
nifi-nar-bundles/nifi-standard-bundle/nifi-standard-processors/src/test/java/org/apache/nifi/processors/standard/TestListDatabaseTables.java
Github user pvillard31 commented on the issue:
https://github.com/apache/nifi/pull/502
Hi @mans2singh, I didn't find time to give it a new try, but I hope I will
this week.
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as
Github user pvillard31 commented on a diff in the pull request:
https://github.com/apache/nifi/pull/534#discussion_r68430574
--- Diff:
nifi-nar-bundles/nifi-scripting-bundle/nifi-scripting-processors/src/main/java/org/apache/nifi/processors/script/InvokeScriptedProcessor.java
Github user pvillard31 commented on the issue:
https://github.com/apache/nifi/pull/379
Thanks for the review @apiri!
I still need to take into account some of your comments but I rebased the
PR against master. While doing some additional testing, for a reason I can't
explai
Github user pvillard31 commented on a diff in the pull request:
https://github.com/apache/nifi/pull/379#discussion_r67956841
--- Diff:
nifi-nar-bundles/nifi-alluxio-bundle/nifi-alluxio-processors/src/main/java/org/apache/nifi/processors/alluxio/AbstractAlluxioProcessor.java
Github user pvillard31 commented on a diff in the pull request:
https://github.com/apache/nifi/pull/379#discussion_r67956421
--- Diff:
nifi-nar-bundles/nifi-alluxio-bundle/nifi-alluxio-processors/src/main/java/org/apache/nifi/processors/alluxio/GetAlluxio.java
---
@@ -0,0 +1,184
Github user pvillard31 commented on a diff in the pull request:
https://github.com/apache/nifi/pull/379#discussion_r67956263
--- Diff:
nifi-nar-bundles/nifi-alluxio-bundle/nifi-alluxio-processors/src/main/java/org/apache/nifi/processors/alluxio/AbstractAlluxioProcessor.java
Github user pvillard31 commented on a diff in the pull request:
https://github.com/apache/nifi/pull/379#discussion_r67956202
--- Diff:
nifi-nar-bundles/nifi-alluxio-bundle/nifi-alluxio-processors/src/main/java/org/apache/nifi/processors/alluxio/AbstractAlluxioProcessor.java
Github user pvillard31 commented on the issue:
https://github.com/apache/nifi/pull/551
Merged in master, 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 project does not have this feature
enabled and
GitHub user pvillard31 opened a pull request:
https://github.com/apache/nifi/pull/552
NIFI-2066 dynamic port allocation in SNMP processors unit tests
You can merge this pull request into a Git repository by running:
$ git pull https://github.com/pvillard31/nifi NIFI-2066
Github user pvillard31 commented on a diff in the pull request:
https://github.com/apache/nifi/pull/496#discussion_r67826945
--- Diff:
nifi-nar-bundles/nifi-enrich-bundle/nifi-enrich-processors/src/test/java/org/apache/nifi/processors/TestQueryDNS.java
---
@@ -0,0 +1,164
Github user pvillard31 commented on a diff in the pull request:
https://github.com/apache/nifi/pull/496#discussion_r67825492
--- Diff:
nifi-nar-bundles/nifi-enrich-bundle/nifi-enrich-processors/src/main/java/org/apache/nifi/processors/AbstractEnrichProcessor.java
---
@@ -0,0
Github user pvillard31 commented on a diff in the pull request:
https://github.com/apache/nifi/pull/534#discussion_r67825033
--- Diff:
nifi-mock/src/main/java/org/apache/nifi/util/MockProcessSession.java ---
@@ -793,13 +798,21 @@ private void validateState(final FlowFile flowFile
Github user pvillard31 commented on a diff in the pull request:
https://github.com/apache/nifi/pull/534#discussion_r67824794
--- Diff:
nifi-mock/src/test/java/org/apache/nifi/util/TestMockProcessSession.java ---
@@ -36,7 +36,7 @@ public void testPenalizeFlowFileFromProcessor
GitHub user pvillard31 opened a pull request:
https://github.com/apache/nifi/pull/547
NIFI-2060 - fixed unit tests to be compatible with 2.7.x
Itâs a shame to limit what is tested here (in terms of event types) but
it seems to be the only way to have both 2.6 and 2.7 lines
Github user pvillard31 commented on the issue:
https://github.com/apache/nifi/pull/398
Yep, not sure it's worth it. I think it is fine to let the PR as is.
---
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 pr
Github user pvillard31 commented on the issue:
https://github.com/apache/nifi/pull/398
Hey @apiri,
Many thanks for the review!
Regarding HTTPS, at the time I submitted the PR I was testing it with
CCProxy (on Windows computer). I can't test it again with this
Github user pvillard31 commented on the issue:
https://github.com/apache/nifi/pull/545
Thanks @olegz! I just merged it into 0.x, could 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 as well. If your project
Github user pvillard31 commented on the issue:
https://github.com/apache/nifi/pull/534
thanks @PuspenduBanerjee, I'm quite busy this WE, will have another look
asap
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as wel
Github user pvillard31 commented on the issue:
https://github.com/apache/nifi/pull/536
Thanks @JPercivall! I didn't realize that...
---
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 pvillard31 commented on the issue:
https://github.com/apache/nifi/pull/536
@jjmeyer0 Thanks! I just merged it in 0.x branch.
---
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 pvillard31 closed the pull request at:
https://github.com/apache/nifi/pull/401
---
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 pvillard31 commented on the issue:
https://github.com/apache/nifi/pull/493
@jjmeyer0 LGTM +1 tested with workflows, mvn build OK.
Merged in master, but would you mind opening a specific PR for 0.x branch
(Java 7 compatible):
GetHDFSEvents.java:[208,78] error: local
Github user pvillard31 commented on a diff in the pull request:
https://github.com/apache/nifi/pull/534#discussion_r67428396
--- Diff:
nifi-nar-bundles/nifi-scripting-bundle/nifi-scripting-processors/src/main/java/org/apache/nifi/processors/script/InvokeScriptedProcessor.java
Github user pvillard31 commented on a diff in the pull request:
https://github.com/apache/nifi/pull/534#discussion_r67424962
--- Diff:
nifi-mock/src/test/java/org/apache/nifi/util/TestMockProcessSession.java ---
@@ -55,4 +55,35 @@ public void onTrigger(final ProcessContext ctx
Github user pvillard31 commented on a diff in the pull request:
https://github.com/apache/nifi/pull/534#discussion_r67423663
--- Diff:
nifi-mock/src/main/java/org/apache/nifi/util/MockProcessSession.java ---
@@ -793,13 +798,21 @@ private void validateState(final FlowFile flowFile
Github user pvillard31 commented on a diff in the pull request:
https://github.com/apache/nifi/pull/534#discussion_r67392290
--- Diff:
nifi-mock/src/test/java/org/apache/nifi/util/TestMockProcessSession.java ---
@@ -55,4 +55,35 @@ public void onTrigger(final ProcessContext ctx
Github user pvillard31 commented on a diff in the pull request:
https://github.com/apache/nifi/pull/534#discussion_r67391289
--- Diff:
nifi-mock/src/test/java/org/apache/nifi/util/TestMockProcessSession.java ---
@@ -36,7 +36,7 @@ public void testPenalizeFlowFileFromProcessor
Github user pvillard31 commented on a diff in the pull request:
https://github.com/apache/nifi/pull/534#discussion_r67390898
--- Diff:
nifi-mock/src/test/java/org/apache/nifi/util/TestMockProcessSession.java ---
@@ -16,6 +16,9 @@
*/
package org.apache.nifi.util
Github user pvillard31 commented on a diff in the pull request:
https://github.com/apache/nifi/pull/534#discussion_r67390752
--- Diff:
nifi-mock/src/main/java/org/apache/nifi/util/MockProcessSession.java ---
@@ -793,13 +798,21 @@ private void validateState(final FlowFile flowFile
Github user pvillard31 commented on a diff in the pull request:
https://github.com/apache/nifi/pull/534#discussion_r67390513
--- Diff:
nifi-mock/src/main/java/org/apache/nifi/util/MockProcessSession.java ---
@@ -587,13 +589,15 @@ public void transfer(final FlowFile flowFile, final
Github user pvillard31 commented on the issue:
https://github.com/apache/nifi/pull/272
It should be OK, let me know if not.
---
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 pvillard31 commented on the issue:
https://github.com/apache/nifi/pull/257
@olegz half-time ;) done!
---
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
Github user pvillard31 commented on the issue:
https://github.com/apache/nifi/pull/527
@olegz It seems it is in conflict with 0.x branch, could you check if I am
wrong and if not submit a PR for 0.x?
---
If your project is set up for it, you can reply to this email and have your
Github user pvillard31 commented on the issue:
https://github.com/apache/nifi/pull/527
LGTM +1. Do you mind fixing the import auto-ordering?
---
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 pvillard31 commented on the issue:
https://github.com/apache/nifi/pull/502
@mans2singh
Just had a new look and ran a workflow to test the processor. Few remarks
even though I am not familiar with Ignite... First, I am not sure to see how I
can check the content of
Github user pvillard31 commented on the issue:
https://github.com/apache/nifi/pull/521
LGTM +1. Merged both in 0.x and master.
---
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 pvillard31 commented on the issue:
https://github.com/apache/nifi/pull/521
reviewing...
---
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
Github user pvillard31 commented on the issue:
https://github.com/apache/nifi/pull/493
Your changes look good to me. Let me know if you want to have a look to
enable EL/RegEx on HDFS_PATH_TO_WATCH property (ReplaceText processor seems to
accept both regex and expression language on
Github user pvillard31 commented on the issue:
https://github.com/apache/nifi/pull/502
Thanks @mans2singh ! I'll have a look asap (probably on Sunday or next
week). Regarding the provenance event, I think we could have a SEND event for
each flow file successfully sent to I
Github user pvillard31 commented on the issue:
https://github.com/apache/nifi/pull/493
@jjmeyer0 Yes you're absolutely right, I was not suggesting to have the
three options together but rather think about what could be done to give more
flexibility. I think there is a processor
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
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
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 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 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 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 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 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 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 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
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 user pvillard31 commented on the issue:
https://github.com/apache/nifi/pull/493
I have played with it and it works great.
One remark: I am wondering if the 'HDFS_PATH_TO_WATCH' property should be
improved:
- accept the expression language to handle ti
Github user pvillard31 commented on the issue:
https://github.com/apache/nifi/pull/257
@trixpan @olegz @michalklempa
I rebased the PR against master
---
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 pvillard31 commented on the issue:
https://github.com/apache/nifi/pull/493
Once comments are addressed, I'll have another look and give it a try but
it looks good to me. It's a nice feature!
---
If your project is set up for it, you can reply to this email and
Github user pvillard31 commented on a diff in the pull request:
https://github.com/apache/nifi/pull/493#discussion_r65967740
--- Diff:
nifi-nar-bundles/nifi-hadoop-bundle/nifi-hdfs-processors/src/main/java/org/apache/nifi/processors/hadoop/inotify/GetHDFSEvents.java
---
@@ -0,0
Github user pvillard31 commented on a diff in the pull request:
https://github.com/apache/nifi/pull/493#discussion_r65966688
--- Diff:
nifi-nar-bundles/nifi-hadoop-bundle/nifi-hdfs-processors/src/main/java/org/apache/nifi/processors/hadoop/inotify/GetHDFSEvents.java
---
@@ -0,0
Github user pvillard31 commented on a diff in the pull request:
https://github.com/apache/nifi/pull/493#discussion_r65961941
--- Diff:
nifi-nar-bundles/nifi-hadoop-bundle/nifi-hdfs-processors/src/main/java/org/apache/nifi/processors/hadoop/inotify/GetHDFSEvents.java
---
@@ -0,0
Github user pvillard31 commented on a diff in the pull request:
https://github.com/apache/nifi/pull/493#discussion_r65957598
--- Diff:
nifi-nar-bundles/nifi-hadoop-bundle/nifi-hdfs-processors/src/main/java/org/apache/nifi/processors/hadoop/inotify/EventAttributes.java
---
@@ -0,0
Github user pvillard31 commented on the issue:
https://github.com/apache/nifi/pull/487
Agree. Merging...
---
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
Github user pvillard31 commented on a diff in the pull request:
https://github.com/apache/nifi/pull/493#discussion_r65879551
--- Diff:
nifi-nar-bundles/nifi-hadoop-bundle/nifi-hdfs-processors/src/main/java/org/apache/nifi/processors/hadoop/inotify/EventAttributes.java
---
@@ -0,0
Github user pvillard31 commented on a diff in the pull request:
https://github.com/apache/nifi/pull/493#discussion_r65878804
--- Diff:
nifi-nar-bundles/nifi-hadoop-bundle/nifi-hdfs-processors/src/main/java/org/apache/nifi/processors/hadoop/inotify/GetHDFSEvents.java
---
@@ -0,0
Github user pvillard31 commented on a diff in the pull request:
https://github.com/apache/nifi/pull/493#discussion_r65878313
--- Diff:
nifi-nar-bundles/nifi-hadoop-bundle/nifi-hdfs-processors/src/main/java/org/apache/nifi/processors/hadoop/inotify/GetHDFSEvents.java
---
@@ -0,0
Github user pvillard31 commented on a diff in the pull request:
https://github.com/apache/nifi/pull/493#discussion_r65878170
--- Diff:
nifi-nar-bundles/nifi-hadoop-bundle/nifi-hdfs-processors/src/main/java/org/apache/nifi/processors/hadoop/inotify/GetHDFSEvents.java
---
@@ -0,0
Github user pvillard31 commented on the issue:
https://github.com/apache/nifi/pull/486
+1 @mattyb149
merged in master and 0.x
---
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 pvillard31 commented on the issue:
https://github.com/apache/nifi/pull/487
@olegz
Just to be sure before merging in: do you plan to add a unit test?
My guess is that it is not possible to simulate this server-side behavior,
but I want to confirm.
---
If your
Github user pvillard31 commented on the issue:
https://github.com/apache/nifi/pull/492
I am running out of time right now, but will have a deeper look later and
give it a try asap. As a general comment, I'd say, even if I know how boring it
can be, that some comments in the
Github user pvillard31 commented on a diff in the pull request:
https://github.com/apache/nifi/pull/492#discussion_r65755925
--- Diff:
nifi-nar-bundles/nifi-evtx-bundle/nifi-evtx-processors/src/main/java/org/apache/nifi/processors/evtx/ParseEvtx.java
---
@@ -0,0 +1,352
Github user pvillard31 commented on a diff in the pull request:
https://github.com/apache/nifi/pull/492#discussion_r65755477
--- Diff:
nifi-nar-bundles/nifi-evtx-bundle/nifi-evtx-processors/src/main/java/org/apache/nifi/processors/evtx/ParseEvtx.java
---
@@ -0,0 +1,352
Github user pvillard31 commented on the issue:
https://github.com/apache/nifi/pull/490
It remains an issue I have not been able to solve at this point: the case
where the new file contains more data than the tailed file before being moved.
In this case we ends up in the same
GitHub user pvillard31 opened a pull request:
https://github.com/apache/nifi/pull/490
NIFI-1959 Added length and timestamp to detect rollover
It improves robustness of the processor in case the property âRolling
filename patternâ is not set and the tailed file is moved for some
Github user pvillard31 commented on the issue:
https://github.com/apache/nifi/pull/481
Good catch. Will merge.
However not sure why the travis build failed...
---
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
Github user pvillard31 commented on the pull request:
https://github.com/apache/nifi/pull/466
LGTM +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 feature
enabled and wishes so, or
GitHub user pvillard31 opened a pull request:
https://github.com/apache/nifi/pull/476
NIFI-1942 Processor to validate CSV against user-supplied schema
This processor is designed to validate a CSV formatted FlowFile against a
user-supplied schema.
It leverages Cell
Github user pvillard31 commented on the pull request:
https://github.com/apache/nifi/pull/462#issuecomment-221058590
There is a double space on line 160 in test class. Travis seems not to be
run on 0.x branch so I don't know if contrib check would fail because of that
(c
Github user pvillard31 commented on the pull request:
https://github.com/apache/nifi/pull/403#issuecomment-220421867
No problem @apiri. I agree: the sooner the better. Let me know if you need
something from my side.
---
If your project is set up for it, you can reply to this email
Github user pvillard31 commented on the pull request:
https://github.com/apache/nifi/pull/443#issuecomment-219855457
Maven build with contrib-check OK.
Reviewed and it LGTM, nice unit testing.
Tested with two different flows and it is working as expected.
I'm a +1
-
Github user pvillard31 commented on the pull request:
https://github.com/apache/nifi/pull/366#issuecomment-219830951
Reviewed and tested. LGTM. All tested flows were OK.
Few minor remarks:
- I'd update @CapabilityDescription to help users differentiate Publish/Put
Github user pvillard31 closed the pull request at:
https://github.com/apache/nifi/pull/451
---
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 pvillard31 opened a pull request:
https://github.com/apache/nifi/pull/451
NIFI-1213 (branch 0.x) Added the possibility to register FlowFile assâ¦
â¦ertions in mock framework
You can merge this pull request into a Git repository by running:
$ git pull https
Github user pvillard31 commented on a diff in the pull request:
https://github.com/apache/nifi/pull/439#discussion_r63429069
--- Diff:
nifi-nar-bundles/nifi-framework-bundle/nifi-framework/nifi-framework-core/src/main/java/org/apache/nifi/controller/repository
Github user pvillard31 commented on a diff in the pull request:
https://github.com/apache/nifi/pull/439#discussion_r63423341
--- Diff:
nifi-nar-bundles/nifi-framework-bundle/nifi-framework/nifi-framework-core/src/main/java/org/apache/nifi/controller/repository
Github user pvillard31 commented on the pull request:
https://github.com/apache/nifi/pull/404#issuecomment-219539616
Thanks @olegz
PR updated
---
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
Github user pvillard31 closed the pull request at:
https://github.com/apache/nifi/pull/438
---
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 pvillard31 opened a pull request:
https://github.com/apache/nifi/pull/439
NIFI-1866 ProcessException handling in StandardProcessSession
You can merge this pull request into a Git repository by running:
$ git pull https://github.com/pvillard31/nifi NIFI-1866
Github user pvillard31 commented on the pull request:
https://github.com/apache/nifi/pull/421#issuecomment-218876809
+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 feature
enabled
GitHub user pvillard31 opened a pull request:
https://github.com/apache/nifi/pull/438
NIFI-1755 (0.x branch) Fixed remote process group status counts by onâ¦
â¦ly considering connected remote ports
You can merge this pull request into a Git repository by running:
$ git pull
Github user pvillard31 commented on the pull request:
https://github.com/apache/nifi/pull/347#issuecomment-218867090
@olegz Thanks Oleg! Not exactly what I had in mind but it does the trick :)
(I was trying to set up an environment with two simple processors sending flow
files to RPG
GitHub user pvillard31 opened a pull request:
https://github.com/apache/nifi/pull/428
NIFI-1863 extended caught exception in cleaning task for HTTPContextMap
You can merge this pull request into a Git repository by running:
$ git pull https://github.com/pvillard31/nifi NIFI
Github user pvillard31 commented on the pull request:
https://github.com/apache/nifi/pull/389#issuecomment-217620180
@JPercivall Done. Let me know if I missed something.
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If
Github user pvillard31 commented on a diff in the pull request:
https://github.com/apache/nifi/pull/382#discussion_r62359879
--- Diff:
nifi-nar-bundles/nifi-standard-bundle/nifi-standard-processors/src/main/java/org/apache/nifi/processors/standard/HandleHttpResponse.java
Github user pvillard31 commented on a diff in the pull request:
https://github.com/apache/nifi/pull/382#discussion_r62359651
--- Diff:
nifi-nar-bundles/nifi-standard-bundle/nifi-standard-processors/src/main/java/org/apache/nifi/processors/standard/HandleHttpResponse.java
Github user pvillard31 commented on the pull request:
https://github.com/apache/nifi/pull/337#issuecomment-217500797
Thanks @JPercivall
Few remarks:
- when there is a timeout, what I observed is a request with a dispatcher
type set to ERROR (I've never seen ASYNC s
Github user pvillard31 commented on the pull request:
https://github.com/apache/nifi/pull/366#issuecomment-216917150
@olegz No problem ;) I was quite in a hurry and I understand why it is not
clear at all.
You are correct. By dumping topic content with Kafka CLI, I can
Github user pvillard31 commented on the pull request:
https://github.com/apache/nifi/pull/366#issuecomment-216842191
I just played a little bit with the processors and here are some remarks:
- Cosmetic remarks
1. add the duration in provenance events
2. add the number
Github user pvillard31 commented on the pull request:
https://github.com/apache/nifi/pull/347#issuecomment-216631894
@olegz I must admit I tried to use your code as mentioned in your link and
got it working but I didn't find a way to validate the change since it seems to
simulat
Github user pvillard31 closed the pull request at:
https://github.com/apache/nifi/pull/390
---
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 pvillard31 opened a pull request:
https://github.com/apache/nifi/pull/409
NIFI-1840 Added compression type property in Kite processors
You can merge this pull request into a Git repository by running:
$ git pull https://github.com/pvillard31/nifi NIFI-1840
1 - 100 of 191 matches
Mail list logo
