Github user olegz commented on the issue:
https://github.com/apache/nifi/pull/710
@trixpan Well, I am not sure what that is (will be looking) and how
reliable mailtrap.io is for such testing giving it's description "Fake smtp
testing server. Dummy smtp email testing". I
Github user olegz commented on the issue:
https://github.com/apache/nifi/pull/710
Digging into the source code for it looks like something is out of
compliance with POP3 protocol
```java
if (line.startsWith("+OK"))
r.ok = true;
else if (line.
Github user olegz commented on the issue:
https://github.com/apache/nifi/pull/710
One advise though is that there are additional JavaMail properties that one
may need to provide to accomodate specific servers. For example in Gmail you
need to specify
```mail.pop3.socketFactory.class
Github user olegz commented on the issue:
https://github.com/apache/nifi/pull/563
@JPercivall +1 on Kafka stuff
---
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 olegz commented on the issue:
https://github.com/apache/nifi/pull/687
@xmlking Back Pressure is the function of all processors so there is
nothing specific required in ConsumeKafka to handle it. Does that answer your
question?
---
If your project is set up for it, you
GitHub user olegz opened a pull request:
https://github.com/apache/nifi/pull/741
NIFI-2322, NIFI-2423, NIFI-2412 Kafka improvements
- Fixed KafkaConsumer's connection block when broker is not available
- Fixed Serializer/Deserializer configs in both Consume/Publish Kafka
Github user olegz commented on a diff in the pull request:
https://github.com/apache/nifi/pull/739#discussion_r72802000
--- Diff:
nifi-nar-bundles/nifi-framework-bundle/nifi-framework/nifi-framework-core/src/main/java/org/apache/nifi/controller/FlowController.java
---
@@ -3279,14
Github user olegz commented on a diff in the pull request:
https://github.com/apache/nifi/pull/718#discussion_r72811314
--- Diff:
nifi-nar-bundles/nifi-framework-bundle/nifi-framework/nifi-web/nifi-web-api/src/test/java/org/apache/nifi/web/util/SnippetUtilsTest.java
---
@@ -0,0
GitHub user olegz opened a pull request:
https://github.com/apache/nifi/pull/758
NIFI-2444 NIFI-2445 fixed PublishKafka
- fixed the logging issue NIFI-2444 by ensuring the ProcessLog is added to
KafkaPublisher
- fixed KafkaPublisher's isAllAcked operation to ensure
GitHub user olegz opened a pull request:
https://github.com/apache/nifi/pull/761
NIFI-2444 NIFI-2445 fixed PublishKafka
- fixed the logging issue NIFI-2444 by ensuring the ProcessLog is added to
KafkaPublisher
- fixed KafkaPublisher's isAllAcked operation to ensure
Github user olegz commented on the issue:
https://github.com/apache/nifi/pull/828
Looks pretty straight forward. Validated that it did not break the
correctness of template import/export. Merging
---
If your project is set up for it, you can reply to this email and have your
reply
Github user olegz commented on the issue:
https://github.com/apache/nifi/pull/828
Reviewing.
---
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 olegz opened a pull request:
https://github.com/apache/nifi/pull/770
NIFI-2425 REVIEW ONLY
You can merge this pull request into a Git repository by running:
$ git pull https://github.com/olegz/nifi NIFI-2425B
Alternatively you can review and apply these changes
Github user olegz commented on a diff in the pull request:
https://github.com/apache/nifi/pull/727#discussion_r73254929
--- Diff:
nifi-nar-bundles/nifi-hive-bundle/nifi-hive-processors/src/main/java/org/apache/nifi/processors/hive/PutHiveStreaming.java
---
@@ -0,0 +1,657
Github user olegz commented on a diff in the pull request:
https://github.com/apache/nifi/pull/727#discussion_r73254901
--- Diff:
nifi-nar-bundles/nifi-hive-bundle/nifi-hive-processors/src/main/java/org/apache/nifi/processors/hive/PutHiveStreaming.java
---
@@ -0,0 +1,657
Github user olegz commented on the issue:
https://github.com/apache/nifi/pull/741
ouch, the code that provided the fix for connection broke the test.
Argh Give me a min
---
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 olegz commented on the issue:
https://github.com/apache/nifi/pull/761
@markap14 done
---
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 olegz commented on a diff in the pull request:
https://github.com/apache/nifi/pull/727#discussion_r73254533
--- Diff:
nifi-nar-bundles/nifi-hive-bundle/nifi-hive-processors/src/main/java/org/apache/nifi/dbcp/hive/HiveConnectionPool.java
---
@@ -160,22 +162,9
Github user olegz commented on a diff in the pull request:
https://github.com/apache/nifi/pull/727#discussion_r73253954
--- Diff:
nifi-nar-bundles/nifi-hive-bundle/nifi-hive-processors/src/main/java/org/apache/hadoop/hive/ql/io/orc/NiFiOrcUtils.java
---
@@ -0,0 +1,466
Github user olegz commented on a diff in the pull request:
https://github.com/apache/nifi/pull/727#discussion_r73254202
--- Diff:
nifi-nar-bundles/nifi-hive-bundle/nifi-hive-processors/src/main/java/org/apache/hadoop/hive/ql/io/orc/NiFiOrcUtils.java
---
@@ -0,0 +1,466
Github user olegz commented on the issue:
https://github.com/apache/nifi/pull/761
@markap14 @brosander done
---
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 olegz commented on a diff in the pull request:
https://github.com/apache/nifi/pull/727#discussion_r73255530
--- Diff:
nifi-nar-bundles/nifi-hive-bundle/nifi-hive-processors/src/main/java/org/apache/nifi/processors/hive/PutHiveStreaming.java
---
@@ -0,0 +1,657
Github user olegz commented on a diff in the pull request:
https://github.com/apache/nifi/pull/727#discussion_r73255367
--- Diff:
nifi-nar-bundles/nifi-hive-bundle/nifi-hive-processors/src/main/java/org/apache/nifi/processors/hive/PutHiveStreaming.java
---
@@ -0,0 +1,657
Github user olegz commented on a diff in the pull request:
https://github.com/apache/nifi/pull/727#discussion_r73252749
--- Diff:
nifi-nar-bundles/nifi-hive-bundle/nifi-hive-processors/src/main/java/org/apache/hadoop/hive/ql/io/orc/NiFiOrcUtils.java
---
@@ -0,0 +1,466
Github user olegz commented on the issue:
https://github.com/apache/nifi/pull/727
Reviewing
---
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 olegz commented on the issue:
https://github.com/apache/nifi/pull/821
+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, or if the feature
Github user olegz commented on the issue:
https://github.com/apache/nifi/pull/821
Reviewing
---
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 olegz commented on a diff in the pull request:
https://github.com/apache/nifi/pull/827#discussion_r74417054
--- Diff:
nifi-nar-bundles/nifi-email-bundle/nifi-email-processors/src/main/java/org/apache/nifi/processors/email/ListenSMTP.java
---
@@ -135,29 +89,17
Github user olegz commented on a diff in the pull request:
https://github.com/apache/nifi/pull/827#discussion_r74418537
--- Diff:
nifi-nar-bundles/nifi-email-bundle/nifi-email-processors/src/main/java/org/apache/nifi/processors/email/ListenSMTP.java
---
@@ -106,26 +69,17
Github user olegz commented on a diff in the pull request:
https://github.com/apache/nifi/pull/827#discussion_r74422525
--- Diff:
nifi-nar-bundles/nifi-email-bundle/nifi-email-processors/src/main/java/org/apache/nifi/processors/email/ListenSMTP.java
---
@@ -106,26 +69,17
Github user olegz commented on a diff in the pull request:
https://github.com/apache/nifi/pull/827#discussion_r74421827
--- Diff:
nifi-nar-bundles/nifi-email-bundle/nifi-email-processors/src/main/java/org/apache/nifi/processors/email/ListenSMTP.java
---
@@ -135,29 +89,17
Github user olegz commented on a diff in the pull request:
https://github.com/apache/nifi/pull/827#discussion_r74425869
--- Diff:
nifi-nar-bundles/nifi-email-bundle/nifi-email-processors/src/main/java/org/apache/nifi/processors/email/ListenSMTP.java
---
@@ -135,29 +89,17
Github user olegz commented on a diff in the pull request:
https://github.com/apache/nifi/pull/827#discussion_r74425691
--- Diff:
nifi-nar-bundles/nifi-email-bundle/nifi-email-processors/src/main/java/org/apache/nifi/processors/email/ListenSMTP.java
---
@@ -106,26 +69,17
Github user olegz commented on the issue:
https://github.com/apache/nifi/pull/827
i see now. Putting it in
---
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 olegz commented on the issue:
https://github.com/apache/nifi/pull/827
Ok @joewitt @trixpan latest comments are addressed in the latest commit.
Please review
---
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 olegz commented on the issue:
https://github.com/apache/nifi/pull/827
@trixpan i think you've closed client while it was in he middle of reading
InputStream. If that's the case then yes the stack trace above is exactly what
one would expect.
---
If your project is set
Github user olegz commented on the issue:
https://github.com/apache/nifi/pull/827
The above is actually correct and is exactly what we were aiming for. No
data loss. There is actually new test that validates for that exact message. So
all is good. User is notified that something went
Github user olegz commented on the issue:
https://github.com/apache/nifi/pull/827
@trixpan just confirmed, the "Read timed out" exception is the correct
behavior, but I will wrap it into a nicer log message. But just to confirm the
behavior is expected
---
If your project
Github user olegz commented on the issue:
https://github.com/apache/nifi/pull/827
@trixpan could you please describe scenario that triggered the above
exception?
---
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 olegz commented on the issue:
https://github.com/apache/nifi/pull/827
@trixpan with regard to "Read timeout" I've just added better exception
handling to notify user
https://github.com/apache/nifi/pull/827/commits/406b37b249e487f73bae5a260bd5aed05d3
Github user olegz commented on a diff in the pull request:
https://github.com/apache/nifi/pull/827#discussion_r74361483
--- Diff:
nifi-nar-bundles/nifi-email-bundle/nifi-email-processors/src/main/java/org/apache/nifi/processors/email/ListenSMTP.java
---
@@ -106,26 +69,17
Github user olegz commented on a diff in the pull request:
https://github.com/apache/nifi/pull/741#discussion_r73352158
--- Diff:
nifi-nar-bundles/nifi-kafka-bundle/nifi-kafka-pubsub-processors/src/main/java/org/apache/nifi/processors/kafka/pubsub/AbstractKafkaProcessor.java
Github user olegz commented on a diff in the pull request:
https://github.com/apache/nifi/pull/827#discussion_r74527137
--- Diff:
nifi-nar-bundles/nifi-email-bundle/nifi-email-processors/src/main/java/org/apache/nifi/processors/email/ListenSMTP.java
---
@@ -106,26 +90,17
Github user olegz closed the pull request at:
https://github.com/apache/nifi/pull/827
---
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 olegz commented on a diff in the pull request:
https://github.com/apache/nifi/pull/836#discussion_r74704476
--- Diff:
nifi-nar-bundles/nifi-framework-bundle/nifi-framework/nifi-framework-core/src/main/java/org/apache/nifi/controller/StandardProcessorNode.java
Github user olegz commented on a diff in the pull request:
https://github.com/apache/nifi/pull/836#discussion_r74704726
--- Diff:
nifi-nar-bundles/nifi-framework-bundle/nifi-framework/nifi-framework-core/src/main/java/org/apache/nifi/controller/StandardProcessorNode.java
Github user olegz commented on the issue:
https://github.com/apache/nifi/pull/856
+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, or if the feature
Github user olegz commented on a diff in the pull request:
https://github.com/apache/nifi/pull/827#discussion_r74362408
--- Diff:
nifi-nar-bundles/nifi-email-bundle/nifi-email-processors/src/main/java/org/apache/nifi/processors/email/ListenSMTP.java
---
@@ -166,317 +108,158
Github user olegz commented on a diff in the pull request:
https://github.com/apache/nifi/pull/827#discussion_r74361725
--- Diff:
nifi-nar-bundles/nifi-email-bundle/nifi-email-processors/src/main/java/org/apache/nifi/processors/email/ListenSMTP.java
---
@@ -13,89 +13,52
Github user olegz commented on a diff in the pull request:
https://github.com/apache/nifi/pull/827#discussion_r74361819
--- Diff:
nifi-nar-bundles/nifi-email-bundle/nifi-email-processors/src/main/java/org/apache/nifi/processors/email/ListenSMTP.java
---
@@ -166,317 +108,158
Github user olegz commented on a diff in the pull request:
https://github.com/apache/nifi/pull/827#discussion_r74362947
--- Diff:
nifi-nar-bundles/nifi-email-bundle/nifi-email-processors/src/main/java/org/apache/nifi/processors/email/ListenSMTP.java
---
@@ -166,317 +108,158
Github user olegz commented on a diff in the pull request:
https://github.com/apache/nifi/pull/827#discussion_r74361967
--- Diff:
nifi-nar-bundles/nifi-email-bundle/nifi-email-processors/src/main/java/org/apache/nifi/processors/email/ListenSMTP.java
---
@@ -166,317 +108,158
Github user olegz commented on a diff in the pull request:
https://github.com/apache/nifi/pull/827#discussion_r74362870
--- Diff:
nifi-nar-bundles/nifi-email-bundle/nifi-email-processors/src/main/java/org/apache/nifi/processors/email/ListenSMTP.java
---
@@ -13,89 +13,52
Github user olegz commented on a diff in the pull request:
https://github.com/apache/nifi/pull/827#discussion_r74362934
--- Diff:
nifi-nar-bundles/nifi-email-bundle/nifi-email-processors/src/main/java/org/apache/nifi/processors/email/ListenSMTP.java
---
@@ -106,26 +69,17
Github user olegz commented on the issue:
https://github.com/apache/nifi/pull/830
+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, or if the feature
GitHub user olegz opened a pull request:
https://github.com/apache/nifi/pull/851
NIFI-2560 added connection timeout to IMAP/POP3
You can merge this pull request into a Git repository by running:
$ git pull https://github.com/olegz/nifi NIFI-2560
Alternatively you can review
GitHub user olegz opened a pull request:
https://github.com/apache/nifi/pull/679
NIFI-826 (part 3)
- fixed ID generation routine that was causing miss-identification of the
components
You can merge this pull request into a Git repository by running:
$ git pull https
Github user olegz closed the pull request at:
https://github.com/apache/nifi/pull/679
---
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 olegz opened a pull request:
https://github.com/apache/nifi/pull/687
NIFI-2298 added Kafka consume attributes to a FlowFile
You can merge this pull request into a Git repository by running:
$ git pull https://github.com/olegz/nifi NIFI-2298
Alternatively you can
Github user olegz commented on the issue:
https://github.com/apache/nifi/pull/563
@ijokarumawak I am not yet done, but good that you have rebased. I am just
tied up with another thing at the moment, but will be reviewing later on
---
If your project is set up for it, you can reply
Github user olegz commented on the issue:
https://github.com/apache/nifi/pull/617
@mcgilman @jtstorck could you please look at the second commit where I had
to ignore two Spoke tests. Basically two issues there.
1. Components are created without ID. With new code changes it now
Github user olegz commented on the issue:
https://github.com/apache/nifi/pull/710
@mattyb149 I was just trying to reproduce the issue with the hanging thread
you described above in relation to ConsumePop3, but I can't. Are you sure that
what you saw is related to this processor
Github user olegz closed the pull request at:
https://github.com/apache/nifi/pull/741
---
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 olegz commented on the issue:
https://github.com/apache/nifi/pull/710
@mattyb149 all the suggested changes have been addressed in the last
commit. I've also included in the additional documentation sample configuration
for GMail, so you can use it to play around/test
Github user olegz commented on the issue:
https://github.com/apache/nifi/pull/814
NOTE: Verified that this commit is cherry-pickable to 0.x branch
---
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 olegz opened a pull request:
https://github.com/apache/nifi/pull/814
NIFI-2515 fixed Kafka serialization/deserialization settings
You can merge this pull request into a Git repository by running:
$ git pull https://github.com/olegz/nifi NIFI-2515B
Alternatively
GitHub user olegz opened a pull request:
https://github.com/apache/nifi/pull/810
NIFI-2504 fixed template export for RPG
- ensured that RPG and connections to it are treated as special case where
instance ID is preserved
You can merge this pull request into a Git repository
GitHub user olegz opened a pull request:
https://github.com/apache/nifi/pull/804
NIFI-2508 fixed compilation errors in AbstractKafkaProcessor
NOTE: This is for 0.x only
You can merge this pull request into a Git repository by running:
$ git pull https://github.com/olegz/nifi
Github user olegz commented on a diff in the pull request:
https://github.com/apache/nifi/pull/710#discussion_r73805385
--- Diff:
nifi-nar-bundles/nifi-email-bundle/nifi-email-processors/src/main/resources/docs/org.apache.nifi.processors.email.ConsumeIMAP/additionalDetails.html
Github user olegz commented on the issue:
https://github.com/apache/nifi/pull/710
@mattyb149 your comments are addressed in the last commit. Thanks for
reviewing
---
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 olegz commented on a diff in the pull request:
https://github.com/apache/nifi/pull/710#discussion_r73805537
--- Diff:
nifi-nar-bundles/nifi-email-bundle/nifi-email-processors/src/main/resources/docs/org.apache.nifi.processors.email.ConsumeIMAP/additionalDetails.html
GitHub user olegz opened a pull request:
https://github.com/apache/nifi/pull/805
NIFI-2509 fixed NPE condition in KafkaConsumer
You can merge this pull request into a Git repository by running:
$ git pull https://github.com/olegz/nifi NIFI-2509
Alternatively you can review
Github user olegz commented on the issue:
https://github.com/apache/nifi/pull/805
NOTE: This PR must be applied to both master and 0.x branch. I've verified
that it is cherry-pickable.
---
If your project is set up for it, you can reply to this email and have your
reply appear
Github user olegz commented on a diff in the pull request:
https://github.com/apache/nifi/pull/710#discussion_r71973578
--- Diff:
nifi-nar-bundles/nifi-email-bundle/nifi-email-processors/src/main/java/org/apache/nifi/processors/email/AbstractEmailProcessor.java
---
@@ -0,0 +1,391
Github user olegz commented on a diff in the pull request:
https://github.com/apache/nifi/pull/710#discussion_r71975039
--- Diff:
nifi-nar-bundles/nifi-email-bundle/nifi-email-processors/src/main/java/org/apache/nifi/processors/email/AbstractEmailProcessor.java
---
@@ -0,0 +1,391
Github user olegz commented on a diff in the pull request:
https://github.com/apache/nifi/pull/710#discussion_r71975032
--- Diff:
nifi-nar-bundles/nifi-email-bundle/nifi-email-processors/src/main/java/org/apache/nifi/processors/email/AbstractEmailProcessor.java
---
@@ -0,0 +1,391
Github user olegz commented on a diff in the pull request:
https://github.com/apache/nifi/pull/710#discussion_r71975044
--- Diff:
nifi-nar-bundles/nifi-email-bundle/nifi-email-processors/src/main/java/org/apache/nifi/processors/email/AbstractEmailProcessor.java
---
@@ -0,0 +1,391
Github user olegz commented on a diff in the pull request:
https://github.com/apache/nifi/pull/718#discussion_r72815141
--- Diff:
nifi-nar-bundles/nifi-framework-bundle/nifi-framework/nifi-web/nifi-web-api/src/test/java/org/apache/nifi/web/util/SnippetUtilsTest.java
---
@@ -0,0
Github user olegz closed the pull request at:
https://github.com/apache/nifi/pull/804
---
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 olegz commented on the issue:
https://github.com/apache/nifi/pull/840
Revewing
---
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 olegz commented on a diff in the pull request:
https://github.com/apache/nifi/pull/840#discussion_r74786994
--- Diff:
nifi-nar-bundles/nifi-provenance-repository-bundle/nifi-persistent-provenance-repository/src/main/java/org/apache/nifi/provenance
GitHub user olegz opened a pull request:
https://github.com/apache/nifi/pull/859
NIFI-2546 fixed NPE in FlowSnippetDTO
fixed NPE caused by non-configured ControllerService
You can merge this pull request into a Git repository by running:
$ git pull https://github.com/olegz
Github user olegz commented on the issue:
https://github.com/apache/nifi/pull/840
@YolandaMDavis That said, please ignore the above comments for now since
they are really improvements of the existing code and are not related to the
problem at hand. I'll raise a separate JIRA
Github user olegz commented on the issue:
https://github.com/apache/nifi/pull/864
Reviewing
---
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 olegz commented on the issue:
https://github.com/apache/nifi/pull/840
+1. 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 olegz commented on the issue:
https://github.com/apache/nifi/pull/864
+1 Verified font changes, icons, alignment. 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
Github user olegz commented on a diff in the pull request:
https://github.com/apache/nifi/pull/871#discussion_r75180197
--- Diff:
nifi-nar-bundles/nifi-email-bundle/nifi-email-processors/src/main/java/org/apache/nifi/processors/email/smtp/SmtpConsumer.java
---
@@ -149,7 +149,7
Github user olegz commented on a diff in the pull request:
https://github.com/apache/nifi/pull/871#discussion_r75184556
--- Diff:
nifi-nar-bundles/nifi-email-bundle/nifi-email-processors/src/main/java/org/apache/nifi/processors/email/smtp/SmtpConsumer.java
---
@@ -149,7 +149,7
Github user olegz commented on the issue:
https://github.com/apache/nifi/pull/1449
@pvillard31 yes you can
```
runner.setProperty(MyProcessor.SOME_PROPERTY, "${some.attribute.key}");
Map<String, String> attributes = new HashMap<>();
attributes.p
Github user olegz commented on a diff in the pull request:
https://github.com/apache/nifi/pull/1436#discussion_r99842926
--- Diff:
nifi-nar-bundles/nifi-registry-bundle/nifi-registry-processors/src/main/java/org/apache/nifi/schemaregistry/processors
GitHub user olegz opened a pull request:
https://github.com/apache/nifi/pull/1435
NIFI-3345 Added support for simple AVRO/CSV/JSON transformers that utâ¦
â¦ilize external Schema
Added support for simple Key/Value Schema Registry as Controller Service
Added support
Github user olegz commented on the issue:
https://github.com/apache/nifi/pull/1435
Closing due to wrong commit message, will resubmit
---
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 olegz closed the pull request at:
https://github.com/apache/nifi/pull/1435
---
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 olegz opened a pull request:
https://github.com/apache/nifi/pull/1436
NIFI-3354 Added support for simple AVRO/CSV/JSON transformers that utâ¦
â¦ilize external Schema
Added support for simple Key/Value Schema Registry as Controller Service
Added support
Github user olegz commented on a diff in the pull request:
https://github.com/apache/nifi/pull/1434#discussion_r97349937
--- Diff:
nifi-nar-bundles/nifi-tcp-bundle/nifi-tcp-processors/src/main/java/org/apache/nifi/processors/gettcp/DisconnectListener.java
---
@@ -0,0 +1,6
Github user olegz commented on a diff in the pull request:
https://github.com/apache/nifi/pull/1434#discussion_r97348509
--- Diff:
nifi-nar-bundles/nifi-tcp-bundle/nifi-tcp-nar/src/main/resources/META-INF/NOTICE
---
@@ -0,0 +1,24 @@
+nifi-gettcp-nar
--- End diff
Github user olegz commented on a diff in the pull request:
https://github.com/apache/nifi/pull/1434#discussion_r97348408
--- Diff: nifi-nar-bundles/nifi-tcp-bundle/.gitignore ---
@@ -0,0 +1,16 @@
+target
+.project
+.settings
+.classpath
+nbactions.xml
+nb
Github user olegz commented on the issue:
https://github.com/apache/nifi/pull/1425
Reviewing. . .
---
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 olegz opened a pull request:
https://github.com/apache/nifi/pull/1449
NIFI-3223 added support for expression language
- EXCHANGE
- ROUTING_KEY
Thank you for submitting a contribution to Apache NiFi.
In order to streamline the review
Github user olegz commented on the issue:
https://github.com/apache/nifi/pull/1436
@joewitt copyrights are fixed. Please see the latest 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 project does
1 - 100 of 342 matches
Mail list logo