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 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 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 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 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 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/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 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 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 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 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 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 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 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 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 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 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 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 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 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 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 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 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 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 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 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 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 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 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 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 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 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 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 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 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 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 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 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 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 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 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 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 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 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 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 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 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 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 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 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 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 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 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 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 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 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 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 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 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 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 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 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 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 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 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 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 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 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 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 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 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 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 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 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 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 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 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 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 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 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 user olegz commented on the issue:
https://github.com/apache/nifi/pull/817
@trixpan sorry for the hold up. Actually I would like for @joewitt to look
at License/Notice (as you have suggested earlier) as I don't feel competent
enough as there are some things I'v
GitHub user olegz opened a pull request:
https://github.com/apache/nifi/pull/1147
NIFI-2919 improved GetFile to fail if target directory is inaccessible
Thank you for submitting a contribution to Apache NiFi.
In order to streamline the review of the contribution we ask you
Github user olegz commented on the issue:
https://github.com/apache/nifi/pull/1147
@alopresto unless you find any major issues that would require me to follow
up with more work, would you be able (before merging) to change in
_testWithUnwritableDir_ the actual directory to be ```File
Github user olegz commented on the issue:
https://github.com/apache/nifi/pull/1146
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 user olegz commented on a diff in the pull request:
https://github.com/apache/nifi/pull/1115#discussion_r84713334
--- Diff:
nifi-commons/nifi-processor-utilities/src/main/java/org/apache/nifi/processor/util/bin/BinFiles.java
---
@@ -273,25 +262,26 @@ private int
GitHub user olegz opened a pull request:
https://github.com/apache/nifi/pull/1160
NIFI-2928 added support to validate accessibility of the file system
Thank you for submitting a contribution to Apache NiFi.
In order to streamline the review of the contribution we ask you
Github user olegz commented on a diff in the pull request:
https://github.com/apache/nifi/pull/1115#discussion_r84719151
--- Diff:
nifi-commons/nifi-processor-utilities/src/main/java/org/apache/nifi/processor/util/bin/BinFiles.java
---
@@ -273,25 +262,26 @@ private int
Github user olegz commented on the issue:
https://github.com/apache/nifi/pull/1156
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 user olegz commented on the issue:
https://github.com/apache/nifi/pull/1156
@bbende While i am still reviewing and testing, I think it is worth
clarifying what per-instance really means. What I am trying to say is the fact
that property values can be changed multiple times per
Github user olegz commented on a diff in the pull request:
https://github.com/apache/nifi/pull/1156#discussion_r86161443
--- Diff:
nifi-nar-bundles/nifi-framework-bundle/nifi-framework/nifi-framework-core-api/src/main/java/org/apache/nifi/controller/AbstractConfiguredComponent.java
Github user olegz commented on a diff in the pull request:
https://github.com/apache/nifi/pull/1156#discussion_r86157718
--- Diff:
nifi-commons/nifi-utils/src/main/java/org/apache/nifi/util/file/classloader/ClassLoaderUtils.java
---
@@ -24,23 +24,57 @@
import java.net.URL
Github user olegz commented on a diff in the pull request:
https://github.com/apache/nifi/pull/1156#discussion_r86158326
--- Diff:
nifi-commons/nifi-utils/src/main/java/org/apache/nifi/util/file/classloader/ClassLoaderUtils.java
---
@@ -24,23 +24,57 @@
import java.net.URL
Github user olegz commented on a diff in the pull request:
https://github.com/apache/nifi/pull/1156#discussion_r86160430
--- Diff:
nifi-nar-bundles/nifi-framework-bundle/nifi-framework/nifi-framework-core-api/src/main/java/org/apache/nifi/controller/AbstractConfiguredComponent.java
Github user olegz commented on a diff in the pull request:
https://github.com/apache/nifi/pull/1156#discussion_r86163462
--- Diff:
nifi-nar-bundles/nifi-framework-bundle/nifi-framework/nifi-framework-core-api/src/main/java/org/apache/nifi/controller/AbstractConfiguredComponent.java
Github user olegz commented on a diff in the pull request:
https://github.com/apache/nifi/pull/1156#discussion_r86160998
--- Diff:
nifi-nar-bundles/nifi-framework-bundle/nifi-framework/nifi-framework-core-api/src/main/java/org/apache/nifi/controller/AbstractConfiguredComponent.java
Github user olegz commented on a diff in the pull request:
https://github.com/apache/nifi/pull/1156#discussion_r86164431
--- Diff:
nifi-nar-bundles/nifi-framework-bundle/nifi-framework/nifi-framework-core-api/src/main/java/org/apache/nifi/controller/AbstractConfiguredComponent.java
Github user olegz commented on a diff in the pull request:
https://github.com/apache/nifi/pull/1156#discussion_r86159055
--- Diff:
nifi-commons/nifi-utils/src/main/java/org/apache/nifi/util/file/classloader/ClassLoaderUtils.java
---
@@ -52,23 +86,29 @@ public static ClassLoader
Github user olegz commented on the issue:
https://github.com/apache/nifi/pull/1166
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
1 - 100 of 410 matches
Mail list logo