[GitHub] nifi pull request #1280: NIFI-3109 Initial refactoring of TimerDrivenSchedul...

2017-07-15 Thread olegz
Github user olegz closed the pull request at: https://github.com/apache/nifi/pull/1280 --- 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] nifi pull request #1278: NIFI-2886 Fixed the lifecycle delay

2017-07-15 Thread olegz
Github user olegz closed the pull request at: https://github.com/apache/nifi/pull/1278 --- 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] nifi pull request #1652: NIFI-1280: Renamed FilterCSVColumns to QueryFlowFil...

2017-04-09 Thread olegz
Github user olegz commented on a diff in the pull request: https://github.com/apache/nifi/pull/1652#discussion_r110541830 --- Diff: nifi-nar-bundles/nifi-standard-services/nifi-record-serialization-services-bundle/nifi-record-serialization-services/src/main/java/org/apache/nifi/grok

[GitHub] nifi pull request #1652: NIFI-1280: Renamed FilterCSVColumns to QueryFlowFil...

2017-04-09 Thread olegz
Github user olegz commented on a diff in the pull request: https://github.com/apache/nifi/pull/1652#discussion_r110541217 --- Diff: nifi-nar-bundles/nifi-standard-bundle/nifi-standard-processors/src/main/java/org/apache/nifi/processors/standard/QueryFlowFile.java --- @@ -0,0

[GitHub] nifi pull request #1652: NIFI-1280: Renamed FilterCSVColumns to QueryFlowFil...

2017-04-09 Thread olegz
Github user olegz commented on a diff in the pull request: https://github.com/apache/nifi/pull/1652#discussion_r110541076 --- Diff: nifi-nar-bundles/nifi-framework-bundle/nifi-framework/nifi-framework-core/src/main/java/org/apache/nifi/controller/repository/StandardProcessSession.java

[GitHub] nifi pull request #1652: NIFI-1280: Renamed FilterCSVColumns to QueryFlowFil...

2017-04-09 Thread olegz
Github user olegz commented on a diff in the pull request: https://github.com/apache/nifi/pull/1652#discussion_r110541128 --- Diff: nifi-nar-bundles/nifi-registry-bundle/nifi-registry-service/src/main/java/org/apache/nifi/schemaregistry/services/AvroSchemaRegistry.java

[GitHub] nifi pull request #1652: NIFI-1280: Renamed FilterCSVColumns to QueryFlowFil...

2017-04-09 Thread olegz
Github user olegz commented on a diff in the pull request: https://github.com/apache/nifi/pull/1652#discussion_r110544739 --- Diff: nifi-nar-bundles/nifi-standard-services/nifi-record-serialization-services-bundle/nifi-record-serialization-services/src/main/java/org/apache/nifi/grok

[GitHub] nifi pull request #1652: NIFI-1280: Renamed FilterCSVColumns to QueryFlowFil...

2017-04-09 Thread olegz
Github user olegz commented on a diff in the pull request: https://github.com/apache/nifi/pull/1652#discussion_r110541788 --- Diff: nifi-nar-bundles/nifi-standard-services/nifi-record-serialization-services-bundle/nifi-record-serialization-services/src/main/java/org/apache/nifi/grok

[GitHub] nifi pull request #1652: NIFI-1280: Renamed FilterCSVColumns to QueryFlowFil...

2017-04-09 Thread olegz
Github user olegz commented on a diff in the pull request: https://github.com/apache/nifi/pull/1652#discussion_r110541764 --- Diff: nifi-nar-bundles/nifi-standard-services/nifi-record-serialization-services-bundle/nifi-record-serialization-services/src/main/java/org/apache/nifi/json

[GitHub] nifi pull request #1652: NIFI-1280: Renamed FilterCSVColumns to QueryFlowFil...

2017-04-09 Thread olegz
Github user olegz commented on a diff in the pull request: https://github.com/apache/nifi/pull/1652#discussion_r110544779 --- Diff: nifi-nar-bundles/nifi-standard-services/nifi-record-serialization-services-bundle/nifi-record-serialization-services/src/main/java/org/apache/nifi/csv

[GitHub] nifi pull request #1652: NIFI-1280: Renamed FilterCSVColumns to QueryFlowFil...

