Github user bbende commented on the issue:
https://github.com/apache/nifi/pull/1004
@trixpan sure...
1) mvn clean install on this branch
2) In some other directory:
`mvn archetype:generate -DarchetypeGroupId=org.apache.nifi
-DarchetypeArtifactId=nifi
Github user bbende commented on a diff in the pull request:
https://github.com/apache/nifi/pull/1097#discussion_r82078303
--- Diff:
nifi-nar-bundles/nifi-kafka-bundle/nifi-kafka-0-10-processors/src/main/java/org/apache/nifi/processors/kafka/pubsub/PublisherLease.java
---
@@ -0,0
Github user bbende commented on the issue:
https://github.com/apache/nifi/pull/1075
Tried this out on against an HDP 2.4 sandbox and got an exception that
boiled down to:
```
Caused by: java.io.IOException: No FileSystem for scheme: hdfs
Github user bbende commented on the issue:
https://github.com/apache/nifi/pull/1097
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 bbende commented on a diff in the pull request:
https://github.com/apache/nifi/pull/1097#discussion_r82070115
--- Diff:
nifi-nar-bundles/nifi-kafka-bundle/nifi-kafka-0-10-processors/src/main/java/org/apache/nifi/processors/kafka/pubsub/PublishKafka_0_10.java
Github user bbende commented on the issue:
https://github.com/apache/nifi/pull/1038
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 bbende commented on the issue:
https://github.com/apache/nifi/pull/1029
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 bbende commented on the issue:
https://github.com/apache/nifi/pull/1028
Minor comment, i noticed viewing downstream connections on input ports, the
component name shows "Object object" rather than the name of the port.
---
If your project is set up for it, you
Github user bbende commented on the issue:
https://github.com/apache/nifi/pull/1028
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 bbende commented on the issue:
https://github.com/apache/nifi/pull/513
@rtempleton can you close this PR since it was included in PR 542? 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 bbende commented on the issue:
https://github.com/apache/nifi/pull/1010
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 bbende commented on the issue:
https://github.com/apache/nifi/pull/1010
+1 looks good, verified this fixes the problem when using identity mapping
with RAW site-to-site, thanks!
---
If your project is set up for it, you can reply to this email and have your
reply appear
Github user bbende commented on the issue:
https://github.com/apache/nifi/pull/1047
@fidget83 thank you for the contribution, can you please create a JIRA that
describes the problem this is trying to fix? and then update this PR to include
the JIRA number. Thanks.
---
If your
Github user bbende commented on the issue:
https://github.com/apache/nifi/pull/879
Looks good, will merge to master, 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 project does not have this feature
GitHub user bbende opened a pull request:
https://github.com/apache/nifi/pull/887
NIFI-2574 Fixing NPE when using kerberos keytab location from contextâ¦
â¦s, and cleaning up hadoop/hbase/hive kerberos variables
You can merge this pull request into a Git repository by running
Github user bbende closed the pull request at:
https://github.com/apache/nifi/pull/887
---
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 bbende commented on the issue:
https://github.com/apache/nifi/pull/892
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 bbende commented on the issue:
https://github.com/apache/nifi/pull/892
Looks good, verified that I can see the controller level bulletins when I
have access to the reporting task, and can't when I remove access, will merge
to master
---
If your project is set up
GitHub user bbende opened a pull request:
https://github.com/apache/nifi/pull/946
NIFI-2664 Moving System.setProperty for krb5.conf to NiFi startup, anâ¦
â¦d removing conflicting property from KerberosProvider config
Rather than add code to all of the Kafka processors
Github user bbende commented on a diff in the pull request:
https://github.com/apache/nifi/pull/834#discussion_r75917246
--- Diff:
nifi-toolkit/nifi-toolkit-encrypt-config/src/main/groovy/org/apache/nifi/properties/ConfigEncryptionTool.groovy
---
@@ -0,0 +1,540
Github user bbende commented on the issue:
https://github.com/apache/nifi/pull/834
Ok that is not working for me... If I run the tool a second time using the
same password, even without changing anything I get:
```
./nifi-toolkit-1.0.0-SNAPSHOT/bin/encrypt-config.sh -b
nifi
Github user bbende commented on the issue:
https://github.com/apache/nifi/pull/922
Looks good, verified this work with array and map columns, will merge to
master.
---
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 bbende commented on the issue:
https://github.com/apache/nifi/pull/834
Performed a couple of test runs and everything is working nicely, thanks
for making all the updates.
My last comment is that we may need to add something to the toolkit NOTICE
for Grooby
Github user bbende commented on the issue:
https://github.com/apache/nifi/pull/834
Running the latest commit, nifi.properties ends up with:
```
nifi.sensitive.props.key.protected=aes/gcm/128
nifi.sensitive.props.key.protected=
```
The app then fails to start because
Github user bbende commented on the issue:
https://github.com/apache/nifi/pull/902
Looks good, will merge to master.
---
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 bbende commented on a diff in the pull request:
https://github.com/apache/nifi/pull/914#discussion_r75858286
--- Diff:
nifi-nar-bundles/nifi-hbase-bundle/nifi-hbase-processors/src/main/java/org/apache/nifi/hbase/AbstractPutHBase.java
---
@@ -170,7 +193,7 @@ public
Github user bbende commented on a diff in the pull request:
https://github.com/apache/nifi/pull/834#discussion_r75733078
--- Diff:
nifi-nar-bundles/nifi-framework-bundle/nifi-framework/nifi-properties-loader/src/main/java/org/apache/nifi/properties/AESSensitivePropertyProvider.java
Github user bbende commented on a diff in the pull request:
https://github.com/apache/nifi/pull/834#discussion_r75728485
--- Diff: nifi-commons/nifi-properties/pom.xml ---
@@ -21,4 +21,10 @@
1.0.0-SNAPSHOT
nifi-properties
Github user bbende commented on the issue:
https://github.com/apache/nifi/pull/834
General comment that may be outside the scope of this effort, but have we
thought about being able to undo the protection using the toolkit? Basically
once you have run the tool, if you overwrote your
Github user bbende commented on a diff in the pull request:
https://github.com/apache/nifi/pull/834#discussion_r75874508
--- Diff:
nifi-toolkit/nifi-toolkit-encrypt-config/src/main/groovy/org/apache/nifi/properties/ConfigEncryptionTool.groovy
---
@@ -0,0 +1,540
Github user bbende commented on a diff in the pull request:
https://github.com/apache/nifi/pull/834#discussion_r75870493
--- Diff:
nifi-nar-bundles/nifi-framework-bundle/nifi-framework/nifi-resources/src/main/resources/conf/nifi.properties
---
@@ -131,8 +131,10
Github user bbende commented on a diff in the pull request:
https://github.com/apache/nifi/pull/834#discussion_r75871505
--- Diff:
nifi-toolkit/nifi-toolkit-encrypt-config/src/test/groovy/org/apache/nifi/properties/ConfigEncryptionToolTest.groovy
---
@@ -0,0 +1,1225
Github user bbende commented on a diff in the pull request:
https://github.com/apache/nifi/pull/914#discussion_r75858615
--- Diff:
nifi-nar-bundles/nifi-hbase-bundle/nifi-hbase-processors/src/main/java/org/apache/nifi/hbase/PutHBaseCell.java
---
@@ -82,8 +84,19 @@ public void
Github user bbende commented on the issue:
https://github.com/apache/nifi/pull/914
Overall this looks good, thanks Andrew!
One overall comment is that we now have several places where we create a
new string from a byte[] like `new String(byte[])` and I think we want
GitHub user bbende opened a pull request:
https://github.com/apache/nifi/pull/1088
NIFI-2841 Refactoring logic in SplitAvro RecordSplitter to avoid makiâ¦
â¦ng two calls in a row to reader.hasNext()
You can merge this pull request into a Git repository by running:
$ git pull
Github user bbende commented on the issue:
https://github.com/apache/nifi/pull/1057
@trixpan @pvillard31 since the three changes in question are not part of
the public API we should be ok to change them as long as it is done on a second
number version change (1.1.0 and not 1.0.1
Github user bbende commented on the issue:
https://github.com/apache/nifi/pull/1057
@pvillard31 I agree, lets remove the deprecated versions and just have the
corrected versions, I'm a +1 after that change, and thanks to @trixpan for
putting this together, didn't realize how many
Github user bbende commented on a diff in the pull request:
https://github.com/apache/nifi/pull/1156#discussion_r86839408
--- Diff:
nifi-nar-bundles/nifi-standard-services/nifi-hbase-client-service-api/src/main/java/org/apache/nifi/hbase/HBaseClientService.java
---
@@ -67,6
Github user bbende commented on a diff in the pull request:
https://github.com/apache/nifi/pull/1156#discussion_r86825589
--- Diff:
nifi-nar-bundles/nifi-standard-services/nifi-hbase-client-service-api/src/main/java/org/apache/nifi/hbase/HBaseClientService.java
---
@@ -67,6
Github user bbende commented on a diff in the pull request:
https://github.com/apache/nifi/pull/1156#discussion_r86839525
--- Diff:
nifi-nar-bundles/nifi-standard-services/nifi-hbase-client-service-api/src/main/java/org/apache/nifi/hbase/HBaseClientService.java
---
@@ -67,6
Github user bbende commented on a diff in the pull request:
https://github.com/apache/nifi/pull/1156#discussion_r86822779
--- Diff:
nifi-nar-bundles/nifi-standard-services/nifi-hbase-client-service-api/src/main/java/org/apache/nifi/hbase/HBaseClientService.java
---
@@ -67,6
Github user bbende commented on the issue:
https://github.com/apache/nifi/pull/1156
@olegz cool thanks! regarding the per-instance vs. per-lifecycle, in this
first pass we are basically loading the resources at the time the properties
are applied in order to be available during
Github user bbende commented on the issue:
https://github.com/apache/nifi/pull/1156
@olegz just pushed up a commit with some changes from the last round of
feedback (renaming jarFile, fixing empty catch block, fixing empty
IllegalArgumentException).
Have you had a chance
Github user bbende commented on the issue:
https://github.com/apache/nifi/pull/1150
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 bbende opened a pull request:
https://github.com/apache/nifi/pull/1156
NIFI-2909 and NIFI-1712 Per-Instance ClassLoading
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 bbende commented on the issue:
https://github.com/apache/nifi/pull/1148
Thank you for contributing. In addition to adding the
expressionLanguageSupported flag on the property descriptors, we would also
need to go through the code and find every place where the properties
Github user bbende commented on the issue:
https://github.com/apache/nifi/pull/1252
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 bbende commented on the issue:
https://github.com/apache/nifi/pull/1256
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 bbende commented on the issue:
https://github.com/apache/nifi/pull/1219
PR updated to address feedback from @olegz, and incorporated licensing
patch posted by @joewitt in JIRA
---
If your project is set up for it, you can reply to this email and have your
reply appear
Github user bbende commented on a diff in the pull request:
https://github.com/apache/nifi/pull/1219#discussion_r88030428
--- Diff:
nifi-nar-bundles/nifi-framework-bundle/nifi-framework/nifi-nar-utils/src/main/java/org/apache/nifi/nar/InstanceClassLoader.java
---
@@ -88,7 +89,7
Github user bbende commented on a diff in the pull request:
https://github.com/apache/nifi/pull/1219#discussion_r88030680
--- Diff:
nifi-nar-bundles/nifi-hadoop-bundle/nifi-hdfs-processors/src/main/java/org/apache/nifi/processors/hadoop/PutHDFS.java
---
@@ -205,177 +207,186
Github user bbende commented on a diff in the pull request:
https://github.com/apache/nifi/pull/1219#discussion_r88032664
--- Diff:
nifi-nar-bundles/nifi-framework-bundle/nifi-framework/nifi-framework-core/src/main/java/org/apache/nifi/groups/StandardProcessGroup.java
GitHub user bbende opened a pull request:
https://github.com/apache/nifi/pull/1232
NIFI-2909 Fix logic in AbstractConfiguredComponent setProperties()
for setting classpath resources in the InstanceClassLoader.
Before if you updated other properties that were not classpath
GitHub user bbende opened a pull request:
https://github.com/apache/nifi/pull/1219
NIFI-3003 Upgrading hadoop.version to 2.7.3 and fixing TDE issue withâ¦
Thank you for submitting a contribution to Apache NiFi.
In order to streamline the review of the contribution we ask
Github user bbende commented on a diff in the pull request:
https://github.com/apache/nifi/pull/1156#discussion_r86212571
--- 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 bbende commented on a diff in the pull request:
https://github.com/apache/nifi/pull/1156#discussion_r86212641
--- 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 bbende commented on a diff in the pull request:
https://github.com/apache/nifi/pull/1156#discussion_r86213356
--- Diff:
nifi-nar-bundles/nifi-framework-bundle/nifi-framework/nifi-framework-core-api/src/main/java/org/apache/nifi/controller/AbstractConfiguredComponent.java
Github user bbende commented on a diff in the pull request:
https://github.com/apache/nifi/pull/1156#discussion_r86212943
--- Diff:
nifi-nar-bundles/nifi-framework-bundle/nifi-framework/nifi-framework-core-api/src/main/java/org/apache/nifi/controller/AbstractConfiguredComponent.java
Github user bbende commented on a diff in the pull request:
https://github.com/apache/nifi/pull/1156#discussion_r86212718
--- 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 bbende commented on a diff in the pull request:
https://github.com/apache/nifi/pull/1156#discussion_r86213277
--- Diff:
nifi-nar-bundles/nifi-framework-bundle/nifi-framework/nifi-framework-core-api/src/main/java/org/apache/nifi/controller/AbstractConfiguredComponent.java
Github user bbende commented on a diff in the pull request:
https://github.com/apache/nifi/pull/1156#discussion_r86216168
--- Diff:
nifi-nar-bundles/nifi-framework-bundle/nifi-framework/nifi-framework-core-api/src/main/java/org/apache/nifi/controller/AbstractConfiguredComponent.java
Github user bbende commented on a diff in the pull request:
https://github.com/apache/nifi/pull/1156#discussion_r86226932
--- 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 bbende commented on the issue:
https://github.com/apache/nifi/pull/1156
@olegz thanks for reviewing, I posted a couple of replies to the inline
comments.
I also wanted to mention that after submitting this PR, I realized that a
slight modification to the approach
Github user bbende commented on the issue:
https://github.com/apache/nifi/pull/1153
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 bbende commented on the issue:
https://github.com/apache/nifi/pull/1156
@olegz I pushed up a second commit that with the change that I described
above, and also some of the changes from your feedback. Unfortunately I had
already rebased my branch before you had started
Github user bbende commented on the issue:
https://github.com/apache/nifi/pull/1155
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 bbende commented on the issue:
https://github.com/apache/nifi/pull/1277
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 bbende commented on the issue:
https://github.com/apache/nifi/pull/1307
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 bbende commented on the issue:
https://github.com/apache/nifi/pull/1307
@randerzander I tested this out and I think we have to move the addition of
the attributes to inside the loop where the flow files are being created. If we
do it after the loop, then session.transfer
Github user bbende commented on the issue:
https://github.com/apache/nifi/pull/1307
Thanks for pointing that out, let me give it a try with minif-cpp and see
if I can reproduce. I was hoping that by putting the attributes in
AbstractFlowFileServerProtocol it would automatically work
Github user bbende commented on the issue:
https://github.com/apache/nifi/pull/1307
@randerzander I haven't been able to reproduce this yet. Running a simple
GetFile -> RPG on minifi-cpp to a regular NiFi with InputPort -> LogAttribute,
I am seeing the two attributes come t
Github user bbende commented on a diff in the pull request:
https://github.com/apache/nifi/pull/1329#discussion_r92809224
--- Diff:
nifi-nar-bundles/nifi-standard-bundle/nifi-standard-processors/src/main/java/org/apache/nifi/processors/standard/Wait.java
---
@@ -0,0 +1,266
Github user bbende commented on a diff in the pull request:
https://github.com/apache/nifi/pull/1329#discussion_r92808940
--- Diff:
nifi-nar-bundles/nifi-standard-bundle/nifi-standard-processors/src/main/java/org/apache/nifi/processors/standard/Notify.java
---
@@ -0,0 +1,191
Github user bbende commented on a diff in the pull request:
https://github.com/apache/nifi/pull/1329#discussion_r92815826
--- Diff:
nifi-nar-bundles/nifi-standard-bundle/nifi-standard-processors/src/main/java/org/apache/nifi/processors/standard/Wait.java
---
@@ -0,0 +1,266
Github user bbende commented on the issue:
https://github.com/apache/nifi/pull/1317
Looks good, will 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
enabled and wishes so
Github user bbende commented on the issue:
https://github.com/apache/nifi/pull/1329
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 bbende commented on the issue:
https://github.com/apache/nifi/pull/1329
That makes sense, I think we can add that functionality later. I created
this JIRA to track the changes for the DistributedMapCacheClient
https://issues.apache.org/jira/browse/NIFI-3214, then we can
Github user bbende commented on the issue:
https://github.com/apache/nifi/pull/1307
@randerzander I wrote a simple Java main method with the following:
```
URI uri = new URI("nifi://techops_web-service.dev_1.techops_dev:42626");
System.out.println("Host:
GitHub user bbende opened a pull request:
https://github.com/apache/nifi/pull/1338
NIFI-3220 Add support for basic auth to Solr processors
Also made the Solr Location support expression location so it can be
paramterized with the variable registry.
You can merge this pull request
GitHub user bbende opened a pull request:
https://github.com/apache/nifi/pull/1320
NIFI-2585 Add attributes to track s2s host and port
I left the three commits to show the history of how this evolved. We can
squash my two commits when merging, but I wanted Randy to get credit
Github user bbende commented on the issue:
https://github.com/apache/nifi/pull/1411
I was able to resolve the checkstyle issues and folded it into your commit,
so I'm a +1 and will merge it master, thanks!
---
If your project is set up for it, you can reply to this email and have
Github user bbende commented on the issue:
https://github.com/apache/nifi/pull/1411
Looks like a checkstyle failure when I ran the contrib-check:
[WARNING]
src/main/java/org/apache/nifi/groups/StandardProcessGroup.java[621] (regexp)
RegexpSinglelineJava: Line has trailing
Github user bbende commented on the issue:
https://github.com/apache/nifi/pull/1287
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 bbende commented on the issue:
https://github.com/apache/nifi/pull/1285
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 bbende commented on the issue:
https://github.com/apache/nifi/pull/1360
+1 Tested this out with Ambari and was able to run two instances of the
reporting task, one for all of NiFi and one for a specific process group, then
created a second dashboard in Grafana
GitHub user bbende opened a pull request:
https://github.com/apache/nifi/pull/1404
NIFI-3299 Adding example of sensitive property key migration to the aâ¦
â¦dmin guide
Thank you for submitting a contribution to Apache NiFi.
In order to streamline the review
Github user bbende commented on the issue:
https://github.com/apache/nifi/pull/1361
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 bbende commented on the issue:
https://github.com/apache/nifi/pull/1352
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 bbende commented on the issue:
https://github.com/apache/nifi/pull/1352
+1 Looks good, I'll get this merged in, 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 project does not have
Github user bbende commented on the issue:
https://github.com/apache/nifi/pull/1361
+1 looks good now, will merge to master, 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 project does not have
Github user bbende commented on the issue:
https://github.com/apache/nifi/pull/1309
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 bbende commented on the issue:
https://github.com/apache/nifi/pull/1309
+1 Verified I now get the more descriptive messages when doing things like
deleting a processor referencing a controller service that I don't have access
to. I'll merge to master.
---
If your
Github user bbende commented on a diff in the pull request:
https://github.com/apache/nifi/pull/1383#discussion_r94950985
--- Diff:
nifi-nar-bundles/nifi-hadoop-bundle/nifi-hdfs-processors/src/main/java/org/apache/nifi/processors/hadoop/ListHDFS.java
---
@@ -176,7 +176,7
Github user bbende commented on the issue:
https://github.com/apache/nifi/pull/1342
@ijokarumawak thanks for digging in to this! I closed the other PR.
Everything you described makes sense...
I'm on the fence about the attribute names... lets say we have MiNiFi ->
GitHub user bbende opened a pull request:
https://github.com/apache/nifi/pull/1349
NIFI-1784 Initial commit for FetchHBaseRow processor
This processor takes an incoming flow file to trigger execution, and the
table name, row id, and columns (optional) can all come from flow file
Github user bbende commented on the issue:
https://github.com/apache/nifi/pull/1342
Thanks @ijokarumawak !
I was testing this and I noticed that whenever I do a push model from
nifi-1 to nifi-2, the value of s2s.hostname on the flow file in nifi-2 ends up
being the IP
Github user bbende closed the pull request at:
https://github.com/apache/nifi/pull/1357
---
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 bbende commented on the issue:
https://github.com/apache/nifi/pull/1342
I think the issue was that I had left nifi.remote.input.host blank on both
instances. So in the push model it was getting the ip address of the incoming
connection which was my laptop's internal IP
GitHub user bbende opened a pull request:
https://github.com/apache/nifi/pull/1357
NIFI-3250 Fixing logic in StandardFlowFileQueue when migrating flow fâ¦
â¦iles to the active queue
You can merge this pull request into a Git repository by running:
$ git pull https
Github user bbende commented on the issue:
https://github.com/apache/nifi/pull/1338
@ijokarumawak yes we should definitely create another JIRA to convert
GetSolr to use state management, I keep forgetting that it doesn't do that
already.
If you need any help with the basic
101 - 200 of 897 matches
Mail list logo