GitHub user brosander opened a pull request:
https://github.com/apache/nifi-minifi/pull/26
MINIFI-48 - Omitting queuePrioritizerClass from flow if it isn't set â¦
â¦in schema
You can merge this pull request into a Git repository by running:
$ git pull https://github.com
Github user brosander commented on a diff in the pull request:
https://github.com/apache/nifi-minifi/pull/21#discussion_r69346935
--- Diff:
minifi-toolkit/minifi-toolkit-configuration/src/test/resources/Splunk_Processors_with_Listen_TCP.xml
---
@@ -0,0 +1,18
Github user brosander commented on a diff in the pull request:
https://github.com/apache/nifi-minifi/pull/21#discussion_r69183488
--- Diff:
minifi-commons/minifi-commons-schema/src/main/java/org/apache/nifi/minifi/commons/schema/BaseSchema.java
---
@@ -15,13 +15,17
Github user brosander commented on a diff in the pull request:
https://github.com/apache/nifi-minifi/pull/21#discussion_r69177842
--- Diff:
minifi-commons/minifi-commons-schema/src/main/java/org/apache/nifi/minifi/commons/schema/BaseSchema.java
---
@@ -15,13 +15,17
Github user brosander commented on a diff in the pull request:
https://github.com/apache/nifi-minifi/pull/21#discussion_r69166911
--- Diff:
minifi-commons/minifi-commons-schema/src/main/java/org/apache/nifi/minifi/commons/schema/BaseSchema.java
---
@@ -15,13 +15,17
Github user brosander commented on a diff in the pull request:
https://github.com/apache/nifi-minifi/pull/21#discussion_r69165446
--- Diff: minifi-toolkit/minifi-toolkit-assembly/NOTICE ---
@@ -0,0 +1,21 @@
+Apache NiFi
+Copyright 2014-2016 The Apache Software Foundation
Github user brosander commented on a diff in the pull request:
https://github.com/apache/nifi-minifi/pull/21#discussion_r69164484
--- Diff: minifi-commons/minifi-commons-schema/pom.xml ---
@@ -0,0 +1,46 @@
+
+
+http://maven.apache.org/POM/4.0.0;
xmlns:xsi="
Github user brosander commented on a diff in the pull request:
https://github.com/apache/nifi/pull/525#discussion_r68990802
--- Diff:
nifi-nar-bundles/nifi-windows-event-log-bundle/nifi-windows-event-log-processors/src/main/java/org/apache/nifi/processors/windows/event/log
Github user brosander commented on a diff in the pull request:
https://github.com/apache/nifi/pull/525#discussion_r68948186
--- Diff:
nifi-nar-bundles/nifi-windows-event-log-bundle/nifi-windows-event-log-processors/src/main/java/org/apache/nifi/processors/windows/event/log
Github user brosander commented on a diff in the pull request:
https://github.com/apache/nifi/pull/525#discussion_r68795836
--- Diff:
nifi-nar-bundles/nifi-windows-event-log-bundle/nifi-windows-event-log-processors/src/main/java/org/apache/nifi/processors/windows/event/log
Github user brosander commented on a diff in the pull request:
https://github.com/apache/nifi/pull/525#discussion_r68795669
--- Diff:
nifi-nar-bundles/nifi-windows-event-log-bundle/nifi-windows-event-log-processors/src/main/java/org/apache/nifi/processors/windows/event/log
Github user brosander commented on a diff in the pull request:
https://github.com/apache/nifi/pull/525#discussion_r68795163
--- Diff:
nifi-nar-bundles/nifi-windows-event-log-bundle/nifi-windows-event-log-processors/src/main/java/org/apache/nifi/processors/windows/event/log
Github user brosander commented on a diff in the pull request:
https://github.com/apache/nifi/pull/525#discussion_r68782625
--- Diff:
nifi-nar-bundles/nifi-windows-event-log-bundle/nifi-windows-event-log-processors/src/main/java/org/apache/nifi/processors/windows/event/log
Github user brosander commented on a diff in the pull request:
https://github.com/apache/nifi/pull/525#discussion_r68759636
--- Diff:
nifi-nar-bundles/nifi-windows-event-log-bundle/nifi-windows-event-log-processors/src/main/java/org/apache/nifi/processors/windows/event/log
Github user brosander commented on the issue:
https://github.com/apache/nifi/pull/525
@JPercivall just pushed change to update duplicate logging in the same way
as
https://github.com/apache/nifi/pull/548/files#diff-a88d41c9ef1b881c177706d4cc4b0bf1R294
---
If your project is set up
Github user brosander closed the pull request at:
https://github.com/apache/nifi-minifi/pull/19
---
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 brosander opened a pull request:
https://github.com/apache/nifi/pull/549
NIFI-2046 - Adding reference to @TriggerSerially in Processor javadoc
You can merge this pull request into a Git repository by running:
$ git pull https://github.com/brosander/nifi NIFI-2046
GitHub user brosander opened a pull request:
https://github.com/apache/nifi/pull/548
NIFI-2045 - Removing mqtt message from queue after after session.comit()
You can merge this pull request into a Git repository by running:
$ git pull https://github.com/brosander/nifi NIFI
Github user brosander commented on the issue:
https://github.com/apache/nifi/pull/525
@JPercivall I believe I have incorporated all of the suggested changes
(except for the XPath validator because it didn't like some of the provided
examples in the api doc). Mind taking another look
GitHub user brosander opened a pull request:
https://github.com/apache/nifi-minifi/pull/19
MINIFI-40 - Defaulting to empty string for keystore, truststore, sslPâ¦
â¦rotocol
You can merge this pull request into a Git repository by running:
$ git pull https://github.com
Github user brosander commented on a diff in the pull request:
https://github.com/apache/nifi/pull/525#discussion_r67514815
--- Diff:
nifi-nar-bundles/nifi-windows-event-log-bundle/nifi-windows-event-log-processors/src/test/java/org/apache/nifi/processors/windows/event/log
Github user brosander commented on a diff in the pull request:
https://github.com/apache/nifi/pull/525#discussion_r67514549
--- Diff:
nifi-nar-bundles/nifi-windows-event-log-bundle/nifi-windows-event-log-processors/src/main/java/org/apache/nifi/processors/windows/event/log
Github user brosander commented on a diff in the pull request:
https://github.com/apache/nifi/pull/525#discussion_r67514326
--- Diff:
nifi-nar-bundles/nifi-windows-event-log-bundle/nifi-windows-event-log-processors/src/main/java/org/apache/nifi/processors/windows/event/log/jna
Github user brosander commented on a diff in the pull request:
https://github.com/apache/nifi/pull/525#discussion_r67513710
--- Diff:
nifi-nar-bundles/nifi-windows-event-log-bundle/nifi-windows-event-log-processors/src/main/java/org/apache/nifi/processors/windows/event/log
Github user brosander commented on a diff in the pull request:
https://github.com/apache/nifi/pull/525#discussion_r67511428
--- Diff:
nifi-nar-bundles/nifi-windows-event-log-bundle/nifi-windows-event-log-processors/src/main/java/org/apache/nifi/processors/windows/event/log
Github user brosander commented on a diff in the pull request:
https://github.com/apache/nifi/pull/525#discussion_r67511182
--- Diff:
nifi-nar-bundles/nifi-windows-event-log-bundle/nifi-windows-event-log-processors/src/main/java/org/apache/nifi/processors/windows/event/log
Github user brosander commented on a diff in the pull request:
https://github.com/apache/nifi/pull/525#discussion_r67509411
--- Diff:
nifi-nar-bundles/nifi-windows-event-log-bundle/nifi-windows-event-log-nar/src/main/resources/META-INF/LICENSE
---
@@ -0,0 +1,239
GitHub user brosander opened a pull request:
https://github.com/apache/nifi/pull/525
NIFI-1976 - Windows Event Log native processor
You can merge this pull request into a Git repository by running:
$ git pull https://github.com/brosander/nifi NIFI-1976-rebase
Alternatively
Github user brosander commented on the issue:
https://github.com/apache/nifi/pull/492
changes merged upstream
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and wishes
Github user brosander closed the pull request at:
https://github.com/apache/nifi/pull/492
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and wishes so, or if the feature
Github user brosander commented on a diff in the pull request:
https://github.com/apache/nifi/pull/492#discussion_r66348434
--- Diff:
nifi-nar-bundles/nifi-evtx-bundle/nifi-evtx-processors/src/test/java/org/apache/nifi/processors/evtx/ParseEvtxTest.java
---
@@ -0,0 +1,481
Github user brosander commented on the issue:
https://github.com/apache/nifi/pull/492
@mattyb149 updated those poms, verified that the nar is in the assembly
---
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 brosander commented on the issue:
https://github.com/apache/nifi/pull/492
@mattyb149 I think I've addressed your suggestions, please let me know if
further changes are needed
---
If your project is set up for it, you can reply to this email and have your
reply appear
Github user brosander commented on a diff in the pull request:
https://github.com/apache/nifi/pull/492#discussion_r66007647
--- Diff:
nifi-nar-bundles/nifi-evtx-bundle/nifi-evtx-processors/src/main/java/org/apache/nifi/processors/evtx/ParseEvtx.java
---
@@ -0,0 +1,353
Github user brosander commented on a diff in the pull request:
https://github.com/apache/nifi/pull/492#discussion_r66007547
--- Diff:
nifi-nar-bundles/nifi-evtx-bundle/nifi-evtx-processors/src/main/java/org/apache/nifi/processors/evtx/ParseEvtx.java
---
@@ -0,0 +1,353
Github user brosander commented on a diff in the pull request:
https://github.com/apache/nifi/pull/492#discussion_r66007477
--- Diff:
nifi-nar-bundles/nifi-evtx-bundle/nifi-evtx-processors/src/main/java/org/apache/nifi/processors/evtx/ParseEvtx.java
---
@@ -0,0 +1,353
Github user brosander commented on a diff in the pull request:
https://github.com/apache/nifi/pull/492#discussion_r66007396
--- Diff: nifi-nar-bundles/nifi-evtx-bundle/nifi-evtx-processors/pom.xml ---
@@ -0,0 +1,68 @@
+
+
+http://maven.apache.org/POM/4.0.0;
xmlns:xsi
Github user brosander commented on the issue:
https://github.com/apache/nifi/pull/492
Added the notice to the root and assembly, updated it in the nar
Added some comments to the code
---
If your project is set up for it, you can reply to this email and have your
reply appear
Github user brosander commented on a diff in the pull request:
https://github.com/apache/nifi/pull/492#discussion_r65769603
--- Diff:
nifi-nar-bundles/nifi-evtx-bundle/nifi-evtx-processors/src/main/java/org/apache/nifi/processors/evtx/ParseEvtx.java
---
@@ -0,0 +1,353
Github user brosander commented on the issue:
https://github.com/apache/nifi/pull/492
@pvillard31 thanks for the review so far! I'll try and beef up the
comments a bit. I think the Java 8 features make for a much cleaner
implementation so I'd like to keep them in for now
Github user brosander commented on a diff in the pull request:
https://github.com/apache/nifi/pull/492#discussion_r65756258
--- 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 brosander opened a pull request:
https://github.com/apache/nifi/pull/492
NIFI-1502 - Processor for parsing evtx files
You can merge this pull request into a Git repository by running:
$ git pull https://github.com/brosander/nifi NIFI-1502-rebase-pr
Alternatively
42 matches
Mail list logo