2017-04-09 Thread olegz
Github user olegz commented on a diff in the pull request: https://github.com/apache/nifi/pull/1652#discussion_r110544874 --- Diff: nifi-nar-bundles/nifi-standard-services/nifi-record-serialization-services-bundle/nifi-record-serialization-services/.gitignore --- @@ -0,0 +1

[GitHub] nifi issue #1652: NIFI-1280: Renamed FilterCSVColumns to QueryFlowFile; intr...

2017-04-06 Thread olegz
Github user olegz commented on the issue: https://github.com/apache/nifi/pull/1652 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, or if

[GitHub] nifi pull request #1651: NIFI-3509 Fixed ID duplication

2017-04-06 Thread olegz
GitHub user olegz opened a pull request: https://github.com/apache/nifi/pull/1651 NIFI-3509 Fixed ID duplication Fixed the possibility of duplicated MSB part of the component during template creation Thank you for submitting a contribution to Apache NiFi. In order

[GitHub] nifi pull request #1629: NIFI-3441 added @Ignore to integration tests due to...

2017-03-28 Thread olegz
GitHub user olegz opened a pull request: https://github.com/apache/nifi/pull/1629 NIFI-3441 added @Ignore to integration tests due to intermittent fail… …ures Thank you for submitting a contribution to Apache NiFi. In order to streamline the review of the

[GitHub] nifi issue #1548: NIFI-3543 Fixed a bug that nifi-jms-cf-service cannot have...

2017-03-03 Thread olegz
Github user olegz commented on the issue: https://github.com/apache/nifi/pull/1548 Thanks you @ShellyLC , will review and merge. --- 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] nifi issue #1555: NIFI-3548: Fixed bug that caused failed requests to not ge...

2017-03-03 Thread olegz
Github user olegz commented on the issue: https://github.com/apache/nifi/pull/1555 Wow!, I was just about to hit merge myself ;) Race condition ;) --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub as well. If your project does not

[GitHub] nifi pull request #1553: NIFI-1449 - Migrate PutEmail tests from Mock class ...

2017-03-02 Thread olegz
Github user olegz commented on a diff in the pull request: https://github.com/apache/nifi/pull/1553#discussion_r104022318 --- Diff: nifi-nar-bundles/nifi-standard-bundle/nifi-standard-processors/src/test/java/org/apache/nifi/processors/standard/TestPutEmail.java --- @@ -230,27

[GitHub] nifi issue #1548: NIFI-3543 Fixed a bug that nifi-jms-cf-service cannot have...

2017-03-02 Thread olegz
Github user olegz commented on the issue: https://github.com/apache/nifi/pull/1548 @ShellyLC also, please amend your commit message preceding it with the JIRA number so it could be automatically linked in JIRA (i.e., "NIFI-3543 Fixed a bug that nifi. . ."). Also, plea

[GitHub] nifi pull request #1548: NIFI-3543 Fixed a bug that nifi-jms-cf-service cann...

2017-03-02 Thread olegz
Github user olegz commented on a diff in the pull request: https://github.com/apache/nifi/pull/1548#discussion_r103945068 --- Diff: nifi-nar-bundles/nifi-jms-bundle/nifi-jms-cf-service/src/main/java/org/apache/nifi/jms/cf/JMSConnectionFactoryProvider.java --- @@ -178,7 +178,7

[GitHub] nifi pull request #1553: NIFI-1449 - Migrate PutEmail tests from Mock class ...

2017-03-02 Thread olegz
Github user olegz commented on a diff in the pull request: https://github.com/apache/nifi/pull/1553#discussion_r103939988 --- Diff: nifi-nar-bundles/nifi-standard-bundle/nifi-standard-processors/src/test/java/org/apache/nifi/processors/standard/TestPutEmail.java --- @@ -102,13

[GitHub] nifi pull request #1553: NIFI-1449 - Migrate PutEmail tests from Mock class ...

2017-03-02 Thread olegz
Github user olegz commented on a diff in the pull request: https://github.com/apache/nifi/pull/1553#discussion_r103938291 --- Diff: nifi-nar-bundles/nifi-standard-bundle/nifi-standard-processors/src/test/java/org/apache/nifi/processors/standard/TestPutEmail.java --- @@ -230,27

[GitHub] nifi pull request #1530: NIFI-3490 added SAN option for TLS toolkit in stand...

