Github user ijokarumawak commented on the issue:
https://github.com/apache/nifi/pull/563
@JPercivall Thanks for your time to review.
I've looked at every existing Processors, Reporting tasks, and Controller
services, but couldn't find other component needing exte
GitHub user ijokarumawak opened a pull request:
https://github.com/apache/nifi/pull/600
NIFI-906: Make NiFi docs responsible
- Switched from absolute layout to responsible using flex-box.
- Added toggle switch to control component listing pane visibility.
- Tested with
Github user ijokarumawak commented on the issue:
https://github.com/apache/nifi/pull/483
@trixpan Thanks for addressing comments promptly! I confirmed that all of
my comments are addressed. However the modification on ExtractEmailHeader.java
didn't work with following
Github user ijokarumawak commented on a diff in the pull request:
https://github.com/apache/nifi/pull/483#discussion_r69245092
--- Diff:
nifi-nar-bundles/nifi-email-bundle/nifi-email-processors/src/test/resources/attachment-only.eml
---
@@ -0,0 +1,156 @@
+Date: Fri, 29 Jul
Github user ijokarumawak commented on a diff in the pull request:
https://github.com/apache/nifi/pull/483#discussion_r69244627
--- Diff:
nifi-nar-bundles/nifi-email-bundle/nifi-email-processors/src/main/java/org/apache/nifi/processors/email/ExtractEmailAttachments.java
Github user ijokarumawak commented on a diff in the pull request:
https://github.com/apache/nifi/pull/483#discussion_r69244471
--- Diff:
nifi-nar-bundles/nifi-email-bundle/nifi-email-processors/src/main/java/org/apache/nifi/processors/email/ExtractEmailHeaders.java
---
@@ -0,0
Github user ijokarumawak commented on a diff in the pull request:
https://github.com/apache/nifi/pull/483#discussion_r69243783
--- Diff:
nifi-nar-bundles/nifi-email-bundle/nifi-email-processors/src/main/java/org/apache/nifi/processors/email/ExtractEmailAttachments.java
Github user ijokarumawak commented on the issue:
https://github.com/apache/nifi/pull/483
@trixpan Thanks for your effort to creating these Processors! I was able to
setup a data flow that can receive emails using Listen SMTP easily and extract
headers and attachments. All unit tests
Github user ijokarumawak commented on a diff in the pull request:
https://github.com/apache/nifi/pull/483#discussion_r68535055
--- Diff:
nifi-nar-bundles/nifi-email-bundle/nifi-email-processors/src/test/resources/attachment-only.eml
---
@@ -0,0 +1,156 @@
+Date: Fri, 29 Jul
Github user ijokarumawak commented on a diff in the pull request:
https://github.com/apache/nifi/pull/483#discussion_r68533372
--- Diff:
nifi-nar-bundles/nifi-email-bundle/nifi-email-processors/src/main/java/org/apache/nifi/processors/email/ListenSMTP.java
---
@@ -0,0 +1,383
Github user ijokarumawak commented on a diff in the pull request:
https://github.com/apache/nifi/pull/483#discussion_r68530403
--- Diff:
nifi-nar-bundles/nifi-email-bundle/nifi-email-processors/src/main/java/org/apache/nifi/processors/email/ExtractEmailHeaders.java
---
@@ -0,0
Github user ijokarumawak commented on a diff in the pull request:
https://github.com/apache/nifi/pull/483#discussion_r68529385
--- Diff:
nifi-nar-bundles/nifi-email-bundle/nifi-email-processors/src/main/java/org/apache/nifi/processors/email/ListenSMTP.java
---
@@ -0,0 +1,383
Github user ijokarumawak commented on a diff in the pull request:
https://github.com/apache/nifi/pull/483#discussion_r68527017
--- Diff:
nifi-nar-bundles/nifi-email-bundle/nifi-email-processors/src/main/java/org/apache/nifi/processors/email/ExtractEmailAttachments.java
Github user ijokarumawak commented on a diff in the pull request:
https://github.com/apache/nifi/pull/483#discussion_r68526891
--- Diff:
nifi-nar-bundles/nifi-email-bundle/nifi-email-processors/src/main/java/org/apache/nifi/processors/email/ExtractEmailHeaders.java
---
@@ -0,0
Github user ijokarumawak commented on a diff in the pull request:
https://github.com/apache/nifi/pull/483#discussion_r68524707
--- Diff:
nifi-nar-bundles/nifi-email-bundle/nifi-email-processors/src/main/java/org/apache/nifi/processors/email/smtp/event/SmtpEventFactory.java
Github user ijokarumawak commented on a diff in the pull request:
https://github.com/apache/nifi/pull/483#discussion_r68524560
--- Diff:
nifi-nar-bundles/nifi-email-bundle/nifi-email-processors/src/main/java/org/apache/nifi/processors/email/smtp/event/SmtpEvent.java
---
@@ -0,0
Github user ijokarumawak commented on a diff in the pull request:
https://github.com/apache/nifi/pull/483#discussion_r68524121
--- Diff:
nifi-nar-bundles/nifi-email-bundle/nifi-email-processors/src/main/java/org/apache/nifi/processors/email/ListenSMTP.java
---
@@ -0,0 +1,383
Github user ijokarumawak commented on a diff in the pull request:
https://github.com/apache/nifi/pull/483#discussion_r68524041
--- Diff:
nifi-nar-bundles/nifi-email-bundle/nifi-email-processors/src/main/java/org/apache/nifi/processors/email/ListenSMTP.java
---
@@ -0,0 +1,383
Github user ijokarumawak commented on a diff in the pull request:
https://github.com/apache/nifi/pull/483#discussion_r68523977
--- Diff:
nifi-nar-bundles/nifi-email-bundle/nifi-email-processors/src/main/java/org/apache/nifi/processors/email/ListenSMTP.java
---
@@ -0,0 +1,383
Github user ijokarumawak commented on a diff in the pull request:
https://github.com/apache/nifi/pull/483#discussion_r68522856
--- Diff:
nifi-nar-bundles/nifi-email-bundle/nifi-email-processors/src/main/java/org/apache/nifi/processors/email/ListenSMTP.java
---
@@ -0,0 +1,383
Github user ijokarumawak commented on a diff in the pull request:
https://github.com/apache/nifi/pull/483#discussion_r68522731
--- Diff:
nifi-nar-bundles/nifi-email-bundle/nifi-email-processors/src/main/java/org/apache/nifi/processors/email/ListenSMTP.java
---
@@ -0,0 +1,383
Github user ijokarumawak commented on a diff in the pull request:
https://github.com/apache/nifi/pull/483#discussion_r68522653
--- Diff:
nifi-nar-bundles/nifi-email-bundle/nifi-email-processors/src/main/java/org/apache/nifi/processors/email/ListenSMTP.java
---
@@ -0,0 +1,383
GitHub user ijokarumawak opened a pull request:
https://github.com/apache/nifi/pull/575
NIFI-619: Make MonitorActivity more cluster friendly
- Added 'Monitoring Scope' property
- Persist the latest activity as Cluster state
- Examine cluster state on each node if
Github user ijokarumawak commented on the issue:
https://github.com/apache/nifi/pull/563
Test with Kerberized Kafka and Zk is done. I followed [confluent Apache
Kafka Security
101](http://www.confluent.io/blog/apache-kafka-security-authorization-authentication-encryption)
to setup
Github user ijokarumawak commented on the issue:
https://github.com/apache/nifi/pull/563
Integration test with Kerberized Kafka and Zookeeper haven't finished yet,
but the implementation is done. Would you please initiate the code review
process?
This PR added External
GitHub user ijokarumawak opened a pull request:
https://github.com/apache/nifi/pull/563
NIFI-2078: External state management.
- Added ExternalStateManager to handle components' state managed
externally
- Added UI codes to display external state
- Added view/
GitHub user ijokarumawak opened a pull request:
https://github.com/apache/nifi/pull/529
NIFI-2028: Fixed Site-to-Site Transit URI
Please review and merge if it looks ok.
I've added unit tests and also did IT test to check https and proxy works
without problem.
Tr
Github user ijokarumawak commented on the issue:
https://github.com/apache/nifi/pull/497
@markap14 Thanks for taking your time to review and test, I'm so glad to
hear that worked!
The CI test haven't been finished, maybe that's why this isn't closed yet.
I
Github user ijokarumawak closed the pull request at:
https://github.com/apache/nifi/pull/509
---
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 ijokarumawak commented on the issue:
https://github.com/apache/nifi/pull/509
@bbende Yes, the same change is in PR #497 . This is only for 0.x. Thanks!
---
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 ijokarumawak commented on the issue:
https://github.com/apache/nifi/pull/497
@markap14 I'm not 100% sure if the change will solve the problem since I
couldn't reproduce the issue myself.
However, I've added additional buffer draining code after rece
Github user ijokarumawak closed the pull request at:
https://github.com/apache/nifi/pull/503
---
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 ijokarumawak commented on the issue:
https://github.com/apache/nifi/pull/503
@bbende Thanks for reviewing and 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 ijokarumawak opened a pull request:
https://github.com/apache/nifi/pull/509
NIFI-1982: Use Compressed check box value.
- The Compressed check box UI input was not used
- This commit enables Site-to-Site compression configuration from UI
You can merge this pull
Github user ijokarumawak commented on the issue:
https://github.com/apache/nifi/pull/497
@markap14 Fixed the sending data issue. The reason was I didn't call
ByteBuffer clear() and didn't closed PipedOutputStream properly. Sorry for the
inconvenience. Please tr
Github user ijokarumawak commented on the issue:
https://github.com/apache/nifi/pull/497
@markap14 I think I was able to reproduce the issue, it seems it happens
when theres more data in the queue than a RemoteProcessGroup can send at once.
I'll fix the issue.
---
If your pr
Github user ijokarumawak commented on the issue:
https://github.com/apache/nifi/pull/497
@markap14 I tested sending data using HTTP Site-to-Site with fresh build of
this PR. It successfully sent data. The exception happens when a client sends
data packet, but doesn't send addit
Github user ijokarumawak commented on the issue:
https://github.com/apache/nifi/pull/497
@markap14 Thanks for taking time to try, and sorry to hear that it didn't
work. I'll check it. Thanks!
---
If your project is set up for it, you can reply to this email and have
Github user ijokarumawak commented on a diff in the pull request:
https://github.com/apache/nifi/pull/497#discussion_r66080066
--- Diff:
nifi-nar-bundles/nifi-framework-bundle/nifi-framework/nifi-web/nifi-web-api/src/main/java/org/apache/nifi/web/api/SiteToSiteResource.java
Github user ijokarumawak commented on a diff in the pull request:
https://github.com/apache/nifi/pull/497#discussion_r66078443
--- Diff:
nifi-nar-bundles/nifi-framework-bundle/nifi-framework/nifi-site-to-site/src/main/java/org/apache/nifi/remote/StandardRootGroupPort.java
Github user ijokarumawak commented on the issue:
https://github.com/apache/nifi/pull/497
@mcgilman @markap14 I haven't touched RemoteProcessGroupAuditor.java with
this PR since I couldn't figure out the usage of this class. Processors shows
its audit in NiFi UI, but where
Github user ijokarumawak commented on a diff in the pull request:
https://github.com/apache/nifi/pull/497#discussion_r66071968
--- Diff:
nifi-nar-bundles/nifi-framework-bundle/nifi-framework/nifi-framework-core/src/main/java/org/apache/nifi/remote/StandardRemoteProcessGroup.java
GitHub user ijokarumawak opened a pull request:
https://github.com/apache/nifi/pull/503
NIFI-1978: Restore RPG yield duration.
The persisted configuration value was not restored properly, and it will be
reset to default value 10 sec after restarting NiFi.
I found this
Github user ijokarumawak commented on a diff in the pull request:
https://github.com/apache/nifi/pull/497#discussion_r66016174
--- Diff:
nifi-nar-bundles/nifi-framework-bundle/nifi-framework/nifi-resources/src/main/resources/conf/nifi.properties
---
@@ -111,9 +111,11
Github user ijokarumawak commented on the issue:
https://github.com/apache/nifi/pull/423
Closing this PR since I've submitted updated version #497
---
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 doe
Github user ijokarumawak closed the pull request at:
https://github.com/apache/nifi/pull/423
---
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 ijokarumawak opened a pull request:
https://github.com/apache/nifi/pull/497
NIFI-1857: HTTPS Site-to-Site
Hi,
I've squashed #423 with the latest master, to add proxy auth and better
connection management. However, I haven't done tests with the latest
Github user ijokarumawak closed the pull request at:
https://github.com/apache/nifi/pull/432
---
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 ijokarumawak commented on the pull request:
https://github.com/apache/nifi/pull/432#issuecomment-218358689
Thanks @mattyb149 !
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have
Github user ijokarumawak commented on the pull request:
https://github.com/apache/nifi/pull/429#issuecomment-218321492
@alopresto I heard that single commit would be easier to apply to master
and 0.x branch. I squashed the commits and sent a PR #432 . I'll update the
same branc
Github user ijokarumawak commented on the pull request:
https://github.com/apache/nifi/pull/429#issuecomment-218323187
I didn't know that I can push --force a squashed commit, thanks!
---
If your project is set up for it, you can reply to this email and have your
reply appe
GitHub user ijokarumawak opened a pull request:
https://github.com/apache/nifi/pull/432
NIFI-1865 Close StreamThrottler when processor stops.
- Also, replaced copyrighted sample texts for tests
You can merge this pull request into a Git repository by running:
$ git pull https
Github user ijokarumawak closed the pull request at:
https://github.com/apache/nifi/pull/429
---
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 ijokarumawak commented on the pull request:
https://github.com/apache/nifi/pull/429#issuecomment-218316718
Thanks for reviewing @mattyb149 , I will fix the texts and resend a PR
shortly.
---
If your project is set up for it, you can reply to this email and have your
GitHub user ijokarumawak opened a pull request:
https://github.com/apache/nifi/pull/429
NIFI-1865 Close StreamThrottler when processor stops.
Add close when processor stops.
You can merge this pull request into a Git repository by running:
$ git pull https://github.com
GitHub user ijokarumawak opened a pull request:
https://github.com/apache/nifi/pull/423
[WIP] NIFI-1857 Added HTTP(S) support for Site-to-Site.
Hello, it's not fully done yet, but there's so many lines of codes already,
so I'd like to share it.
Any comment would
GitHub user ijokarumawak opened a pull request:
https://github.com/apache/nifi/pull/422
NIFI-1860 Added ContainerRequestFilter to redirect
- redirect /controller to /site-to-site
You can merge this pull request into a Git repository by running:
$ git pull https://github.com
GitHub user ijokarumawak opened a pull request:
https://github.com/apache/nifi/pull/415
NIFI-1773 Removed unused import statement.
This commit removes the unused import statement which is added at [this
commit](https://github.com/apache/nifi/commit
GitHub user ijokarumawak opened a pull request:
https://github.com/apache/nifi/pull/414
NIFI-1843 Default log level for LogAttribute.
- added a logger entry for LogAttribute to specify INFO as log level
- replaced some tab characters to whitespace (existing checkstyle error
GitHub user ijokarumawak opened a pull request:
https://github.com/apache/nifi/pull/336
NIFI-1180 Adding more IT tests.
I've checkout the latest master, built and played with it, it works without
any issue!
The Integration Tests worked fine with my AWS environment.
By
GitHub user ijokarumawak opened a pull request:
https://github.com/apache/nifi/pull/333
nifi-1736 Move kafka.StreamScanner to nifi-utils.
You can merge this pull request into a Git repository by running:
$ git pull https://github.com/ijokarumawak/nifi nifi-1736
Alternatively
GitHub user ijokarumawak opened a pull request:
https://github.com/apache/nifi/pull/96
NIFI-992: Adding nifi-couchbase-bundle
Adding Couchbase Server integration to NiFi project.
- new CouchbaseClusterControllerService: provides connection management to
Couchbase Server
62 matches
Mail list logo
