Github user trixpan commented on the issue:
https://github.com/apache/nifi/pull/399
@slachterman - thank you for the handy link
---
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/1332
@ijokarumawak - This PR needs 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 this feature
Github user trixpan commented on the issue:
https://github.com/apache/nifi/pull/1378
@pvillard31 interestingly enough I just gone through a PR by @apsaltis that
has more or less the same ethos as yours (PR#1331). His is focused on MiNiFi,
while yours is perhaps a bit more NiFi
Github user trixpan commented on the issue:
https://github.com/apache/nifi/pull/349
@joewitt - I am happy to adopt the stalled PR and address @JPercivall 's
comments.
---
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/1521
NIFI-1767 - Added 1st iteration of IOT-Processors for reading and wriâ¦
â¦ting AWS IOT shadow and MQTT topics
This closes: #349
Thank you for submitting a contribution to Apache
Github user trixpan commented on the issue:
https://github.com/apache/nifi/pull/1521
@KayLerch - Thank you for the contribution, I squashed your commit but left
attribution untouched so credit is due to who deserves it.
@JPercivall this is not ready to review. For now the PR
Github user trixpan commented on a diff in the pull request:
https://github.com/apache/nifi/pull/1521#discussion_r101915841
--- Diff:
nifi-nar-bundles/nifi-aws-bundle/nifi-aws-processors/src/main/java/org/apache/nifi/processors/aws/iot/AbstractAWSIoTProcessor.java
---
@@ -0,0
Github user trixpan commented on a diff in the pull request:
https://github.com/apache/nifi/pull/1521#discussion_r101915884
--- Diff:
nifi-nar-bundles/nifi-aws-bundle/nifi-aws-processors/src/main/java/org/apache/nifi/processors/aws/iot/AbstractAWSIoTProcessor.java
---
@@ -0,0
Github user trixpan commented on the issue:
https://github.com/apache/nifi/pull/1521
JPercivall
Does AWS IoT support communicating over SSL with MQTT or shadows? If so,
that option should be added.
---
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/1521
**JPercivall**
The more I type out and try to easily distinguish GetAWSIoT/PutAWSIoT and
GetAWSIoTShadow/PutAWSIoTShadow the more I think MQTT should be integrated into
the name and all four of
Github user trixpan commented on the issue:
https://github.com/apache/nifi/pull/1521
**JPercivall**
I actually already committed general MQTT processors [1] and they have some
good "best practices" with regards to publishing/consuming messages and MQTT I
think wou
Github user trixpan commented on the issue:
https://github.com/apache/nifi/pull/1521
**JPercivall**
A bit nit picky and I may be alone here, but I find putting "if" statements
and their content on the same line a bit hard to read when trying to read
someone else
Github user trixpan commented on a diff in the pull request:
https://github.com/apache/nifi/pull/1521#discussion_r101916002
--- Diff:
nifi-nar-bundles/nifi-aws-bundle/nifi-aws-processors/src/main/java/org/apache/nifi/processors/aws/iot/util/MqttWebSocketAsyncClient.java
Github user trixpan commented on the issue:
https://github.com/apache/nifi/pull/1521
**JPercivall**
For the new dependencies, NOTICE/LICENSE information needs to be added if
applicable. Paho is EPL 1.0 licensed and requires being added to the nar NOTICE
like so[1]. Do the other
Github user trixpan commented on a diff in the pull request:
https://github.com/apache/nifi/pull/1521#discussion_r101916052
--- Diff:
nifi-nar-bundles/nifi-aws-bundle/nifi-aws-processors/src/main/java/org/apache/nifi/processors/aws/iot/AbstractAWSIoTProcessor.java
---
@@ -0,0
Github user trixpan commented on a diff in the pull request:
https://github.com/apache/nifi/pull/1521#discussion_r101916039
--- Diff:
nifi-nar-bundles/nifi-aws-bundle/nifi-aws-processors/src/main/java/org/apache/nifi/processors/aws/iot/AbstractAWSIoTProcessor.java
---
@@ -0,0
Github user trixpan commented on the issue:
https://github.com/apache/nifi/pull/1521
**JPercivall**
The Get/PutAWSIoT processors read and overwrite mqttClient many times
without a read/write lock, this can lead to problems in multi-threaded
scenarios. For example, the both
Github user trixpan commented on a diff in the pull request:
https://github.com/apache/nifi/pull/1521#discussion_r101916076
--- Diff:
nifi-nar-bundles/nifi-aws-bundle/nifi-aws-processors/src/main/java/org/apache/nifi/processors/aws/iot/AbstractAWSIoTProcessor.java
---
@@ -0,0
Github user trixpan commented on a diff in the pull request:
https://github.com/apache/nifi/pull/1521#discussion_r101916101
--- Diff:
nifi-nar-bundles/nifi-aws-bundle/nifi-aws-processors/src/main/java/org/apache/nifi/processors/aws/iot/util/IoTMessage.java
---
@@ -0,0 +1,44
Github user trixpan commented on a diff in the pull request:
https://github.com/apache/nifi/pull/1521#discussion_r101916129
--- Diff:
nifi-nar-bundles/nifi-aws-bundle/nifi-aws-processors/src/main/java/org/apache/nifi/processors/aws/iot/PutAWSIoTShadow.java
---
@@ -0,0 +1,107
Github user trixpan commented on a diff in the pull request:
https://github.com/apache/nifi/pull/1521#discussion_r101916202
--- Diff:
nifi-nar-bundles/nifi-aws-bundle/nifi-aws-processors/src/main/java/org/apache/nifi/processors/aws/iot/GetAWSIoTShadow.java
---
@@ -0,0 +1,114
Github user trixpan commented on a diff in the pull request:
https://github.com/apache/nifi/pull/1521#discussion_r101916245
--- Diff:
nifi-nar-bundles/nifi-aws-bundle/nifi-aws-processors/src/main/java/org/apache/nifi/processors/aws/iot/GetAWSIoTShadow.java
---
@@ -0,0 +1,114
Github user trixpan commented on the issue:
https://github.com/apache/nifi/pull/1521
**JPercivall**
A couple places have weird comments like "// @TRACE 252=connect to host {0}
port {1} timeout {2}". Is this a convention I'm not aware of or left over from
copy/
Github user trixpan commented on the issue:
https://github.com/apache/nifi/pull/1521
**JPercivall**
Currently the only tests are integration tests that rely on an AWS
instance. I would like to see more true unit tests. For inspiration, check out
the Publish/Consume MQTT unit
Github user trixpan commented on the issue:
https://github.com/apache/nifi/pull/1331
@apsaltis - glad comments were helpful.
New PR or not, the work is truly great and I can confess that with the
exception of Azure components I will very likely be using it the outcome of the
Github user trixpan commented on the issue:
https://github.com/apache/nifi/pull/1521
@JPercivall The code, being based on 0.7 wouldn't compile cleanly when
merged with 1.x so I am working in getting it to compile.
I also I think I transferred most of the comments acros
Github user trixpan commented on the issue:
https://github.com/apache/nifi/pull/1521
@JPercivall @joewitt
The original PR seems to contain significant sections of code that are
close to an old version of the following code:
https://github.com/inventit/mqtt
Github user trixpan commented on the issue:
https://github.com/apache/nifi/pull/1521
https://github.com/aws/aws-iot-device-sdk-java
---
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/1521
To further my above comments, it seems that shortly after the original PR,
AWS released an IoT SDK for Java under ASL2 license.
I vote we close this PR and restart work using AWS' ow
Github user trixpan commented on the issue:
https://github.com/apache/nifi/pull/1521
Close due to reasons presented above
---
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 closed the pull request at:
https://github.com/apache/nifi/pull/1521
---
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/349
Thabk Kay, would it be possible to 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 not have this
Github user trixpan commented on the issue:
https://github.com/apache/nifi/pull/1395
@nickcarenza - welcome. We all have been in that very same boat once.
:smiley:
---
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 trixpan commented on the issue:
https://github.com/apache/nifi/pull/1378
@pvillard31 :rofl: yeah... wrong PR but glad you got it
---
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/1240
@bbende @markap14 or @joewitt
I suspect this PR and its 0.x counterpart will benefit from having one of
you reviewing it?
---
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/1519
@mcgilman any chance of having a look on 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 not have this
Github user trixpan commented on the issue:
https://github.com/apache/nifi/pull/379
@pvillard31 not sure if it is the case but have you by any chance tried to
use the Hadoop alluxio interfaces with puthdfs and co instead of a new
processor?
e.g
https
Github user trixpan commented on the issue:
https://github.com/apache/nifi/pull/379
And by last comment I naturally refer to use a customised core-site.xml
that includes:
```
fs.alluxio.impl
alluxio.hadoop.FileSystem
The Alluxio FileSystem (Hadoop 1.x
Github user trixpan commented on the issue:
https://github.com/apache/nifi/pull/379
@pvillard31 that connection strategy is agnostic to the underlying
filesystem supporting alluxio. What it does is to allow a hadoop compatible
application to access alluxio without further coding and
GitHub user trixpan opened a pull request:
https://github.com/apache/nifi/pull/1526
NIFI-3514 - Ensures ListGCPBucket triggers serially and forbids inputs
Thank you for submitting a contribution to Apache NiFi.
In order to streamline the review of the contribution we ask
Github user trixpan commented on the issue:
https://github.com/apache/nifi/pull/1526
@jvwing @gene-telligent - would you mind having a look ?
---
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/1529
@WilliamNouet to squash commit just checkout your branch and run:
`git rebase -i`
and follow the squash process as very well exemplified here:
http://gitready.com/advanced
Github user trixpan commented on a diff in the pull request:
https://github.com/apache/nifi/pull/1529#discussion_r102853739
--- Diff:
nifi-nar-bundles/nifi-morphlines-bundle/nifi-morphlines-processor/pom.xml ---
@@ -0,0 +1,130 @@
+
+
+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/1529#discussion_r102854106
--- Diff: nifi-nar-bundles/nifi-morphlines-bundle/pom.xml ---
@@ -0,0 +1,39 @@
+
+
+http://maven.apache.org/POM/4.0.0";
xmlns:xsi="htt
Github user trixpan commented on a diff in the pull request:
https://github.com/apache/nifi/pull/1529#discussion_r102854253
--- Diff:
nifi-nar-bundles/nifi-morphlines-bundle/nifi-morphlines-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/1529#discussion_r102854353
--- Diff: nifi-nar-bundles/nifi-kite-bundle/nifi-kite-processors/pom.xml ---
@@ -40,7 +40,21 @@
org.apache.nifi
nifi-utils
Github user trixpan commented on a diff in the pull request:
https://github.com/apache/nifi/pull/1529#discussion_r102855578
--- Diff:
nifi-nar-bundles/nifi-morphlines-bundle/nifi-morphlines-processor/pom.xml ---
@@ -0,0 +1,130 @@
+
+
+http://maven.apache.org/POM/4.0.0
Github user trixpan commented on the issue:
https://github.com/apache/nifi/pull/1529
@WilliamNouet - thank you for the contribution. Had a quick look and added
some comments. Other will probably want to have a look as well.
---
If your project is set up for it, you can reply to this
Github user trixpan commented on a diff in the pull request:
https://github.com/apache/nifi/pull/1529#discussion_r102856855
--- Diff:
nifi-nar-bundles/nifi-morphlines-bundle/nifi-morphlines-processor/src/main/java/org/apache/nifi/processors/morphlines/Morphlines.java
---
@@ -0,0
Github user trixpan commented on a diff in the pull request:
https://github.com/apache/nifi/pull/1529#discussion_r102860592
--- Diff:
nifi-nar-bundles/nifi-morphlines-bundle/nifi-morphlines-processor/src/main/java/org/apache/nifi/processors/morphlines/Morphlines.java
---
@@ -0,0
Github user trixpan commented on a diff in the pull request:
https://github.com/apache/nifi/pull/1529#discussion_r102860737
--- Diff:
nifi-nar-bundles/nifi-morphlines-bundle/nifi-morphlines-processor/src/main/java/org/apache/nifi/processors/morphlines/Morphlines.java
---
@@ -0,0
Github user trixpan commented on the issue:
https://github.com/apache/nifi/pull/1528
@WilliamNouet this looks like a duplicated PR from the other one
introducing Morphlines? If so, mind to close it?
---
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/1528
Thanks for the heads up @pvillard31
---
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 opened a pull request:
https://github.com/apache/nifi/pull/1541
NIFI-329 - Introduce IRC Client Services and ConsumeIRC processor
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/1541
this PR still not ready to review, however given this is the first time I
am putting together a ControllerService I would appreciate some early feedback.
As you can see, design for the
Github user trixpan commented on the issue:
https://github.com/apache/nifi/pull/1541
also note that the easier way of setting up an IRC for testing is to spin
up miniircd, which works like a charm for basic testing
---
If your project is set up for it, you can reply to this email
Github user trixpan commented on a diff in the pull request:
https://github.com/apache/nifi/pull/1418#discussion_r103599319
--- 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_r103599241
--- 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_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/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_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/1418
@mattyb149 - hopefully addressed your feedback.
---
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/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";
xmln
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";
xmln
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. If
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 commented on the issue:
https://github.com/apache/nifi/pull/1551
@alopresto may I ask your opinion on 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 feature
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
to
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 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/1553#discussion_r104021856
--- Diff:
nifi-nar-bundles/nifi-standard-bundle/nifi-standard-processors/src/test/java/org/apache/nifi/processors/standard/TestPutEmail.java
---
@@ -230,27
Github user trixpan commented on a diff in the pull request:
https://github.com/apache/nifi/pull/1553#discussion_r104027407
--- Diff:
nifi-nar-bundles/nifi-standard-bundle/nifi-standard-processors/src/test/java/org/apache/nifi/processors/standard/TestPutEmail.java
---
@@ -230,27
Github user trixpan commented on a diff in the pull request:
https://github.com/apache/nifi/pull/1553#discussion_r104027988
--- Diff:
nifi-nar-bundles/nifi-standard-bundle/nifi-standard-processors/src/test/java/org/apache/nifi/processors/standard/TestPutEmail.java
---
@@ -102,13
Github user trixpan commented on the issue:
https://github.com/apache/nifi/pull/1543
@joetrite is there a chance you can squash this commit into a single commit?
---
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/1553#discussion_r104113002
--- Diff:
nifi-nar-bundles/nifi-standard-bundle/nifi-standard-processors/src/test/java/org/apache/nifi/processors/standard/TestPutEmail.java
---
@@ -230,27
Github user trixpan commented on a diff in the pull request:
https://github.com/apache/nifi/pull/1553#discussion_r104114824
--- Diff:
nifi-nar-bundles/nifi-standard-bundle/nifi-standard-processors/src/test/java/org/apache/nifi/processors/standard/TestPutEmail.java
---
@@ -230,27
Github user trixpan commented on a diff in the pull request:
https://github.com/apache/nifi/pull/1553#discussion_r104143931
--- Diff:
nifi-nar-bundles/nifi-standard-bundle/nifi-standard-processors/src/test/java/org/apache/nifi/processors/standard/TestPutEmail.java
---
@@ -230,27
Github user trixpan commented on a diff in the pull request:
https://github.com/apache/nifi/pull/1553#discussion_r104170950
--- Diff:
nifi-nar-bundles/nifi-standard-bundle/nifi-standard-processors/src/test/java/org/apache/nifi/processors/standard/TestPutEmail.java
---
@@ -230,27
Github user trixpan commented on the issue:
https://github.com/apache/nifi/pull/1553
Thanks for the assistance previously, it should be ready for 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
GitHub user trixpan opened a pull request:
https://github.com/apache/nifi/pull/1563
NIFI- - Renames jBcrypt dependency to jBCrypt
Thank you for submitting a contribution to Apache NiFi.
In order to streamline the review of the contribution we ask you
to ensure the
Github user trixpan commented on the issue:
https://github.com/apache/nifi/pull/1563
merged pere's patch
---
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 wish
Github user trixpan closed the pull request at:
https://github.com/apache/nifi/pull/1563
---
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/1551
@joewitt thanks for your comments.
I understand your concerns but isn't the user change of the default
settings (we do ship with default location settings after all) a voluntary
a
Github user trixpan commented on the issue:
https://github.com/apache/nifi/pull/1551
@joewitt. Thanks for the comment. I will go through your notes and adjust
the PR accordingly.
---
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/1581
@apsaltis LGTM but is there chance of making reference to proxyuser groups/
proxyuser hosts in the description or additional info? HDFS Impersonation is
one of those thing people always get wrong the
GitHub user trixpan opened a pull request:
https://github.com/apache/nifi/pull/1614
NIFI-3466 - Addresses ParseCEF Locale related failures and introduce
ability to define Locale of logs
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/1614
@pvillard31 @apiri this should hopefully address the Locale related errors
affecting ParseCEF.
---
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/1614
also please note there seem to be an unrelated issue affecting another test
unit that fails unter pt_BR and fr_FR but passes ja_JP. I suspect it is a comma
vs dot number notation issue
Github user trixpan commented on the issue:
https://github.com/apache/nifi/pull/1614
please ignore the last comment regarding the error above, seems to be a
rebasing issue.
---
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 opened a pull request:
https://github.com/apache/nifi/pull/1619
NIFI-2747 - Introduce FuzzyHashContent processor
Thank you for submitting a contribution to Apache NiFi.
In order to streamline the review of the contribution we ask you
to ensure the
401 - 500 of 761 matches
Mail list logo