2017-03-01 Thread olegz
Github user olegz commented on a diff in the pull request: https://github.com/apache/nifi/pull/1530#discussion_r103752061 --- Diff: nifi-toolkit/nifi-toolkit-tls/src/main/java/org/apache/nifi/toolkit/tls/util/TlsHelper.java --- @@ -215,4 +210,17 @@ public static

[GitHub] nifi pull request #1539: NIFI-3520 Updates classloading for components annot...

2017-03-01 Thread olegz
Github user olegz commented on a diff in the pull request: https://github.com/apache/nifi/pull/1539#discussion_r103750596 --- Diff: nifi-nar-bundles/nifi-framework-bundle/nifi-framework/nifi-nar-utils/src/main/java/org/apache/nifi/nar/ExtensionManager.java --- @@ -199,8 +199,20

[GitHub] nifi pull request #1542: NIFI-3523: Ensure that if we are unable to read Pro...

2017-03-01 Thread olegz
Github user olegz commented on a diff in the pull request: https://github.com/apache/nifi/pull/1542#discussion_r103749486 --- Diff: nifi-nar-bundles/nifi-provenance-repository-bundle/nifi-persistent-provenance-repository/src/main/java/org/apache/nifi/provenance/serialization

[GitHub] nifi pull request #1214: NIFI-2876 refactored demarcators into a common abst...

2017-02-23 Thread olegz
Github user olegz commented on a diff in the pull request: https://github.com/apache/nifi/pull/1214#discussion_r102830048 --- Diff: nifi-commons/nifi-utils/src/main/java/org/apache/nifi/stream/io/util/AbstractDemarcator.java --- @@ -0,0 +1,138 @@ +/* + * Licensed to the

[GitHub] nifi pull request #1214: NIFI-2876 refactored demarcators into a common abst...

2017-02-23 Thread olegz
Github user olegz commented on a diff in the pull request: https://github.com/apache/nifi/pull/1214#discussion_r102775809 --- Diff: nifi-commons/nifi-utils/src/main/java/org/apache/nifi/stream/io/util/AbstractDemarcator.java --- @@ -98,22 +136,26 @@ void fill() throws IOException

[GitHub] nifi issue #1532: NIFI-3522: When creating a clone of a FlowFile, ensure tha...

2017-02-22 Thread olegz
Github user olegz commented on the issue: https://github.com/apache/nifi/pull/1532 +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, or if the

[GitHub] nifi issue #1436: NIFI-3354 Added support for simple AVRO/CSV/JSON transform...

2017-02-17 Thread olegz
Github user olegz commented on the issue: https://github.com/apache/nifi/pull/1436 Merged --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub as well. If your project does not have this feature enabled and wishes so, or if the

[GitHub] nifi pull request #1436: NIFI-3354 Added support for simple AVRO/CSV/JSON tr...

2017-02-17 Thread olegz
Github user olegz closed the pull request at: https://github.com/apache/nifi/pull/1436 --- 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] nifi issue #1436: NIFI-3354 Added support for simple AVRO/CSV/JSON transform...

2017-02-17 Thread olegz
Github user olegz commented on the issue: https://github.com/apache/nifi/pull/1436 @joewitt added new JIRAs to handle multi-line CSV input and multi-char delimiters suggested by @patricker https://issues.apache.org/jira/browse/NIFI-3499 https://issues.apache.org/jira/browse

[GitHub] nifi issue #1394: NIFI-3255 removed dependency on session.merge from SplitTe...

2017-02-17 Thread olegz
Github user olegz commented on the issue: https://github.com/apache/nifi/pull/1394 @ijokarumawak addressing it now and will merge. So basically I will be creating splits via ```FlowFile mergedFlowFile = session.create(sourceFlowFile);``` passing the source file from ```onTrigger

[GitHub] nifi pull request #1518: NIFI-3495 fixed the index issue with TextLineDemarc...

2017-02-16 Thread olegz
GitHub user olegz opened a pull request: https://github.com/apache/nifi/pull/1518 NIFI-3495 fixed the index issue with TextLineDemarcator Thank you for submitting a contribution to Apache NiFi. In order to streamline the review of the contribution we ask you to ensure

[GitHub] nifi issue #1436: NIFI-3354 Added support for simple AVRO/CSV/JSON transform...

