Github user trixpan commented on a diff in the pull request:
https://github.com/apache/nifi/pull/1462#discussion_r99038757
--- Diff:
nifi-nar-bundles/nifi-standard-bundle/nifi-standard-processors/src/main/java/org/apache/nifi/processors/standard/GetHTTP.java
---
@@ -467,6 +486,18
GitHub user trixpan opened a pull request:
https://github.com/apache/nifi-site/pull/11
NIFI-1448 - Update NiFi and MiNiFi site to use the new ASF branding
@apiri can you have a look at this PR?
You can merge this pull request into a Git repository by running:
$ git pull https
Github user trixpan commented on the issue:
https://github.com/apache/nifi/pull/1016
@brianburnett can't you simply rebase?
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 opened a pull request:
https://github.com/apache/nifi/pull/1480
NIFI-1657 - Configure travis-ci to run mvn verify in jp and fr
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/1480
@pvillard31 I changed the commit a bit so to use envs (I think things
should move faster this way) I reckon we should merge and use it to correct
residual issues. This second attempt should hopefully
Github user trixpan commented on the issue:
https://github.com/apache/nifi/pull/1480
by the way, just in case someone ever wonder the rationale behind the
countries: basic English (Java's default env), a Latin and a "Asian" language
with no official status at the UN.
-
Github user trixpan commented on the issue:
https://github.com/apache/nifi/pull/1480
@pvillard31 good catch!
---
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/1480
Only if you commit to port NiFi to minitel. :rofl: ...
Just kidding, done
---
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 the issue:
https://github.com/apache/nifi/pull/1480
@pvillard31 seems good to go. failures were caused by TCP processors
---
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/1462
@apiri just pushed a refreshed version of the code. let me know how 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
Github user trixpan commented on the issue:
https://github.com/apache/nifi/pull/1473
still not ready to commit as sftp and ftp modification have not yet been
added to commit. Hope to finish those in the next few days.
---
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/1473
NIFI-940 - Exposes filename attribute to DFM on PutFile, PutHDFS, Putâ¦
â¦SFTP, PutFTP
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/1474
@markap14 you may want to have a look
note that in the case of transfer method I simply removed the EL
evaluation. Happy to follow the opposite approach if needed
---
If your project is set
Github user trixpan commented on the issue:
https://github.com/apache/nifi/pull/1108
@joewitt when you have time can you have a final look on this one? LGTM and
I am happy to address the two remaining cosmetic comments as part of a separate
PR.
---
If your project is set up
Github user trixpan commented on the issue:
https://github.com/apache/nifi/pull/1472
@pvillard31 @apiri done.
Please note I added a second commit as this helps keeping track of why the
travis file was changed.
---
If your project is set up for it, you can reply
Github user trixpan commented on the issue:
https://github.com/apache/nifi/pull/1418
@nickcarenza thanks for the 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 feature
enabled
Github user trixpan commented on the issue:
https://github.com/apache/nifi/pull/1418
For future reference:
https://discuss.elastic.co/t/testers-wanted-beat-integration-with-apache-nifi/73671/
---
If your project is set up for it, you can reply to this email and have your
GitHub user trixpan opened a pull request:
https://github.com/apache/nifi/pull/1472
NIFI-3434 - Introduce the ability to skip the packaging of zip and taâ¦
â¦r.gz archive via maven profile (i.e. -Pdir-only)
Thank you for submitting a contribution to Apache NiFi
Github user trixpan commented on the issue:
https://github.com/apache/nifi/pull/1467
@gglanzani - thank you for the contribution. Code looks great but would it
be possible for you to extend the test harness to include a test that will
trigger your code(i.e. failure
Github user trixpan commented on a diff in the pull request:
https://github.com/apache/nifi/pull/1422#discussion_r97304871
--- Diff: nifi-assembly/pom.xml ---
@@ -19,6 +19,10 @@ language governing permissions and limitations under the
License. -->
nifi-assem
Github user trixpan commented on the issue:
https://github.com/apache/nifi/pull/1421
@joewitt
sure thing:
https://github.com/elastic/logstash-forwarder/blob/master/PROTOCOL.md#ack-frame-type
---
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 reviewing 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 this feature
enabled
Github user trixpan commented on the issue:
https://github.com/apache/nifi/pull/1335
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 feature
enabled and wishes so
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
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 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/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/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
Github user trixpan commented on the issue:
https://github.com/apache/nifi/pull/1148
@mauriziocolleluori thank you for the contribution.
any intent on addressing @pvillard31 's comment? While a positive answer is
more than welcome, a negative answer is also welcome... all we
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 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 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_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 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/1387#discussion_r101913412
--- Diff:
nifi-nar-bundles/nifi-hadoop-bundle/nifi-hdfs-processors/src/main/java/org/apache/nifi/processors/hadoop/CreateHadoopSequenceFile.java
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 well
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**
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/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
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 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 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/1395
@nickcarenza thank you for the contribution. May I ask you to squash the
commit as requested by the PR template?
` Is your initial contribution a single, squashed commit?`
At least
Github user trixpan commented on the issue:
https://github.com/apache/nifi/pull/1331
@apsaltis - loved the idea behind this contribution - I'm also very much in
favor of fitting MiNiFi with existing systems - and perhaps that's the reason
for my first question
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
**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 e
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 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 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
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_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 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
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
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' own SDK
Github user trixpan commented on the issue:
https://github.com/apache/nifi/pull/1290
@klinvill
Is this PR still needed or was it identified as a data quality issue?
---
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/1016
@brianburnett
I suspect the reason for that is because you are trying to cherry-pick from
the branch you already have a commit.
Note how I suggested you to circle around
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 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="http://w
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 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
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 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 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_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 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_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 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 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
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 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
Github user trixpan commented on the issue:
https://github.com/apache/nifi/pull/1335
@ijokarumawak merged. However I forgot to add reference to the PR in the
commit message. Would you mind closing it manually ?
I thank you in advance
---
If your project is set up
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
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 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 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
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
Github user trixpan commented on the issue:
https://github.com/apache/nifi/pull/689
@markap14 @JPercivall
Given we getting well into 1.2, should this PR still go ahead?
---
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/913
@jskora closed it.
Whenever this happens you can simply push an empty commit to git with the
fixed reference to the PR
---
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/709
@simonellistonball ?
---
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/1328
@JPercivall - needs rebasing again. :-(
---
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/1480
@pvillard31 - this seems to be working now
---
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/1370
@pvillard31 LGTM :+1:
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and wishes so
Github user trixpan commented on the issue:
https://github.com/apache/nifi/pull/1474
@alopresto - the missing amendments have been uploaded and hopefully also
addressed the comments made earlier
---
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/1474
PR still not complete Andy. My bad
---
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/1474#discussion_r101007316
--- Diff:
nifi-nar-bundles/nifi-standard-bundle/nifi-standard-processors/pom.xml ---
@@ -230,6 +230,12 @@ language governing permissions and limitations
GitHub user trixpan opened a pull request:
https://github.com/apache/nifi/pull/1505
NIFI-2565 - Add Elastic copyright to notice due to the use of grok patterns
during test
since the patterns used during testing were originally defined here
https://github.com/logstash
Github user trixpan commented on a diff in the pull request:
https://github.com/apache/nifi/pull/1474#discussion_r101007689
--- Diff:
nifi-nar-bundles/nifi-standard-bundle/nifi-standard-processors/src/test/java/org/apache/nifi/processors/standard/TestPutFTP.java
---
@@ -0,0 +1,90
Github user trixpan commented on a diff in the pull request:
https://github.com/apache/nifi/pull/1474#discussion_r101007664
--- Diff:
nifi-nar-bundles/nifi-standard-bundle/nifi-standard-processors/src/test/java/org/apache/nifi/processors/standard/TestPutFTP.java
---
@@ -0,0 +1,90
Github user trixpan commented on the issue:
https://github.com/apache/nifi/pull/1480
by the way, good catch, I had faced this issue before but totally forgot
about it.
---
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 the issue:
https://github.com/apache/nifi/pull/1480
seems great. will do it and force push soon
---
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/1480
@pvillard31 - after a few hundred builds and options build seems to be
having as expected:
https://travis-ci.org/trixpan/nifi/builds/200568234
when environment is set to en_US
Github user trixpan commented on the issue:
https://github.com/apache/nifi/pull/1383
@pvillard31 @bbende 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
Github user trixpan commented on the issue:
https://github.com/apache/nifi/pull/1481
@mosermw ohhh. wasn't aware of it. thanks for sharing
---
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/1505
@apiri hopefully this time it looks a bit better... yes... it is a fork of
a fork. :crying_cat_face:
---
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/1326
you did a merge and that dragged dozens of committed commits into this PR.
Instead you may want to do:
```
git checkout master -b NIFI-2779v2
git cherry-pick
git push
101 - 200 of 658 matches
Mail list logo