Github user trixpan commented on the issue:
https://github.com/apache/nifi/pull/1481
@markap14 I suspect the reason is @mosermw added the closure after the
signature and closure should come above sign-off-by?
---
If your project is set up for it, you can reply to this email
Github user trixpan commented on the issue:
https://github.com/apache/nifi/pull/1326
@patricker merge after build, reviewed code and ran against an exchange
server. Seems to be working fine.
thank you for the contribution!
---
If your project is set up for it, you can
Github user trixpan commented on the issue:
https://github.com/apache/nifi/pull/1505
Epic. Will adjust
---
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 trixpan commented on the issue:
https://github.com/apache/nifi/pull/972
@brosander is the PR still wanted? If yes, mind rebasing?
---
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 trixpan commented on the issue:
https://github.com/apache/nifi/pull/1326
@patricker can you please rebase as changes to master are causing conflicts
?
---
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 trixpan commented on the issue:
https://github.com/apache/nifi/pull/453
@eorgad , this PR is nearly 1 year old do you still plan to address some of
the feedback? Would you mind closing the PR for now?
---
If your project is set up for it, you can reply to this email
Github user trixpan commented on the issue:
https://github.com/apache/nifi/pull/1335
@ijokarumawak my bad. I was just running a extra set of compilations and I
noticed that under certain conditions there seems to be a race condition
affecting
```
Tests run: 12, Failures
Github user trixpan commented on the issue:
https://github.com/apache/nifi/pull/1016
That was a merge, not a rebase :-(
---
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 trixpan commented on the issue:
https://github.com/apache/nifi/pull/1016
Try
git checkout master -b nifi-2724v2
git cherry-pick
git push origin nifi-2724
---
If your project is set up for it, you can reply to this email and have your
reply appear
Github user trixpan commented on the issue:
https://github.com/apache/nifi/pull/1016
Try
git checkout master -b nifi-2724v2
git cherry-pick your commit hash
git push - f origin nifi-2724
This will create a temporary branch
Allow you to copy a commit from
Github user trixpan commented on the issue:
https://github.com/apache/nifi/pull/1016
No no. Cherry pick your commit number.
You will likely to fix conflicts in the process but git will flag those
files with conflict so once you fix it should be ok at the end
---
If your
Github user trixpan commented on the issue:
https://github.com/apache/nifi/pull/1402
LGTM
---
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 trixpan commented on a diff in the pull request:
https://github.com/apache/nifi/pull/1108#discussion_r96236367
--- Diff:
nifi-nar-bundles/nifi-standard-bundle/nifi-standard-processors/src/main/java/org/apache/nifi/processors/standard/ExtractGrok.java
---
@@ -0,0
Github user trixpan commented on a diff in the pull request:
https://github.com/apache/nifi/pull/420#discussion_r96251033
--- Diff:
nifi-nar-bundles/nifi-standard-bundle/nifi-standard-processors/pom.xml ---
@@ -235,6 +235,11 @@ language governing permissions and limitations under
Github user trixpan commented on the issue:
https://github.com/apache/nifi/pull/1198
@jfrazee - odd. Should the bootstrap.conf `run.as` be
[populated](https://github.com/apache/nifi/blob/d838f61291d2582592754a37314911b701c6891b/nifi-nar-bundles/nifi-framework-bundle/nifi-framework
Github user trixpan commented on the issue:
https://github.com/apache/nifi/pull/1302
@alopresto ?
---
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 trixpan closed the pull request at:
https://github.com/apache/nifi/pull/991
---
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 trixpan commented on the issue:
https://github.com/apache/nifi/pull/397
@jdye64 is the PR active or should we close it?
Cheers
---
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
GitHub user trixpan opened a pull request:
https://github.com/apache/nifi/pull/1421
NIFI-3352 - Address issue where ListenLumberjack was sending long numâ¦
â¦bers back to clients - instead of int
Thank you for submitting a contribution to Apache NiFi.
In order
Github user trixpan commented on the issue:
https://github.com/apache/nifi/pull/709
@simonellistonball @mattyb149 should we perhaps close this 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 does
Github user trixpan commented on a diff in the pull request:
https://github.com/apache/nifi/pull/1326#discussion_r96258831
--- Diff:
nifi-nar-bundles/nifi-email-bundle/nifi-email-processors/src/main/java/org/apache/nifi/processors/email/ConsumeEWS.java
---
@@ -0,0 +1,518
Github user trixpan commented on a diff in the pull request:
https://github.com/apache/nifi/pull/1326#discussion_r96259066
--- Diff:
nifi-nar-bundles/nifi-email-bundle/nifi-email-processors/src/main/java/org/apache/nifi/processors/email/ConsumeEWS.java
---
@@ -0,0 +1,518
Github user trixpan commented on the issue:
https://github.com/apache/nifi/pull/1326
@patricker Joe will have a more complete view on this but looking at the
MSFT code there seem to be some dependencies with NOTICE. From top of my mind,
joda-time, commons-io and others that must
Github user trixpan commented on a diff in the pull request:
https://github.com/apache/nifi/pull/1422#discussion_r96312325
--- Diff: nifi-assembly/pom.xml ---
@@ -19,6 +19,10 @@ language governing permissions and limitations under the
License. -->
nifi-assem
Github user trixpan commented on a diff in the pull request:
https://github.com/apache/nifi/pull/1422#discussion_r96984003
--- Diff: nifi-assembly/pom.xml ---
@@ -19,6 +19,10 @@ language governing permissions and limitations under the
License. -->
nifi-assem
Github user trixpan commented on a diff in the pull request:
https://github.com/apache/nifi/pull/1422#discussion_r96322520
--- Diff: nifi-assembly/pom.xml ---
@@ -19,6 +19,10 @@ language governing permissions and limitations under the
License. -->
nifi-assem
GitHub user trixpan opened a pull request:
https://github.com/apache/nifi/pull/1422
NIFI-3015 - Corrects issue were RPM profile run.as property was not bâ¦
â¦eing used
This supersedes #1198
Thank you for submitting a contribution to Apache NiFi.
In order
Github user trixpan commented on the issue:
https://github.com/apache/nifi/pull/1422
@fidget83 and @jfrazee mind having a look and let me know if it addresses
the issue?
Cheers
---
If your project is set up for it, you can reply to this email and have your
reply appear
Github user trixpan commented on the issue:
https://github.com/apache/nifi/pull/1422
Reviewers:
Key here is that bootstrap.conf with RPM profile should result on
`run.as=nifi` while vanilla build should result in `run.as=`
(naturally assuming nothing else breaks
GitHub user trixpan opened a pull request:
https://github.com/apache/nifi/pull/871
NIFI-2519 - ListenSMTP addresses mismatch between smtp.src attribute â¦
â¦documentation and attribute.put
You can merge this pull request into a Git repository by running:
$ git pull https
Github user trixpan commented on a diff in the pull request:
https://github.com/apache/nifi/pull/858#discussion_r75580268
--- Diff:
nifi-nar-bundles/nifi-enrich-bundle/nifi-enrich-processors/src/main/java/org/apache/nifi/processors/enrich/QueryWhois.java
---
@@ -0,0 +1,334
Github user trixpan commented on the issue:
https://github.com/apache/nifi/pull/858
@pvillard31
The error is expected as the regex does not contain a capture group named
KEY. It is a bug as I should detect KEY or whatever value was chosen.
But your feedback
Github user trixpan commented on a diff in the pull request:
https://github.com/apache/nifi/pull/858#discussion_r75579997
--- Diff:
nifi-nar-bundles/nifi-enrich-bundle/nifi-enrich-processors/pom.xml ---
@@ -63,5 +63,31 @@
nifi-mock
test
Github user trixpan commented on a diff in the pull request:
https://github.com/apache/nifi/pull/858#discussion_r75580295
--- Diff:
nifi-nar-bundles/nifi-enrich-bundle/nifi-enrich-processors/src/main/java/org/apache/nifi/processors/enrich/QueryWhois.java
---
@@ -0,0 +1,334
Github user trixpan commented on a diff in the pull request:
https://github.com/apache/nifi/pull/858#discussion_r75580308
--- Diff:
nifi-nar-bundles/nifi-enrich-bundle/nifi-enrich-processors/src/test/java/org/apache/nifi/processors/enrich/TestQueryWhois.java
---
@@ -0,0 +1,187
Github user trixpan commented on a diff in the pull request:
https://github.com/apache/nifi/pull/858#discussion_r75579988
--- Diff:
nifi-nar-bundles/nifi-enrich-bundle/nifi-enrich-processors/pom.xml ---
@@ -63,5 +63,31 @@
nifi-mock
test
Github user trixpan commented on the issue:
https://github.com/apache/nifi/pull/858
As you can see the KEY group is documented within NiFi code but the UI
lacks such explanation
https://github.com/apache/nifi/pull/858/files#diff-d13cbfd36645869513f9d09b76636240R183
Github user trixpan commented on a diff in the pull request:
https://github.com/apache/nifi/pull/1418#discussion_r103645596
--- Diff:
nifi-nar-bundles/nifi-beats-bundle/nifi-beats-nar/src/main/resources/META-INF/LICENSE
---
@@ -0,0 +1,233
Github user trixpan commented on a diff in the pull request:
https://github.com/apache/nifi/pull/1418#discussion_r103647459
--- Diff: nifi-nar-bundles/nifi-beats-bundle/nifi-beats-processors/pom.xml
---
@@ -0,0 +1,76 @@
+
+
+http://maven.apache.org/POM/4.0.0
Github user trixpan commented on a diff in the pull request:
https://github.com/apache/nifi/pull/1418#discussion_r103648185
--- Diff:
nifi-nar-bundles/nifi-lumberjack-bundle/nifi-lumberjack-processors/src/main/java/org/apache/nifi/processors/lumberjack/ListenLumberjack.java
Github user trixpan commented on a diff in the pull request:
https://github.com/apache/nifi/pull/1418#discussion_r103648209
--- Diff:
nifi-nar-bundles/nifi-lumberjack-bundle/nifi-lumberjack-processors/src/main/java/org/apache/nifi/processors/lumberjack/event/LumberjackEvent.java
Github user trixpan commented on a diff in the pull request:
https://github.com/apache/nifi/pull/1418#discussion_r103649158
--- Diff:
nifi-nar-bundles/nifi-beats-bundle/nifi-beats-processors/src/main/java/org/apache/nifi/processors/beats/ListenBeats.java
---
@@ -0,0 +1,216
Github user trixpan commented on a diff in the pull request:
https://github.com/apache/nifi/pull/1418#discussion_r103650361
--- Diff:
nifi-nar-bundles/nifi-beats-bundle/nifi-beats-processors/src/main/java/org/apache/nifi/processors/beats/frame/BeatsDecoder.java
---
@@ -0,0 +1,330
Github user trixpan commented on a diff in the pull request:
https://github.com/apache/nifi/pull/1418#discussion_r103651066
--- Diff:
nifi-nar-bundles/nifi-beats-bundle/nifi-beats-processors/src/main/java/org/apache/nifi/processors/beats/frame/BeatsDecoder.java
---
@@ -0,0 +1,330
Github user trixpan commented on a diff in the pull request:
https://github.com/apache/nifi/pull/1418#discussion_r103652588
--- Diff:
nifi-nar-bundles/nifi-beats-bundle/nifi-beats-processors/src/main/java/org/apache/nifi/processors/beats/frame/BeatsDecoder.java
---
@@ -0,0 +1,330
Github user trixpan commented on a diff in the pull request:
https://github.com/apache/nifi/pull/1418#discussion_r103653787
--- Diff:
nifi-nar-bundles/nifi-beats-bundle/nifi-beats-processors/src/main/java/org/apache/nifi/processors/beats/ListenBeats.java
---
@@ -0,0 +1,216
Github user trixpan commented on the issue:
https://github.com/apache/nifi/pull/1541
processor logic is theoretically complete. Now finishing the IRC over TLS
logic
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well
Github user trixpan commented on a diff in the pull request:
https://github.com/apache/nifi/pull/1016#discussion_r103875353
--- Diff: nifi-nar-bundles/nifi-jmx-bundle/nifi-jmx-processors/pom.xml ---
@@ -0,0 +1,67 @@
+
+
+http://maven.apache.org/POM/4.0.0;
xmlns:xsi
Github user trixpan commented on the issue:
https://github.com/apache/nifi/pull/1016
Looks good. Will review
---
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 trixpan commented on a diff in the pull request:
https://github.com/apache/nifi/pull/1016#discussion_r103875643
--- Diff:
nifi-nar-bundles/nifi-jmx-bundle/nifi-jmx-processors/src/main/java/org/apache/nifi/processors/jmx/GetJMX.java
---
@@ -0,0 +1,415
Github user trixpan commented on a diff in the pull request:
https://github.com/apache/nifi/pull/1016#discussion_r103875149
--- Diff: nifi-nar-bundles/nifi-jmx-bundle/nifi-jmx-processors/pom.xml ---
@@ -0,0 +1,67 @@
+
+
+http://maven.apache.org/POM/4.0.0;
xmlns:xsi
GitHub user trixpan opened a pull request:
https://github.com/apache/nifi/pull/1551
NIFI-3054 - Create default all default directions as part of maven-asâ¦
â¦sembly
Update Admin Guide with recommended directory permissions
Update some minor
GitHub user trixpan opened a pull request:
https://github.com/apache/nifi/pull/1553
NIFI-1449 - Migrate PutEmail tests from Mock class to ListenSMTP
Thank you for submitting a contribution to Apache NiFi.
In order to streamline the review of the contribution we ask you
GitHub user trixpan opened a pull request:
https://github.com/apache/nifi/pull/1552
NIFI-3544 - Fixes minor typo n ExtractEmailHeaders log message
Thank you for submitting a contribution to Apache NiFi.
In order to streamline the review of the contribution we ask you
Github user trixpan commented on the issue:
https://github.com/apache/nifi/pull/1553
@joewitt - Perhaps you would you be able to give me some insight here.
For some reason when I run the following lines of
testOutgoingMessageAttachment
Github user trixpan commented on a diff in the pull request:
https://github.com/apache/nifi/pull/1418#discussion_r103599675
--- Diff:
nifi-nar-bundles/nifi-beats-bundle/nifi-beats-processors/src/main/java/org/apache/nifi/processors/beats/ListenBeats.java
---
@@ -0,0 +1,216
Github user trixpan commented on a diff in the pull request:
https://github.com/apache/nifi/pull/1418#discussion_r103599499
--- Diff:
nifi-nar-bundles/nifi-beats-bundle/nifi-beats-nar/src/main/resources/META-INF/LICENSE
---
@@ -0,0 +1,233
Github user trixpan commented on a diff in the pull request:
https://github.com/apache/nifi/pull/858#discussion_r76220146
--- Diff:
nifi-nar-bundles/nifi-enrich-bundle/nifi-enrich-processors/src/main/java/org/apache/nifi/processors/enrich/QueryWhois.java
---
@@ -0,0 +1,334
Github user trixpan commented on a diff in the pull request:
https://github.com/apache/nifi/pull/858#discussion_r76225271
--- Diff:
nifi-nar-bundles/nifi-enrich-bundle/nifi-enrich-processors/src/main/java/org/apache/nifi/processors/enrich/QueryWhois.java
---
@@ -0,0 +1,334
Github user trixpan commented on a diff in the pull request:
https://github.com/apache/nifi/pull/858#discussion_r76225290
--- Diff:
nifi-nar-bundles/nifi-enrich-bundle/nifi-enrich-processors/src/main/java/org/apache/nifi/processors/enrich/QueryWhois.java
---
@@ -0,0 +1,334
Github user trixpan commented on the issue:
https://github.com/apache/nifi/pull/858
@pvillard31 - reworded validators, unified the regex / split parses to use
numeric capture groups/columns
---
If your project is set up for it, you can reply to this email and have your
reply appear
GitHub user trixpan opened a pull request:
https://github.com/apache/nifi/pull/940
NIFI-2662 - Fix checkstyle violations introduced by NIFI-2605
You can merge this pull request into a Git repository by running:
$ git pull https://github.com/trixpan/nifi NIFI-2662
Github user trixpan commented on the issue:
https://github.com/apache/nifi/pull/940
previous title and commit message referred to incorrect JIRA number.
Correct number according to Github's blame view is NIFI-2635
---
If your project is set up for it, you can reply
GitHub user trixpan opened a pull request:
https://github.com/apache/nifi/pull/941
NIFI-2662 - Fix style violations previously missed
Fix styles missed during PR940
You can merge this pull request into a Git repository by running:
$ git pull https://github.com/trixpan/nifi
Github user trixpan commented on the issue:
https://github.com/apache/nifi/pull/941
@mattyb149
---
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 trixpan commented on the issue:
https://github.com/apache/nifi/pull/785
@mattyb149
feedback addressed.
My comment is to play particular attention on threat concurrency around
Jackson's ObjectMapper (jackson say it is threat safe but thread safety
Github user trixpan commented on the issue:
https://github.com/apache/nifi/pull/785
Rofl. Well at least I got 1 out 3 right. As you can see we security folks
are a bit obsessed with threats and risks :-)
---
If your project is set up for it, you can reply to this email and have
GitHub user trixpan opened a pull request:
https://github.com/apache/nifi/pull/985
NIFI-2728 - Attempt to fix travis-ci build woes
You can merge this pull request into a Git repository by running:
$ git pull https://github.com/trixpan/nifi NIFI-2728
Alternatively you can
Github user trixpan commented on the issue:
https://github.com/apache/nifi/pull/985
@mcgilman
can you please review this and if in agreement merge it to master?
Current travis-ci settings seem to be unable to reliably build nifi. I
fiddled with Xmx settings, enabled
Github user trixpan commented on the issue:
https://github.com/apache/nifi/pull/985
Seems like moving `mvn -T2.0C clean install` to travis install phase and
`mvn checkstyle:check` to the script phase did the trick.
around 25% reduction in build time and a far more reliable
Github user trixpan commented on the issue:
https://github.com/apache/nifi/pull/985
@apiri makes complete sense. afaik this can be achieved using
`mvn apache-rat:check and checkstyle:check `
so I added the above command to the before_install stage of the travis job
Github user trixpan commented on a diff in the pull request:
https://github.com/apache/nifi/pull/817#discussion_r77625743
--- Diff:
nifi-nar-bundles/nifi-email-bundle/nifi-email-processors/src/main/java/org/apache/nifi/processors/email/ExtractTNEFAttachments.java
---
@@ -0,0
Github user trixpan commented on a diff in the pull request:
https://github.com/apache/nifi/pull/817#discussion_r77625649
--- Diff:
nifi-nar-bundles/nifi-email-bundle/nifi-email-processors/src/main/java/org/apache/nifi/processors/email/ExtractTNEFAttachments.java
---
@@ -0,0
Github user trixpan closed the pull request at:
https://github.com/apache/nifi/pull/960
---
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 trixpan commented on the issue:
https://github.com/apache/nifi/pull/991
@rdblue would you have any feedback over this change? Not ideal but I
couldn't find a better way. Open to suggestions
---
If your project is set up for it, you can reply to this email and have your
Github user trixpan commented on the issue:
https://github.com/apache/nifi/pull/991
@joewitt I will be happy to address that! Changing the master LICENSE and
NOTICE files is one of my part time hobbies! :neutral_face:
---
If your project is set up for it, you can reply to this email
GitHub user trixpan opened a pull request:
https://github.com/apache/nifi/pull/990
NIFI-2744 - turn hbase-client version into a pom.xml property
You can merge this pull request into a Git repository by running:
$ git pull https://github.com/trixpan/nifi NIFI-2744
Github user trixpan commented on the issue:
https://github.com/apache/nifi/pull/990
@mattyb149 seems like this one is also required to address @xmlking
concerns around MapR-DB
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub
Github user trixpan commented on the issue:
https://github.com/apache/nifi/pull/985
Matt, the profile is still being run but it run after compilation.
First (install) travis performs a multi threaded mvn build. Immediately
aftet this completes it runs a mvn checkstyle:check
Github user trixpan commented on the issue:
https://github.com/apache/nifi/pull/985
@mcgillman, what do you think about leaving the - T2.0C setting to speed up
build and adding an extra post-compile step to run - Pcontrib-check single
threaded (this is similar to spark's approach
Github user trixpan commented on the issue:
https://github.com/apache/nifi/pull/817
@olegz
Rebased.
Please note that while I will be happy to keep rebasing as peer review is
in progress, this PR makes additions to the nifi-assembly LICENSE and NOTICE
files so
Github user trixpan closed the pull request at:
https://github.com/apache/nifi/pull/990
---
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 trixpan commented on the issue:
https://github.com/apache/nifi/pull/990
No worries
---
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 trixpan commented on the issue:
https://github.com/apache/nifi/pull/980
Let me know when this is ready to test and I will give it a go
---
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
Github user trixpan commented on the issue:
https://github.com/apache/nifi/pull/817
@olegz how is testing looking like?
Cheers
---
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 trixpan commented on the issue:
https://github.com/apache/nifi/pull/858
@pvillard31 any chance of completing this PR ? :smile:
---
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 trixpan opened a pull request:
https://github.com/apache/nifi/pull/1003
NIFI-2755 - Fixes minor typo in Developers Guide
You can merge this pull request into a Git repository by running:
$ git pull https://github.com/trixpan/nifi NIFI-2755
Alternatively you can
GitHub user trixpan opened a pull request:
https://github.com/apache/nifi/pull/1004
NIFI-2756 - Add displayName to maven archetypes
@alopresto Would you mind having a look on this PR?
I am not familiar with maven archetypes but I noticed that the ones we have
seem
Github user trixpan commented on the issue:
https://github.com/apache/nifi/pull/858
@pvillard31 feedback addressed.
---
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 trixpan commented on a diff in the pull request:
https://github.com/apache/nifi/pull/980#discussion_r78770355
--- Diff:
nifi-nar-bundles/nifi-standard-bundle/nifi-standard-processors/src/main/java/org/apache/nifi/processors/standard/TailFile.java
---
@@ -117,31
Github user trixpan commented on a diff in the pull request:
https://github.com/apache/nifi/pull/858#discussion_r78476147
--- Diff:
nifi-nar-bundles/nifi-enrich-bundle/nifi-enrich-processors/src/main/java/org/apache/nifi/processors/enrich/AbstractEnrichProcessor.java
---
@@ -65,7
Github user trixpan commented on a diff in the pull request:
https://github.com/apache/nifi/pull/858#discussion_r78475823
--- Diff:
nifi-nar-bundles/nifi-enrich-bundle/nifi-enrich-processors/src/main/java/org/apache/nifi/processors/enrich/AbstractEnrichProcessor.java
Github user trixpan commented on a diff in the pull request:
https://github.com/apache/nifi/pull/858#discussion_r78476100
--- Diff:
nifi-nar-bundles/nifi-enrich-bundle/nifi-enrich-processors/src/main/java/org/apache/nifi/processors/enrich/AbstractEnrichProcessor.java
Github user trixpan commented on the issue:
https://github.com/apache/nifi/pull/858
@pvillard31 can you confirm if this was generated by Split or Regex capture?
Note that the regex is a multiline match, therefore user can remove the
newline via regular expression
Github user trixpan commented on the issue:
https://github.com/apache/nifi/pull/985
@ijokarumawak
---
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 trixpan commented on the issue:
https://github.com/apache/nifi/pull/1025
@ijokarumawak
there's already a PR for that keen for you to look at
https://github.com/apache/nifi/pull/985
and merge it if you don't mind
---
If your project is set
Github user trixpan commented on a diff in the pull request:
https://github.com/apache/nifi/pull/985#discussion_r78266044
--- Diff: .travis.yml ---
@@ -6,12 +21,22 @@ os:
jdk:
- oraclejdk8
+# Caches mvn repository in order to speed upbuilds
+cache
Github user trixpan commented on the issue:
https://github.com/apache/nifi/pull/985
@apiri, @mcgilman
I did a few new tweaks so that job sequence is:
1. rat check (before_install)
2. parallel compile with skipTests (install)
3. style check and test via mvn
Github user trixpan commented on the issue:
https://github.com/apache/nifi/pull/1002
@joewitt this commit is quite small (intent is just aesthetics / UX) but
ventures into waters I don't usually sail... Would you mind having a look?
I thank you in advance
---
If your
Github user trixpan commented on the issue:
https://github.com/apache/nifi/pull/1057
@pvillard31 note to myself: Never try to hunt typos after midnight ...
:smiley:
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well
201 - 300 of 658 matches
Mail list logo