2017-02-16 Thread olegz
Github user olegz commented on the issue: https://github.com/apache/nifi/pull/1436 @joewitt with regard to content type and name change comment, 100% agree. Will make a change. --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub as

[GitHub] nifi issue #1436: NIFI-3354 Added support for simple AVRO/CSV/JSON transform...

2017-02-16 Thread olegz
Github user olegz commented on the issue: https://github.com/apache/nifi/pull/1436 @joewitt If you were trying to use the linked CSV with the provided template (the one attached in JIRA) it would not work since the template relies on different schema. I need to come up with the

[GitHub] nifi issue #1179: NIFI-969 Added support for standard JSON

2017-02-14 Thread olegz
Github user olegz commented on the issue: https://github.com/apache/nifi/pull/1179 The issue described in the underlying JIRA and this PR is essentially addressed or to be addressed/maintained by a new Schema Registry based transformer effort - https://github.com/apache/nifi/pull

[GitHub] nifi pull request #1179: NIFI-969 Added support for standard JSON

2017-02-14 Thread olegz
Github user olegz closed the pull request at: https://github.com/apache/nifi/pull/1179 --- 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] nifi pull request #1493: NIFI-3356: Initial implementation of writeahead pro...

2017-02-14 Thread olegz
Github user olegz commented on a diff in the pull request: https://github.com/apache/nifi/pull/1493#discussion_r101130301 --- Diff: nifi-framework-api/src/main/java/org/apache/nifi/provenance/IdentifierLookup.java --- @@ -0,0 +1,88 @@ +/* + * Licensed to the Apache

[GitHub] nifi pull request #584: closing InputStream after execution completes

2017-02-14 Thread olegz
Github user olegz commented on a diff in the pull request: https://github.com/apache/nifi/pull/584#discussion_r101118510 --- Diff: nifi-external/nifi-spark-receiver/src/main/java/org/apache/nifi/spark/NiFiReceiver.java --- @@ -162,17 +162,19 @@ public void run

[GitHub] nifi issue #584: closing InputStream after execution completes

2017-02-14 Thread olegz
Github user olegz commented on the issue: https://github.com/apache/nifi/pull/584 @vcharmcaster Since it appears this is your initial participation with Apache NiFi I'd like to welcome you and thank you for your contribution and participation. Aside from making thing

[GitHub] nifi issue #1436: NIFI-3354 Added support for simple AVRO/CSV/JSON transform...

2017-02-14 Thread olegz
Github user olegz commented on the issue: https://github.com/apache/nifi/pull/1436 Agreed about _ ViaSchemaRegistry_ --- 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

[GitHub] nifi pull request #1493: NIFI-3356: Initial implementation of writeahead pro...

2017-02-14 Thread olegz
Github user olegz commented on a diff in the pull request: https://github.com/apache/nifi/pull/1493#discussion_r101099586 --- Diff: nifi-nar-bundles/nifi-framework-bundle/nifi-framework/nifi-framework-core/src/main/java/org/apache/nifi/controller/repository/claim

[GitHub] nifi pull request #1493: NIFI-3356: Initial implementation of writeahead pro...

2017-02-14 Thread olegz
Github user olegz commented on a diff in the pull request: https://github.com/apache/nifi/pull/1493#discussion_r101097614 --- Diff: nifi-nar-bundles/nifi-framework-bundle/nifi-framework/nifi-framework-core/src/main/java/org/apache/nifi/controller/repository/claim

[GitHub] nifi pull request #1493: NIFI-3356: Initial implementation of writeahead pro...

2017-02-14 Thread olegz
Github user olegz commented on a diff in the pull request: https://github.com/apache/nifi/pull/1493#discussion_r101094014 --- Diff: nifi-nar-bundles/nifi-framework-bundle/nifi-framework/nifi-framework-core/src/main/java/org/apache/nifi/controller/repository/claim

[GitHub] nifi pull request #1493: NIFI-3356: Initial implementation of writeahead pro...

2017-02-14 Thread olegz
Github user olegz commented on a diff in the pull request: https://github.com/apache/nifi/pull/1493#discussion_r101088999 --- Diff: nifi-nar-bundles/nifi-framework-bundle/nifi-framework/nifi-framework-core/src/test/java/org/apache/nifi/controller/repository

[GitHub] nifi pull request #1493: NIFI-3356: Initial implementation of writeahead pro...

