Github user bbende commented on a diff in the pull request:
https://github.com/apache/nifi/pull/2561#discussion_r175863187
--- Diff:
nifi-nar-bundles/nifi-solr-bundle/nifi-solr-processors/src/main/java/org/apache/nifi/processors/solr/PutSolrRecord.java
---
@@ -0,0 +1,351
Github user bbende commented on a diff in the pull request:
https://github.com/apache/nifi/pull/2561#discussion_r175863036
--- Diff:
nifi-nar-bundles/nifi-solr-bundle/nifi-solr-processors/src/main/java/org/apache/nifi/processors/solr/PutSolrRecord.java
---
@@ -0,0 +1,351
Github user bbende commented on the issue:
https://github.com/apache/nifi/pull/2573
Tested a bunch of scenarios involving version control with granular
restricted component permissions and everything looks good now, going to merge
---
Github user bbende commented on the issue:
https://github.com/apache/nifi/pull/2552
@markap14 do you think it makes sense to update the default
nifi-env.sh/.bat so that the environment variable is already there (defaulting
to allow) and can easily be toggled?
https
Github user bbende commented on a diff in the pull request:
https://github.com/apache/nifi/pull/2561#discussion_r175858048
--- Diff:
nifi-nar-bundles/nifi-solr-bundle/nifi-solr-processors/src/main/java/org/apache/nifi/processors/solr/SolrUtils.java
---
@@ -280,5 +291,115
GitHub user bbende opened a pull request:
https://github.com/apache/nifi-minifi/pull/119
MINIFI-447 - Adding FlowMapper and FlowRetrievalService
Thank you for submitting a contribution to Apache NiFi - MiNiFi.
In order to streamline the review of the contribution we ask you
Github user bbende commented on the issue:
https://github.com/apache/nifi/pull/2478
One other question, what do you envision people most likely do with the
output of this processor?
The reason I'm asking is because I'm debating if it makes sense to write
multiple JSON
Github user bbende commented on the issue:
https://github.com/apache/nifi/pull/2478
@bdesert Thanks for the updates, was reviewing the code again and I think
we need to change to way the `ScanHBaseResultHandler` works...
Currently it adds rows to a list in memory until bulk
GitHub user bbende opened a pull request:
https://github.com/apache/nifi/pull/2505
NIFI-4920 Skipping sensitive properties when updating component propeâ¦
â¦rties from versioned component
Thank you for submitting a contribution to Apache NiFi.
In order
Github user bbende commented on the issue:
https://github.com/apache/nifi/pull/2498
@MikeThomsen thanks for the heads up, i don't anticipate this one being
able to be merged any time soon since it will probably still be a while before
a GA 2.0.0 hbase-client is available, so we
Github user bbende commented on the issue:
https://github.com/apache/nifi/pull/2582
@jtstorck I tested this against a 2.6 Hadoop with Kerberos and also a 3.0.0
and it appears to work against both so I'm planning to merge this.
I do think we should make it as easy as possible
Github user bbende commented on the issue:
https://github.com/apache/nifi/pull/2660
Looks good, going to merge, thanks!
---
Github user bbende commented on the issue:
https://github.com/apache/nifi/pull/2660
Reviewing...
---
Github user bbende closed the pull request at:
https://github.com/apache/nifi/pull/2809
---
Github user bbende commented on the issue:
https://github.com/apache/nifi/pull/2809
Thanks @zenfenan ! will close the PR
---
Github user bbende commented on the issue:
https://github.com/apache/nifi/pull/3040
I agree with the points made by @joewitt , although this PR did make me
compare what we are doing for processors vs. controller services...
I noticed that for processors the try/catch wraps
Github user bbende commented on the issue:
https://github.com/apache/nifi/pull/3057
Ok using your test.avro going into PutORC with an AvroReader that uses
embedded schema I can get the error you showed earlier.
I also tested ConvertRecord using the AvroReader and JsonWrtier
GitHub user bbende opened a pull request:
https://github.com/apache/nifi/pull/3065
NIFI-5680 Handling trailing slashes on URLs of registry clients
Thank you for submitting a contribution to Apache NiFi.
In order to streamline the review of the contribution we ask you
Github user bbende commented on the issue:
https://github.com/apache/nifi/pull/3070
Verified this fixes the issue, looks good to me.
---
Github user bbende commented on the issue:
https://github.com/apache/nifi/pull/3057
Looks good to me, was able to verify the functionality, going to merge
---
GitHub user bbende opened a pull request:
https://github.com/apache/nifi/pull/3088
NIFI-5719 Ensuring FetchFile routes to failure if the move completionâ¦
⦠strategy can't be completed
Thank you for submitting a contribution to Apache NiFi.
In order
Github user bbende commented on a diff in the pull request:
https://github.com/apache/nifi/pull/3079#discussion_r226017591
--- Diff:
nifi-nar-bundles/nifi-parquet-bundle/nifi-parquet-processors/src/main/java/org/apache/nifi/processors/parquet/ConvertAvroToParquet.java
---
@@ -0,0
Github user bbende commented on a diff in the pull request:
https://github.com/apache/nifi/pull/3079#discussion_r226020300
--- Diff:
nifi-nar-bundles/nifi-parquet-bundle/nifi-parquet-processors/src/main/java/org/apache/nifi/processors/parquet/ConvertAvroToParquet.java
---
@@ -0,0
Github user bbende commented on a diff in the pull request:
https://github.com/apache/nifi/pull/3079#discussion_r226021329
--- Diff:
nifi-nar-bundles/nifi-parquet-bundle/nifi-parquet-processors/src/main/java/org/apache/nifi/processors/parquet/ConvertAvroToParquet.java
---
@@ -0,0
Github user bbende commented on a diff in the pull request:
https://github.com/apache/nifi/pull/3079#discussion_r226019319
--- Diff:
nifi-nar-bundles/nifi-parquet-bundle/nifi-parquet-processors/src/main/java/org/apache/nifi/processors/parquet/ConvertAvroToParquet.java
---
@@ -0,0
Github user bbende commented on a diff in the pull request:
https://github.com/apache/nifi/pull/3079#discussion_r226018942
--- Diff:
nifi-nar-bundles/nifi-parquet-bundle/nifi-parquet-processors/src/main/java/org/apache/nifi/processors/parquet/ConvertAvroToParquet.java
---
@@ -0,0
Github user bbende commented on a diff in the pull request:
https://github.com/apache/nifi/pull/3079#discussion_r226017678
--- Diff:
nifi-nar-bundles/nifi-parquet-bundle/nifi-parquet-processors/src/main/java/org/apache/nifi/processors/parquet/ConvertAvroToParquet.java
---
@@ -0,0
Github user bbende commented on the issue:
https://github.com/apache/nifi/pull/3082
Looks good, verified the functionality and will merge
---
GitHub user bbende opened a pull request:
https://github.com/apache/nifi/pull/3081
NIFI-5708 Fixing the creation of ValidationContextFactory for controlâ¦
â¦ler services so it uses the ComponentVariableRegistry and not the
file-based instance
Thank you for submitting
Github user bbende commented on the issue:
https://github.com/apache/nifi/pull/3082
Reviewing...
---
Github user bbende commented on a diff in the pull request:
https://github.com/apache/nifi/pull/3079#discussion_r227125269
--- Diff:
nifi-nar-bundles/nifi-parquet-bundle/nifi-parquet-processors/src/main/java/org/apache/nifi/processors/parquet/utils/ParquetBuilderProperties.java
Github user bbende commented on the issue:
https://github.com/apache/nifi/pull/3079
Busy with some other things at the moment, but will take a look when I get
a chance.
---
GitHub user bbende opened a pull request:
https://github.com/apache/nifi/pull/3095
NIFI-5673 Fixing connection handling in MQTT processors
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 opened a pull request:
https://github.com/apache/nifi/pull/3096
NIFI-5721 Fixing connection handling in MQTT processors
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 closed the pull request at:
https://github.com/apache/nifi/pull/3095
---
Github user bbende commented on the issue:
https://github.com/apache/nifi/pull/3079
@MikeThomsen the advantage is not having to write the parquet out to local
disk somewhere, and then have a disconnected flow where another part reads it
back in. With this approach the data stays
GitHub user bbende opened a pull request:
https://github.com/apache/nifi/pull/3104
NIFI-5740 Ensuring permissions are restored after test completes in Tâ¦
â¦estFetchFile
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/3106
Looks good, will merge
---
Github user bbende commented on a diff in the pull request:
https://github.com/apache/nifi/pull/3041#discussion_r228563247
--- Diff: nifi-nar-bundles/nifi-solr-bundle/nifi-solr-processors/pom.xml ---
@@ -147,6 +147,12 @@
2.2.1
test
Github user bbende commented on the issue:
https://github.com/apache/nifi/pull/3118
Reviewing...
---
Github user bbende commented on the issue:
https://github.com/apache/nifi/pull/3118
Looks good, will merge
---
GitHub user bbende opened a pull request:
https://github.com/apache/nifi/pull/3119
NIFI-5673 Support auto loading of new NARs
This PR introduces a new property in nifi.properties where you can specify
a directory that will be watched for new NARs (defaults ./extensions), and upon
Github user bbende commented on a diff in the pull request:
https://github.com/apache/nifi/pull/3041#discussion_r228542809
--- Diff: nifi-nar-bundles/nifi-solr-bundle/nifi-solr-processors/pom.xml ---
@@ -147,6 +147,12 @@
2.2.1
test
GitHub user bbende opened a pull request:
https://github.com/apache/nifi-registry/pull/144
NIFIREG-209 Rebuild metadata DB from FlowPersistenceProvider when empâ¦
â¦ty DB AND when instance of MetadataAwareFlowPersistenceProvider
You can merge this pull request into a Git
Github user bbende commented on the issue:
https://github.com/apache/nifi-registry/pull/144
@kevdoran @ijokarumawak what do you guys think of this?
---
Github user bbende commented on a diff in the pull request:
https://github.com/apache/nifi/pull/3119#discussion_r230134587
--- Diff:
nifi-nar-bundles/nifi-framework-bundle/nifi-framework/nifi-framework-nar-loading-utils/src/main/java/org/apache/nifi/nar/NarAutoLoaderTask.java
Github user bbende commented on the issue:
https://github.com/apache/nifi/pull/3119
@markap14 additional commit pushed that addresses your initial comments
---
Github user bbende commented on the issue:
https://github.com/apache/nifi/pull/3132
Code looks good and have been running this branch this morning and
everything seems to work as expected, so going to merge shortly, thanks!
---
Github user bbende commented on the issue:
https://github.com/apache/nifi/pull/3171
Reviewing...
---
Github user bbende commented on the issue:
https://github.com/apache/nifi/pull/3171
+1 code looks good, verified functionality, will merge
---
Github user bbende commented on the issue:
https://github.com/apache/nifi/pull/3173
Code looks good and tests pass, will merge
---
Github user bbende commented on the issue:
https://github.com/apache/nifi/pull/3136
Reviewing...
---
Github user bbende commented on the issue:
https://github.com/apache/nifi-registry/pull/146
Looks good, going to merge, thanks!
---
Github user bbende commented on the issue:
https://github.com/apache/nifi/pull/3136
Looks good, going to merge
---
Github user bbende commented on the issue:
https://github.com/apache/nifi/pull/3124
I'm good with removing the README once 1.9.0 is released, or changing it to
a link to the new toolkit docs.
---
Github user bbende commented on a diff in the pull request:
https://github.com/apache/nifi/pull/3041#discussion_r222377115
--- Diff: nifi-nar-bundles/nifi-solr-bundle/nifi-solr-client-api/pom.xml ---
@@ -0,0 +1,68 @@
+
+
+http://maven.apache.org/POM/4.0.0;
xmlns:xsi
Github user bbende commented on a diff in the pull request:
https://github.com/apache/nifi/pull/3041#discussion_r222376374
--- Diff:
nifi-nar-bundles/nifi-solr-bundle/nifi-solr-processors/src/main/java/org/apache/nifi/processors/solr/SolrProcessor.java
---
@@ -176,70 +172,11
Github user bbende commented on a diff in the pull request:
https://github.com/apache/nifi/pull/3041#discussion_r222379389
--- Diff: nifi-nar-bundles/nifi-solr-bundle/nifi-solr-processors/pom.xml ---
@@ -147,6 +147,12 @@
2.2.1
test
Github user bbende commented on the issue:
https://github.com/apache/nifi/pull/3057
@VikingK in your schema it has OItems defined as an array, but then in the
JSON OItems is not an array, its an object with a field called array. So
running with that schema and example JSON I get
Github user bbende commented on the issue:
https://github.com/apache/nifi-registry/pull/142
Reviewing..
---
GitHub user bbende opened a pull request:
https://github.com/apache/nifi-registry/pull/143
NIFIREG-201 Refactoring project structure to better isolate extensions
You can merge this pull request into a Git repository by running:
$ git pull https://github.com/bbende/nifi
Github user bbende commented on the issue:
https://github.com/apache/nifi-registry/pull/142
This looks good and I'll merge to master.
Something I noticed while reviewing this (not a result of anything in this
PR), is that the nifi-registry-extensions modules are impacted
GitHub user bbende opened a pull request:
https://github.com/apache/nifi-registry/pull/149
NIFIREG-215 Extension Bundle Improvements
I left the changes as separate commits in order to more easily review each
change on it's own, but feel free to squash upon merging.
You can merge
Github user bbende commented on a diff in the pull request:
https://github.com/apache/nifi-registry/pull/149#discussion_r241190389
--- Diff:
nifi-registry-core/nifi-registry-web-api/src/main/java/org/apache/nifi/registry/web/api/ExtensionResource.java
---
@@ -93,15 +94,25
Github user bbende commented on the issue:
https://github.com/apache/nifi-registry/pull/149
Thanks for the review, I'll look into that lastModified timestamp issue as
well.
---
Github user bbende commented on a diff in the pull request:
https://github.com/apache/nifi-registry/pull/149#discussion_r241191410
--- Diff:
nifi-registry-core/nifi-registry-data-model/src/main/java/org/apache/nifi/registry/bucket/Bucket.java
---
@@ -41,6 +41,8
Github user bbende commented on the issue:
https://github.com/apache/nifi-registry/pull/149
@kevdoran I pushed up a commit to update the filter params documentation. I
decided not to do anything with the lastModified value on the bundle table
because technically nothing on that row
Github user bbende commented on a diff in the pull request:
https://github.com/apache/nifi-maven/pull/7#discussion_r240374218
--- Diff:
src/main/java/org/apache/nifi/extension/definition/extraction/ExtensionDefinitionFactory.java
---
@@ -0,0 +1,247 @@
+/*
+ * Licensed
Github user bbende commented on a diff in the pull request:
https://github.com/apache/nifi-maven/pull/7#discussion_r240314647
--- Diff: src/main/java/org/apache/nifi/XmlDefinitionWriter.java ---
@@ -0,0 +1,165 @@
+/*
+ * Licensed to the Apache Software Foundation (ASF
Github user bbende commented on a diff in the pull request:
https://github.com/apache/nifi-maven/pull/7#discussion_r240308506
--- Diff: src/main/java/org/apache/nifi/NarMojo.java ---
@@ -426,12 +479,260 @@
protected boolean cloneDuringInstanceClassLoading
Github user bbende commented on a diff in the pull request:
https://github.com/apache/nifi-maven/pull/7#discussion_r240314701
--- Diff: src/main/java/org/apache/nifi/PropertiesDefinitionWriter.java ---
@@ -0,0 +1,75 @@
+/*
+ * Licensed to the Apache Software Foundation (ASF
Github user bbende commented on a diff in the pull request:
https://github.com/apache/nifi-maven/pull/7#discussion_r240310495
--- Diff: src/main/java/org/apache/nifi/NarMojo.java ---
@@ -426,12 +479,260 @@
protected boolean cloneDuringInstanceClassLoading
Github user bbende commented on the issue:
https://github.com/apache/nifi/pull/3192
Will review...
---
Github user bbende commented on a diff in the pull request:
https://github.com/apache/nifi/pull/3192#discussion_r237982934
--- Diff:
nifi-api/src/main/java/org/apache/nifi/documentation/AbstractDocumentationWriter.java
---
@@ -0,0 +1,301 @@
+/*
+ * Licensed to the Apache
Github user bbende commented on a diff in the pull request:
https://github.com/apache/nifi/pull/3192#discussion_r237974350
--- Diff:
nifi-nar-bundles/nifi-standard-services/nifi-standard-services-api-nar/pom.xml
---
@@ -26,6 +26,12 @@
true
Github user bbende commented on a diff in the pull request:
https://github.com/apache/nifi/pull/3192#discussion_r237917798
--- Diff:
nifi-api/src/main/java/org/apache/nifi/documentation/AbstractDocumentationWriter.java
---
@@ -0,0 +1,301 @@
+/*
+ * Licensed to the Apache
Github user bbende commented on the issue:
https://github.com/apache/nifi-registry/pull/148
Just pushed a new commit that allows the client to specify a SHA-256 when
uploading the bundle. If they specify one and it doesn't match what the server
calculates, then it is rejected
Github user bbende commented on the issue:
https://github.com/apache/nifi/pull/3202
Will review...
---
GitHub user bbende opened a pull request:
https://github.com/apache/nifi/pull/3211
NIFI-5884 Bumping hbase-client version from 1.1.2 to 1.1.13
Thank you for submitting a contribution to Apache NiFi.
In order to streamline the review of the contribution we ask you
GitHub user bbende opened a pull request:
https://github.com/apache/nifi/pull/3206
NIFI-5877 Protecting against null values when obtaining master address
Thank you for submitting a contribution to Apache NiFi.
In order to streamline the review of the contribution we ask you
Github user bbende commented on the issue:
https://github.com/apache/nifi/pull/3192
LGTM, going to merge, thanks!
---
Github user bbende commented on the issue:
https://github.com/apache/nifi/pull/3202
Code looks good, verified I can see the debug logging when enabled, going
to merge
---
Github user bbende commented on a diff in the pull request:
https://github.com/apache/nifi-registry/pull/148#discussion_r236850477
--- Diff:
nifi-registry-core/nifi-registry-web-api/src/main/java/org/apache/nifi/registry/web/api/ExtensionRepositoryResource.java
---
@@ -0,0 +1,378
Github user bbende commented on a diff in the pull request:
https://github.com/apache/nifi-registry/pull/148#discussion_r236849392
--- Diff:
nifi-registry-core/nifi-registry-data-model/src/main/java/org/apache/nifi/registry/extension/ExtensionBundleVersionMetadata.java
Github user bbende commented on a diff in the pull request:
https://github.com/apache/nifi-registry/pull/148#discussion_r236853819
--- Diff:
nifi-registry-core/nifi-registry-data-model/src/main/java/org/apache/nifi/registry/extension/ExtensionBundleVersionMetadata.java
Github user bbende commented on a diff in the pull request:
https://github.com/apache/nifi-registry/pull/148#discussion_r236853713
--- Diff:
nifi-registry-core/nifi-registry-data-model/src/main/java/org/apache/nifi/registry/extension/ExtensionBundleVersionMetadata.java
Github user bbende commented on a diff in the pull request:
https://github.com/apache/nifi-registry/pull/148#discussion_r236849686
--- Diff:
nifi-registry-core/nifi-registry-framework/src/main/resources/db/migration/V3__AddExtensions.sql
---
@@ -0,0 +1,62 @@
+-- Licensed
Github user bbende commented on a diff in the pull request:
https://github.com/apache/nifi-registry/pull/148#discussion_r236852286
--- Diff:
nifi-registry-core/nifi-registry-data-model/src/main/java/org/apache/nifi/registry/extension/ExtensionBundleVersionMetadata.java
Github user bbende commented on a diff in the pull request:
https://github.com/apache/nifi-registry/pull/148#discussion_r236849819
--- Diff:
nifi-registry-core/nifi-registry-framework/src/main/java/org/apache/nifi/registry/service/extension/StandardExtensionService.java
Github user bbende commented on a diff in the pull request:
https://github.com/apache/nifi-registry/pull/148#discussion_r236865785
--- Diff:
nifi-registry-core/nifi-registry-data-model/src/main/java/org/apache/nifi/registry/extension/ExtensionBundleVersionMetadata.java
GitHub user bbende opened a pull request:
https://github.com/apache/nifi-registry/pull/148
NIFIREG-211 Initial work for adding extenion bundles to NiFi Registry
This PR includes the foundational work for adding extension bundles as a
new type of versioned item. With this PR
Github user bbende commented on the issue:
https://github.com/apache/nifi-registry/pull/148
All versioned items (flows and now bundles) live in a bucket, and a bucket
is where security policies are applied. So each bucket can be kind of like a
mini extension repo, there could
Github user bbende commented on the issue:
https://github.com/apache/nifi-registry/pull/148
Before releasing any of this we would definitely have sections in the user
guide and admin guide related to extension bundles, but we can't really write
all that until the work is done
Github user bbende commented on a diff in the pull request:
https://github.com/apache/nifi-registry/pull/148#discussion_r237093889
--- Diff:
nifi-registry-core/nifi-registry-data-model/src/main/java/org/apache/nifi/registry/extension/ExtensionBundleVersionMetadata.java
Github user bbende commented on the issue:
https://github.com/apache/nifi-registry/pull/148
@kevdoran just pushed a commit that addresses the review comments, let me
know if you find anything else. As an FYI, I did try to change that swagger
annotation for LinkableEntity
GitHub user bbende opened a pull request:
https://github.com/apache/nifi-registry/pull/140
NIFIREG-199 - Adding interfaces to represent configurable components â¦
â¦and extension components
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-registry/pull/139
+1 looks good and will merge, thanks!
---
801 - 897 of 897 matches
Mail list logo