Github user trixpan commented on the issue:
https://github.com/apache/nifi/pull/2016
@pvillard31 will rebase and resubmit...
---
Github user trixpan commented on the issue:
https://github.com/apache/nifi/pull/2017
@jvwing Apologies for the delay
I believe the feedback has been addressed.
---
Github user trixpan commented on the issue:
https://github.com/apache/nifi/pull/2348
@joewitt my bad. I forked from an old repo.
1.2.6 should be online.
---
GitHub user trixpan opened a pull request:
https://github.com/apache/nifi/pull/2348
NIFI-4706 - Bump ParCEFone version
Thank you for submitting a contribution to Apache NiFi.
In order to streamline the review of the contribution we ask you
to ensure the following steps
Github user trixpan commented on the issue:
https://github.com/apache/nifi/pull/1968
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, or if the
Github user trixpan commented on the issue:
https://github.com/apache/nifi/pull/2022
@pvillard31 - I assume the processor should be able to control nifi
components only. Should we rename it to make this explicit?
---
If your project is set up for it, you can reply to this email and
Github user trixpan commented on the issue:
https://github.com/apache/nifi/pull/2016
hopefully fixed
---
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 trixpan commented on a diff in the pull request:
https://github.com/apache/nifi/pull/2018#discussion_r128152991
--- Diff:
nifi-nar-bundles/nifi-standard-bundle/nifi-standard-processors/src/main/java/org/apache/nifi/processors/standard/util/SFTPTransfer.java
Github user trixpan commented on a diff in the pull request:
https://github.com/apache/nifi/pull/2018#discussion_r128152995
--- Diff:
nifi-nar-bundles/nifi-standard-bundle/nifi-standard-processors/src/main/java/org/apache/nifi/processors/standard/util/SFTPTransfer.java
Github user trixpan commented on the issue:
https://github.com/apache/nifi/pull/1696
@jfrazee I resolved conflict with master (unrelated to the change).
I would agree that the "text" qualifier was indeed missing. Fixed.
---
If your project is set up for it, you ca
Github user trixpan commented on the issue:
https://github.com/apache/nifi/pull/1644
@mcgilman - Sorry for the delay.
rebased and hopefully addressed the jUnit issue
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as
GitHub user trixpan opened a pull request:
https://github.com/apache/nifi/pull/2018
NIFI-4175 - Add HTTP proxy support to *SFTP processors
Thank you for submitting a contribution to Apache NiFi.
In order to streamline the review of the contribution we ask you
to ensure
GitHub user trixpan opened a pull request:
https://github.com/apache/nifi/pull/2017
NIFI-4197 - Expose some proxy settings to GCS Processors
Thank you for submitting a contribution to Apache NiFi.
In order to streamline the review of the contribution we ask you
to
Github user trixpan commented on the issue:
https://github.com/apache/nifi/pull/2016
Please note the displayName settings were not added as most of the AWS
related processors lack these
---
If your project is set up for it, you can reply to this email and have your
reply appear on
GitHub user trixpan opened a pull request:
https://github.com/apache/nifi/pull/2016
NIFI-4196 - Expose AWS proxy authentication settings
Thank you for submitting a contribution to Apache NiFi.
In order to streamline the review of the contribution we ask you
to ensure
GitHub user trixpan opened a pull request:
https://github.com/apache/nifi/pull/2013
NIFI-4098 - Adjust ParseCEF to re-use bean validator in order to imprâ¦
â¦ove performance
Thank you for submitting a contribution to Apache NiFi.
In order to streamline the review
Github user trixpan commented on the issue:
https://github.com/apache/nifi/pull/1921
@pvillard31 I accept it is a bit of bit picking but is there any chance we
call it _raw? IIRC this is the name we used on ParseCEF. Naming conventions
make user experience smoother ð
---
If your
Github user trixpan commented on the issue:
https://github.com/apache/nifi/pull/1696
@jfrazee?
---
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
GitHub user trixpan opened a pull request:
https://github.com/apache/nifi/pull/1950
NIFI-4126 - Add SupportBatching, SideEffectFree and EventDriven annotâ¦
â¦ations to ExtractGrok
Thank you for submitting a contribution to Apache NiFi.
In order to streamline the
Github user trixpan commented on the issue:
https://github.com/apache/nifi/pull/1644
@mcgilman indeed. Hopefully the last commit should address the remaining
ones.
Cheers
---
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/1644
@mcgilman Hopefully I managed to address your feedback in this last
iteration
---
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/1794
@apiri @jfrazee - should I go ahead and merge this?
---
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 trixpan commented on the issue:
https://github.com/apache/nifi/pull/1623
@joewitt any chance of having a look on this PR?
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 commented on the issue:
https://github.com/apache/nifi/pull/1541
@joewitt feedback should be addressed.
@trkurc - special thanks for the review and PR. I have been having been
able to commit enough resources to the project lately and your PR certainly
Github user trixpan commented on a diff in the pull request:
https://github.com/apache/nifi/pull/1541#discussion_r119568598
--- Diff:
nifi-nar-bundles/nifi-irc-bundle/nifi-irc-client-service/src/main/java/org/apache/nifi/irc/StandardIRCClientService.java
---
@@ -0,0 +1,233
Github user trixpan commented on a diff in the pull request:
https://github.com/apache/nifi/pull/1541#discussion_r119568494
--- Diff: nifi-nar-bundles/nifi-irc-bundle/nifi-irc-processors-nar/pom.xml
---
@@ -0,0 +1,41 @@
+
+
+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/1541#discussion_r119568348
--- Diff: nifi-assembly/NOTICE ---
@@ -1364,6 +1364,8 @@ The following binary components are provided under
the MIT License. See project
(MIT
GitHub user trixpan opened a pull request:
https://github.com/apache/nifi/pull/1875
NIFI-4001 - Bump ParCEFone version
Thank you for submitting a contribution to Apache NiFi.
In order to streamline the review of the contribution we ask you
to ensure the following steps
Github user trixpan commented on the issue:
https://github.com/apache/nifi/pull/1696
@jfrazee this seems to be passing tests now. Let me know what you think
---
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 a diff in the pull request:
https://github.com/apache/nifi/pull/1541#discussion_r119273679
--- Diff:
nifi-nar-bundles/nifi-irc-bundle/nifi-irc-processors/src/main/java/org/apache/nifi/processors/irc/PublishIRC.java
---
@@ -0,0 +1,139
Github user trixpan commented on the issue:
https://github.com/apache/nifi/pull/1696
@jfrazee now that HL7 is out of the way, giving it another try... Will let
you know when I am happy with code.
Cheers
---
If your project is set up for it, you can reply to this email and
Github user trixpan commented on the issue:
https://github.com/apache/nifi/pull/1845
thanks for the contribution
LGTM. 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
Github user trixpan commented on the issue:
https://github.com/apache/nifi/pull/1794
@jfrazee I intentionally used ID as I noticed ID_LIKE seemed like a mix
bag...
https://gist.github.com/djcp/6132025
---
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/1794
@jfrazee sorry. I misread your comment. It works on leap?! Great. :-)
---
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 trixpan commented on the issue:
https://github.com/apache/nifi/pull/1794
@jfrazee thanks. I haven't tested on leap.
as not working you mean the file is not being linked or the service isn't
being started?
Cheers
---
If your project is set
Github user trixpan commented on the issue:
https://github.com/apache/nifi/pull/1794
@apiri
SuSE may or may not be using systemd. Reason why we rely on systemd support
to init scripts to achieve the outcome.
@jfrazee disagree regarding the use of `lsb_release`: it
Github user trixpan commented on the issue:
https://github.com/apache/nifi/pull/1799
@mcgilman here it goes.
---
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 opened a pull request:
https://github.com/apache/nifi/pull/1799
NIFI-3896 - Makes DeprecationNotice more intuitive
Update developers guide on how to deprecate a component
Thank you for submitting a contribution to Apache NiFi.
In
Github user trixpan commented on the issue:
https://github.com/apache/nifi/pull/1541
Good news! work has been "completed" and PR squashed and rebased to master.
Things to notice:
1. At this stage, test units are marked as ignore as they expect the
availab
Github user trixpan commented on the issue:
https://github.com/apache/nifi/pull/1741
@apsaltis
Is it my impression or the new dependencies weren't added to the
nifi-assembly `NOTICE` ?
---
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/1583
@josephxsxn in light of Joe's comments, would you like to close this PR?
Cheers!
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitH
Github user trixpan commented on the issue:
https://github.com/apache/nifi/pull/1794
@apiri may I ask you to have a look on this?
Idea here is to use `/etc/os-release` to identify the Linux distribution
and use that to flag some particular distro specific issues such as use
GitHub user trixpan opened a pull request:
https://github.com/apache/nifi/pull/1794
NIFI-1709 - Introduce logic to probe Linux version using /etc/os-releâ¦
â¦ase to nifi.sh
Add explicit paths to support SLES 11 SP4 / OpenSUSE init.d
layout
Thank you
Github user trixpan commented on a diff in the pull request:
https://github.com/apache/nifi/pull/1718#discussion_r116357827
--- Diff:
nifi-api/src/main/java/org/apache/nifi/annotation/documentation/DeprecationNotice.java
---
@@ -0,0 +1,51 @@
+/*
+ * Licensed to the Apache
Github user trixpan commented on a diff in the pull request:
https://github.com/apache/nifi/pull/1718#discussion_r116357266
--- Diff:
nifi-api/src/main/java/org/apache/nifi/annotation/documentation/DeprecationNotice.java
---
@@ -0,0 +1,51 @@
+/*
+ * Licensed to the Apache
Github user trixpan commented on the issue:
https://github.com/apache/nifi/pull/1767
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, or if the
Github user trixpan commented on the issue:
https://github.com/apache/nifi/pull/1718
@mcgilman I suspect when you wrote `DocumentTypeDTO` you meant
`DocumentedTypeDTO`? If that is the case, hopefully this address the comment
---
If your project is set up for it, you can reply to
Github user trixpan commented on the issue:
https://github.com/apache/nifi/pull/1718
@mcgilman - 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
Github user trixpan commented on the issue:
https://github.com/apache/nifi/pull/1718
@mcgilman Thank you for looking at this. Yes, I would say as long this part
is ok we could merge. I also would prefer having the UI work done separately.
Cheers
---
If your project is set
Github user trixpan commented on the issue:
https://github.com/apache/nifi/pull/1718
@mcgilman
based on the following screenshot( this is a snipped of data related to a
locally deprecated `ListenLumberjack`) I suspect this should be a good starting
point for the UX work
Github user trixpan commented on the issue:
https://github.com/apache/nifi/pull/1746
adjusted the commit to refer to to the correct JIRA ticket (NIFI-3794)
---
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 trixpan opened a pull request:
https://github.com/apache/nifi/pull/1746
NIFI-3726 - Expose the control of ListenRELP's CLIENT_AUTH property tâ¦
â¦o DFM
Thank you for submitting a contribution to Apache NiFi.
In order to streamline the review o
Github user trixpan commented on the issue:
https://github.com/apache/nifi/pull/1738
Oops. GMT+10. Forgot my daylight savings is over. :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
Github user trixpan commented on the issue:
https://github.com/apache/nifi/pull/1738
The people from the future salute you, oh fellow space traveler!
LGTM. And works on GMT+11.
Suggest waiting for travis to complete to ensure it runs on other Time
zones (I assume you
Github user trixpan commented on the issue:
https://github.com/apache/nifi/pull/1692
@mattyb149 just to register: TLSH requires more than 512 characters to work
so your hello message triggers a safety mechanism that routes the flow to
error.
I should have used error but I
Github user trixpan commented on the issue:
https://github.com/apache/nifi/pull/1692
pom has also been fixed
---
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 the issue:
https://github.com/apache/nifi/pull/1692
@alopresto @mattyb149 last two commits should address the two residual
issues you identified.
One was a simple reference to HashContent.class instead of the fully
qualified class path as
Github user trixpan commented on the issue:
https://github.com/apache/nifi/pull/1692
Andy, easy to solve
---
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/1696
@jfrazee while we are doing HL7 is there any chance you can look at the
EncryptContent tests as well?
Due to the nature of PGP ASCII Armor files they use a mix of DOS and Unix
line format as
Github user trixpan closed the pull request at:
https://github.com/apache/nifi/pull/1729
---
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 trixpan commented on the issue:
https://github.com/apache/nifi/pull/1729
closing PR as it seems @markap14 has merged it to master but forgot to
refer to PR when siging off the commit
---
If your project is set up for it, you can reply to this email and have your
reply
Github user trixpan commented on the issue:
https://github.com/apache/nifi/pull/1339
@lexvanderstoep
Given @joewitt 's JIRA comments, would you mind closing this PR and update
the JIRA as Not a bug?
Kind regards
---
If your project is set up for it, you can
Github user trixpan commented on the issue:
https://github.com/apache/nifi/pull/399
@simonellistonball - Seems like your PR has been adopted by @jtstorck and
@aperepel so closing this PR as we currently have 3 PRs for the same JIRA.
---
If your project is set up for it, you can
GitHub user trixpan opened a pull request:
https://github.com/apache/nifi/pull/1729
NIFI-3769 - Document detail value of 'nifi.provenance.repository.warmâ¦
â¦.cache.frequency'
Thank you for submitting a contribution to Apache NiFi.
In order to stre
Github user trixpan commented on the issue:
https://github.com/apache/nifi/pull/1721
@apiri - I used the embedded HTML markdown notation as this seems to be a
more universal approach to image resizing. There's are other options
http://stackoverflow.com/questions/1467591
GitHub user trixpan opened a pull request:
https://github.com/apache/nifi/pull/1721
NIFI-3651 - Add logo to README.md
Thank you for submitting a contribution to Apache NiFi.
In order to streamline the review of the contribution we ask you
to ensure the following steps
Github user trixpan commented on the issue:
https://github.com/apache/nifi/pull/1720
@bbende - hopefully this will address the intent behind the JIRA ticket.
Please note that I also:
- added displayName to all properties
- re-indented some of the code (as it had double
GitHub user trixpan opened a pull request:
https://github.com/apache/nifi/pull/1720
NIFI-3670 - Expose the control of ListenSyslog's CLIENT_AUTH propertyâ¦
⦠to DFM
Thank you for submitting a contribution to Apache NiFi.
In order to streamline the review o
Github user trixpan commented on the issue:
https://github.com/apache/nifi/pull/1142
Adding @ToivoAdams comment here as his last comment on JIRA did use
github's @ mentions instead of JIRAs [~ ] mentions:
> @ijokarumawak
>
> Sorry I didn't hav
Github user trixpan commented on the issue:
https://github.com/apache/nifi/pull/1102
@combineads - thanks for the PR.
Apologies for lacking the background but I was wondering what the issue
effectively is?
From what you say the issue is that PutSQL requires -MM
Github user trixpan commented on the issue:
https://github.com/apache/nifi/pull/1696
@joewitt as suspected HL7 does inded use CR as a segment separator so I
renamed the files to hl7 and treated hl7 as binary. Tests passed locally and I
am now checking what travis has to say
---
If
Github user trixpan commented on the issue:
https://github.com/apache/nifi/pull/1015
@jfrazee any chance of rebasing this?
Keen to revisit this PR and get the review going, otherwise happy for you
to close it as well.
Cheers
---
If your project is set up for it
Github user trixpan commented on the issue:
https://github.com/apache/nifi/pull/584
@vcharmcaster - there has been a while since you opened this PR and @olegz
question still an answered. Is this PR still needed?
Otherwise, would you mind closing it?
Kind regards
Github user trixpan commented on the issue:
https://github.com/apache/nifi/pull/1608
@pvillard31 thanks for the rebase. LGTM. Merged
---
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 trixpan opened a pull request:
https://github.com/apache/nifi/pull/1718
NIFI-391 - Introduce the ability to deprecate a component
Thank you for submitting a contribution to Apache NiFi.
In order to streamline the review of the contribution we ask you
to
Github user trixpan commented on a diff in the pull request:
https://github.com/apache/nifi/pull/1639#discussion_r114039631
--- Diff:
nifi-nar-bundles/nifi-standard-bundle/nifi-standard-processors/src/main/java/org/apache/nifi/processors/standard/ParseSyslog.java
---
@@ -57,13
Github user trixpan commented on a diff in the pull request:
https://github.com/apache/nifi/pull/1692#discussion_r113932077
--- Diff:
nifi-nar-bundles/nifi-cybersecurity-bundle/nifi-cybersecurity-processors/src/main/java/org/apache/nifi/processors/cybersecurity/FuzzyHashContent.java
Github user trixpan commented on a diff in the pull request:
https://github.com/apache/nifi/pull/1692#discussion_r113917927
--- Diff:
nifi-nar-bundles/nifi-cybersecurity-bundle/nifi-cybersecurity-processors/src/main/java/org/apache/nifi/processors/cybersecurity/CompareFuzzyHash.java
Github user trixpan commented on a diff in the pull request:
https://github.com/apache/nifi/pull/1692#discussion_r113918024
--- Diff:
nifi-nar-bundles/nifi-cybersecurity-bundle/nifi-cybersecurity-processors/src/main/java/org/apache/nifi/processors/cybersecurity/CompareFuzzyHash.java
Github user trixpan commented on the issue:
https://github.com/apache/nifi/pull/1716
Speed and overall project roadmap.
The approach introduced there brings significant advantages to the use of
PutHDFS.
https://github.com/apache/nifi/pull/1712
---
If your
Github user trixpan commented on the issue:
https://github.com/apache/nifi/pull/1716
@jonashartwig thank you for your contribution.
I was looking at your code and was wondering if have you had a chance to
see the put / fetch parquet @bbende submitted a few hours ago
Github user trixpan commented on a diff in the pull request:
https://github.com/apache/nifi/pull/1692#discussion_r113587423
--- Diff:
nifi-nar-bundles/nifi-cybersecurity-bundle/nifi-cybersecurity-processors/src/main/java/org/apache/nifi/processors/cybersecurity/FuzzyHashContent.java
Github user trixpan commented on a diff in the pull request:
https://github.com/apache/nifi/pull/1692#discussion_r113586627
--- Diff:
nifi-nar-bundles/nifi-cybersecurity-bundle/nifi-cybersecurity-processors/src/main/java/org/apache/nifi/processors/cybersecurity/CompareFuzzyHash.java
Github user trixpan commented on a diff in the pull request:
https://github.com/apache/nifi/pull/1692#discussion_r113586323
--- Diff:
nifi-nar-bundles/nifi-cybersecurity-bundle/nifi-cybersecurity-processors/src/main/java/org/apache/nifi/processors/cybersecurity/CompareFuzzyHash.java
Github user trixpan commented on a diff in the pull request:
https://github.com/apache/nifi/pull/1692#discussion_r113585955
--- Diff:
nifi-nar-bundles/nifi-cybersecurity-bundle/nifi-cybersecurity-processors/src/main/java/org/apache/nifi/processors/cybersecurity/CompareFuzzyHash.java
Github user trixpan commented on a diff in the pull request:
https://github.com/apache/nifi/pull/1692#discussion_r113585822
--- Diff:
nifi-nar-bundles/nifi-cybersecurity-bundle/nifi-cybersecurity-processors/src/main/java/org/apache/nifi/processors/cybersecurity/CompareFuzzyHash.java
Github user trixpan commented on the issue:
https://github.com/apache/nifi/pull/1692
@alopresto - I put together the interface based approach and indeed it
seems a much cleaner code so thank you for pointing me into this direction.
The code has not been optimised and still
Github user trixpan commented on a diff in the pull request:
https://github.com/apache/nifi/pull/1692#discussion_r113463146
--- Diff:
nifi-nar-bundles/nifi-cybersecurity-bundle/nifi-cybersecurity-processors/src/main/java/org/apache/nifi/processors/cybersecurity/CompareFuzzyHash.java
Github user trixpan commented on a diff in the pull request:
https://github.com/apache/nifi/pull/1692#discussion_r113462407
--- Diff:
nifi-nar-bundles/nifi-cybersecurity-bundle/nifi-cybersecurity-processors/src/main/java/org/apache/nifi/processors/cybersecurity/CompareFuzzyHash.java
Github user trixpan commented on a diff in the pull request:
https://github.com/apache/nifi/pull/1692#discussion_r113462423
--- Diff:
nifi-nar-bundles/nifi-cybersecurity-bundle/nifi-cybersecurity-processors/src/main/java/org/apache/nifi/processors/cybersecurity/CompareFuzzyHash.java
Github user trixpan commented on a diff in the pull request:
https://github.com/apache/nifi/pull/1692#discussion_r113462459
--- Diff:
nifi-nar-bundles/nifi-cybersecurity-bundle/nifi-cybersecurity-processors/src/main/java/org/apache/nifi/processors/cybersecurity/CompareFuzzyHash.java
Github user trixpan commented on the issue:
https://github.com/apache/nifi/pull/1696
@joewitt - When doing this change I noticed some changes to HL7 files and
looking online it seems like HL7 uses CR as a "segment separator".
Would you mind if I renamed the rele
GitHub user trixpan opened a pull request:
https://github.com/apache/nifi/pull/1696
NIFI-1655 - Add .gitattributes to specifically define
the line ending format of particular file type (i.e. .java =
LF, .bat=CRLF)
Thank you for submitting a contribution to
Github user trixpan commented on a diff in the pull request:
https://github.com/apache/nifi/pull/1650#discussion_r113431000
--- Diff:
nifi-nar-bundles/nifi-enrich-bundle/nifi-enrich-processors/src/test/java/org/apache/nifi/processors/TestISPEnrichIP.java
---
@@ -0,0 +1,310
Github user trixpan commented on a diff in the pull request:
https://github.com/apache/nifi/pull/1650#discussion_r113428044
--- Diff:
nifi-nar-bundles/nifi-enrich-bundle/nifi-enrich-processors/src/main/java/org/apache/nifi/processors/ISPEnrichIP.java
---
@@ -0,0 +1,132
Github user trixpan commented on a diff in the pull request:
https://github.com/apache/nifi/pull/1650#discussion_r113411573
--- Diff:
nifi-nar-bundles/nifi-enrich-bundle/nifi-enrich-processors/src/main/java/org/apache/nifi/processors/GeoEnrichIP.java
---
@@ -208,6 +120,8
Github user trixpan commented on a diff in the pull request:
https://github.com/apache/nifi/pull/1650#discussion_r113411154
--- Diff:
nifi-nar-bundles/nifi-enrich-bundle/nifi-enrich-processors/src/main/java/org/apache/nifi/processors/GeoEnrichIP.java
---
@@ -70,6 +55,7
Github user trixpan commented on a diff in the pull request:
https://github.com/apache/nifi/pull/1650#discussion_r113340946
--- Diff:
nifi-nar-bundles/nifi-enrich-bundle/nifi-enrich-processors/src/main/java/org/apache/nifi/processors/GeoEnrichIP.java
---
@@ -208,6 +120,8
Github user trixpan commented on a diff in the pull request:
https://github.com/apache/nifi/pull/1692#discussion_r113340505
--- Diff:
nifi-nar-bundles/nifi-cybersecurity-bundle/nifi-cybersecurity-processors/src/main/java/org/apache/nifi/processors/cybersecurity/CompareFuzzyHash.java
Github user trixpan commented on the issue:
https://github.com/apache/nifi/pull/1693
@andrewmlim :+1: :+1: :+1: :+1: for catching this...
---
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
1 - 100 of 761 matches
Mail list logo