2017-02-14 Thread olegz
Github user olegz commented on a diff in the pull request: https://github.com/apache/nifi/pull/1493#discussion_r101084757 --- Diff: nifi-nar-bundles/nifi-framework-bundle/nifi-framework/nifi-framework-core/src/main/java/org/apache/nifi/controller/repository/claim

[GitHub] nifi pull request #1493: NIFI-3356: Initial implementation of writeahead pro...

2017-02-14 Thread olegz
Github user olegz commented on a diff in the pull request: https://github.com/apache/nifi/pull/1493#discussion_r101084265 --- Diff: nifi-nar-bundles/nifi-framework-bundle/nifi-framework/nifi-framework-core/src/main/java/org/apache/nifi/controller/repository/claim

[GitHub] nifi pull request #1493: NIFI-3356: Initial implementation of writeahead pro...

2017-02-14 Thread olegz
Github user olegz commented on a diff in the pull request: https://github.com/apache/nifi/pull/1493#discussion_r101080280 --- Diff: nifi-docs/src/main/asciidoc/administration-guide.adoc --- @@ -2074,7 +2074,25 @@ The Provenance Repository contains the information related to Data

[GitHub] nifi pull request #1493: NIFI-3356: Initial implementation of writeahead pro...

2017-02-14 Thread olegz
Github user olegz commented on a diff in the pull request: https://github.com/apache/nifi/pull/1493#discussion_r101079356 --- Diff: nifi-docs/src/main/asciidoc/administration-guide.adoc --- @@ -2074,7 +2074,25 @@ The Provenance Repository contains the information related to Data

[GitHub] nifi issue #1493: NIFI-3356: Initial implementation of writeahead provenance...

2017-02-14 Thread olegz
Github user olegz commented on the issue: https://github.com/apache/nifi/pull/1493 @markap14 there seem to be some unintended files that made into the commit (i.e., /bin/.., .gitignore etc) --- If your project is set up for it, you can reply to this email and have your reply appear

[GitHub] nifi issue #1394: NIFI-3255 removed dependency on session.merge from SplitTe...

2017-02-14 Thread olegz
Github user olegz commented on the issue: https://github.com/apache/nifi/pull/1394 @ijokarumawak addressed some of the PR comments and commented on the others. Changed the name of the _merge(..)_ operation to _concatenateContents(..)_. Pushed. . . --- If your project is set up

[GitHub] nifi pull request #1394: NIFI-3255 removed dependency on session.merge from ...

2017-02-14 Thread olegz
Github user olegz commented on a diff in the pull request: https://github.com/apache/nifi/pull/1394#discussion_r101050962 --- Diff: nifi-nar-bundles/nifi-standard-bundle/nifi-standard-processors/src/main/java/org/apache/nifi/processors/standard/SplitText.java --- @@ -326,6

[GitHub] nifi pull request #1394: NIFI-3255 removed dependency on session.merge from ...

2017-02-14 Thread olegz
Github user olegz commented on a diff in the pull request: https://github.com/apache/nifi/pull/1394#discussion_r101050107 --- Diff: nifi-nar-bundles/nifi-standard-bundle/nifi-standard-processors/src/main/java/org/apache/nifi/processors/standard/SplitText.java --- @@ -326,6

[GitHub] nifi issue #1394: NIFI-3255 removed dependency on session.merge from SplitTe...

2017-02-14 Thread olegz
Github user olegz commented on the issue: https://github.com/apache/nifi/pull/1394 @ijokarumawak sorry, didn't get a chance to get to it yet. Will definitely address. --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub as wel

[GitHub] nifi pull request #1436: NIFI-3354 Added support for simple AVRO/CSV/JSON tr...

2017-02-07 Thread olegz
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] nifi issue #1449: NIFI-3223 added support for expression language

2017-01-31 Thread olegz
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 attributes = new HashMap<>(); attributes.put("some.attribute.

[GitHub] nifi issue #1452: NIFI-3179 Added support for default UTF-8 char encoding

2017-01-30 Thread olegz
Github user olegz commented on the issue: https://github.com/apache/nifi/pull/1452 @pvillard31 should be all good now, sorry about that --- 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] nifi pull request #1452: NIFI-3179 Added support for default UTF-8 char enco...

2017-01-30 Thread olegz
Github user olegz commented on a diff in the pull request: https://github.com/apache/nifi/pull/1452#discussion_r98547614 --- Diff: nifi-nar-bundles/nifi-standard-bundle/nifi-standard-processors/src/main/java/org/apache/nifi/processors/standard/MergeContent.java --- @@ -780,7

[GitHub] nifi pull request #1401: NIFI-3290 Reporting task to send bulletins with S2S

2017-01-29 Thread olegz
Github user olegz commented on a diff in the pull request: https://github.com/apache/nifi/pull/1401#discussion_r98357001 --- Diff: nifi-nar-bundles/nifi-site-to-site-reporting-bundle/nifi-site-to-site-reporting-task/src/main/java/org/apache/nifi/reporting

[GitHub] nifi pull request #1401: NIFI-3290 Reporting task to send bulletins with S2S

2017-01-29 Thread olegz
Github user olegz commented on a diff in the pull request: https://github.com/apache/nifi/pull/1401#discussion_r98356790 --- Diff: nifi-nar-bundles/nifi-site-to-site-reporting-bundle/nifi-site-to-site-reporting-task/src/main/java/org/apache/nifi/reporting

[GitHub] nifi pull request #1401: NIFI-3290 Reporting task to send bulletins with S2S

2017-01-29 Thread olegz
Github user olegz commented on a diff in the pull request: https://github.com/apache/nifi/pull/1401#discussion_r98356724 --- Diff: nifi-nar-bundles/nifi-site-to-site-reporting-bundle/nifi-site-to-site-reporting-task/src/main/java/org/apache/nifi/reporting

[GitHub] nifi pull request #1401: NIFI-3290 Reporting task to send bulletins with S2S

2017-01-29 Thread olegz
Github user olegz commented on a diff in the pull request: https://github.com/apache/nifi/pull/1401#discussion_r98356661 --- Diff: nifi-nar-bundles/nifi-site-to-site-reporting-bundle/nifi-site-to-site-reporting-task/src/main/java/org/apache/nifi/reporting

[GitHub] nifi issue #1401: NIFI-3290 Reporting task to send bulletins with S2S

2017-01-29 Thread olegz
Github user olegz commented on the issue: https://github.com/apache/nifi/pull/1401 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, or if

[GitHub] nifi pull request #1400: NIFI-957 Added the possibility to use DefaultSchedu...

2017-01-29 Thread olegz
Github user olegz commented on a diff in the pull request: https://github.com/apache/nifi/pull/1400#discussion_r98356568 --- Diff: nifi-nar-bundles/nifi-framework-bundle/nifi-framework/nifi-framework-core/src/main/java/org/apache/nifi/controller/reporting

[GitHub] nifi pull request #1400: NIFI-957 Added the possibility to use DefaultSchedu...

2017-01-29 Thread olegz
Github user olegz commented on a diff in the pull request: https://github.com/apache/nifi/pull/1400#discussion_r98356528 --- Diff: nifi-nar-bundles/nifi-framework-bundle/nifi-framework/nifi-framework-core/src/main/java/org/apache/nifi/controller/reporting

[GitHub] nifi issue #1400: NIFI-957 Added the possibility to use DefaultSchedule anno...

2017-01-29 Thread olegz
Github user olegz commented on the issue: https://github.com/apache/nifi/pull/1400 Reviewing. . . @pvillard31 could you please address the merge conflict, looks trivial --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub as well

[GitHub] nifi issue #1419: NIFI-3314 Adjusting Dockerfile for Docker Hub to use defau...

2017-01-29 Thread olegz
Github user olegz commented on the issue: https://github.com/apache/nifi/pull/1419 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, or if

[GitHub] nifi issue #1419: NIFI-3314 Adjusting Dockerfile for Docker Hub to use defau...

2017-01-29 Thread olegz
Github user olegz commented on the issue: https://github.com/apache/nifi/pull/1419 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, or if

[GitHub] nifi pull request #1453: NIFI-3161 added ASM exclusion

2017-01-29 Thread olegz
GitHub user olegz opened a pull request: https://github.com/apache/nifi/pull/1453 NIFI-3161 added ASM exclusion Thank you for submitting a contribution to Apache NiFi. In order to streamline the review of the contribution we ask you to ensure the following steps have

[GitHub] nifi pull request #1452: NIFI-3179 Added support for default UTF-8 char enco...

2017-01-29 Thread olegz
GitHub user olegz opened a pull request: https://github.com/apache/nifi/pull/1452 NIFI-3179 Added support for default UTF-8 char encoding removed deprected usage of BAOS and BAIS Thank you for submitting a contribution to Apache NiFi. In order to streamline the

[GitHub] nifi pull request #1451: NIFI-3180 Fixed NPE in TemplateUtils

2017-01-29 Thread olegz
GitHub user olegz opened a pull request: https://github.com/apache/nifi/pull/1451 NIFI-3180 Fixed NPE in TemplateUtils added null check for ProcessorDTO.getRelationship() removed deprecated usage of ByteArrayInputStream Thank you for submitting a contribution to Apache

[GitHub] nifi pull request #1449: NIFI-3223 added support for expression language

2017-01-27 Thread olegz
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 of the

[GitHub] nifi issue #1436: NIFI-3354 Added support for simple AVRO/CSV/JSON transform...

2017-01-27 Thread olegz
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 not

[GitHub] nifi issue #1426: NIFI-3363: PutKafka NPE with User-Defined partition (0.x)

2017-01-27 Thread olegz
Github user olegz commented on the issue: https://github.com/apache/nifi/pull/1426 @ijokarumawak please close this PR as it was merged 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] nifi issue #1425: NIFI-3363: PutKafka NPE with User-Defined partition

2017-01-27 Thread olegz
Github user olegz commented on the issue: https://github.com/apache/nifi/pull/1425 Tested with Kafka 0.8 broker, looks good, will merge once build completes --- 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] nifi issue #1425: NIFI-3363: PutKafka NPE with User-Defined partition

2017-01-26 Thread olegz
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, or if

[GitHub] nifi issue #1436: NIFI-3354 Added support for simple AVRO/CSV/JSON transform...

2017-01-24 Thread olegz
Github user olegz commented on the issue: https://github.com/apache/nifi/pull/1436 @joewitt Thanks Joe. That was indeed copy/paste issue from unrelated internal effort. Will fix. --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub

[GitHub] nifi pull request #1434: NIFI-2615 Adding GetTCP processor

2017-01-23 Thread olegz
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] nifi pull request #1434: NIFI-2615 Adding GetTCP processor

2017-01-23 Thread olegz
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] nifi pull request #1434: NIFI-2615 Adding GetTCP processor

2017-01-23 Thread olegz
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] nifi pull request #1436: NIFI-3354 Added support for simple AVRO/CSV/JSON tr...

2017-01-22 Thread olegz
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 for

[GitHub] nifi pull request #1435: NIFI-3345 Added support for simple AVRO/CSV/JSON tr...

2017-01-22 Thread olegz
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 is

[GitHub] nifi issue #1435: NIFI-3345 Added support for simple AVRO/CSV/JSON transform...

2017-01-22 Thread olegz
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 this

[GitHub] nifi pull request #1435: NIFI-3345 Added support for simple AVRO/CSV/JSON tr...

2017-01-22 Thread olegz
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 for

[GitHub] nifi issue #1433: NIFI-2615 Adding support for GetTCP processor

2017-01-19 Thread olegz
Github user olegz commented on the issue: https://github.com/apache/nifi/pull/1433 @mosermw +1 to the name change! --- 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

[GitHub] nifi pull request #1118: NIFI-2716 initial implementation of JMS Broker proc...

2017-01-08 Thread olegz
Github user olegz closed the pull request at: https://github.com/apache/nifi/pull/1118 --- 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] nifi issue #1118: NIFI-2716 initial implementation of JMS Broker processor

2017-01-08 Thread olegz
Github user olegz commented on the issue: https://github.com/apache/nifi/pull/1118 There seems to be no traction at the moment, closing for 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

[GitHub] nifi pull request #1394: NIFI-3255 removed dependency on session.merge from ...

2017-01-04 Thread olegz
GitHub user olegz opened a pull request: https://github.com/apache/nifi/pull/1394 NIFI-3255 removed dependency on session.merge from SplitText Thank you for submitting a contribution to Apache NiFi. In order to streamline the review of the contribution we ask you to

[GitHub] nifi issue #1393: NIFI-3225 removed dependency on session.merge from SplitTe...

2017-01-04 Thread olegz
Github user olegz commented on the issue: https://github.com/apache/nifi/pull/1393 Closing due to typo in JIRA # --- 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

[GitHub] nifi pull request #1393: NIFI-3225 removed dependency on session.merge from ...

2017-01-04 Thread olegz
Github user olegz closed the pull request at: https://github.com/apache/nifi/pull/1393 --- 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] nifi pull request #1393: NIFI-3225 removed dependency on session.merge from ...

2017-01-04 Thread olegz
GitHub user olegz opened a pull request: https://github.com/apache/nifi/pull/1393 NIFI-3225 removed dependency on session.merge from SplitText Thank you for submitting a contribution to Apache NiFi. In order to streamline the review of the contribution we ask you to

[GitHub] nifi pull request #1389: NIFI-3278 Fixed ArrayIndexOutOfBoundsException in T...

2017-01-04 Thread olegz
Github user olegz commented on a diff in the pull request: https://github.com/apache/nifi/pull/1389#discussion_r94649300 --- Diff: nifi-commons/nifi-utils/src/main/java/org/apache/nifi/stream/io/util/TextLineDemarcator.java --- @@ -159,8 +159,11 @@ private int isEol(byte

[GitHub] nifi pull request #1389: NIFI-3278 Fixed ArrayIndexOutOfBoundsException in T...

2017-01-04 Thread olegz
GitHub user olegz opened a pull request: https://github.com/apache/nifi/pull/1389 NIFI-3278 Fixed ArrayIndexOutOfBoundsException in TextLineDemarcator Thank you for submitting a contribution to Apache NiFi. In order to streamline the review of the contribution we ask you

[GitHub] nifi issue #1289: NIFI-3141: Fixed TailFile ArrayIndexOutOfBounds.

2016-12-02 Thread olegz
Github user olegz commented on the issue: https://github.com/apache/nifi/pull/1289 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, or if

[GitHub] nifi pull request #1280: NIFI-3109 Initial refactoring of TimerDrivenSchedul...

2016-11-29 Thread olegz
GitHub user olegz opened a pull request: https://github.com/apache/nifi/pull/1280 NIFI-3109 Initial refactoring of TimerDrivenSchedulingAgent Thank you for submitting a contribution to Apache NiFi. In order to streamline the review of the contribution we ask you to

[GitHub] nifi pull request #1278: NIFI-2886 Fixed the lifecycle delay

2016-11-29 Thread olegz
GitHub user olegz opened a pull request: https://github.com/apache/nifi/pull/1278 NIFI-2886 Fixed the lifecycle delay Thank you for submitting a contribution to Apache NiFi. In order to streamline the review of the contribution we ask you to ensure the following steps

[GitHub] nifi pull request #1259: Added posix guard to ignore tests on windows

2016-11-22 Thread olegz
GitHub user olegz opened a pull request: https://github.com/apache/nifi/pull/1259 Added posix guard to ignore tests on windows 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] nifi issue #1221: NIFI-3024 - Encrypt-toolkit flow.xml.gz support

2016-11-21 Thread olegz
Github user olegz commented on the issue: https://github.com/apache/nifi/pull/1221 @brosander not sure if you noticed there is a merge conflict that needs to be addressed --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub as well

[GitHub] nifi issue #1245: Nifi 3066

2016-11-21 Thread olegz
Github user olegz commented on the issue: https://github.com/apache/nifi/pull/1245 +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, or if the

[GitHub] nifi pull request #1233: NIFI-3011: Added Elasticsearch5 processors

2016-11-21 Thread olegz
Github user olegz commented on a diff in the pull request: https://github.com/apache/nifi/pull/1233#discussion_r88927684 --- Diff: nifi-nar-bundles/nifi-elasticsearch-bundle/nifi-elasticsearch-5-processors/src/main/java/org/apache/nifi/processors/elasticsearch/FetchElasticsearch5.java

[GitHub] nifi pull request #1233: NIFI-3011: Added Elasticsearch5 processors

2016-11-20 Thread olegz
Github user olegz commented on a diff in the pull request: https://github.com/apache/nifi/pull/1233#discussion_r88806585 --- Diff: nifi-nar-bundles/nifi-elasticsearch-bundle/nifi-elasticsearch-5-processors/src/main/java/org/apache/nifi/processors/elasticsearch/FetchElasticsearch5.java

  1   2   3   4   5   >